Debugger MemoryWidget,: Move "Set value from file" to input combo box…

[TryTwo]

…. Add valid address check before writing.

For the UI it's better as an input type. For coding, it's bit of an outlier, but doesn't require many changes. You need to click the Set Value button after selecting "Set value from file".

[AdmiralCurtiss]

FWIW I think this is not good from a UX perspective, I'd rather keep the separate button.

[TryTwo]

it just seems like an infrequently used (but quite useful) feature for where the button is placed.

That and memory dumps feel more suitable as some kind of menu item more than buttons, but we don't have any menus.

[AdmiralCurtiss]

Yeah, a menu or something would be a much better place to put those buttons.

[TryTwo]

I figured out how to add a mini menubar. Would this be desirable? If we wanted to add more sidebar widgets we could use a view menu to show/hide stuff.

[AdmiralCurtiss]

Yeah, I think that's much preferable to both the current state and the combo box in this PR.