search

Tomivix / hpa

First release (v0.9.9) is available here: https://github.com/Tomivix/hpa/raw/master/rel/HPA_Studio_0.9.9_%5B576235A3%5D.jar
Other
5 stars 2 forks source link

(Yet another) GUI updates #24

Closed Aicedosh closed 7 years ago

Aicedosh commented 7 years ago

-Added address label to memory cells with labels (temporary solution) -Using images made by @mrwasp (slightly modified) as buttons -Scaling window size if necessary (for smaller displays)

mrwasp commented 7 years ago

Looks good, but why hovered is red?

Aicedosh commented 7 years ago

Colors and overall look of buttons are up to you @mrwasp. I simply desatureted disabled one and used the old graphics for hovered over.

Tomivix commented 7 years ago

Looks good to me but what's up with the colors @Aicedosh ? Aren't we gonna use the red-styled images for anything? I thought we will have grayed-out-style for inactive, red-styled for acticve and green-styled for hovered (or the other way around) or am I missing something here?

Aicedosh commented 7 years ago

Together with @mrwasp we changed the concept so that the buttons look more reasonable. I honestly don't see why we'd use red image as hovered over (Also are you testing the version from today?)

Tomivix commented 7 years ago

I've checked both versions (from yesterday and from today). Well if it's as you say and you're both okay with how things are now (looking at the conversation flow I wasn't sure about @mrwasp opinion) then I guess that puts me in minority. Anyways good job @Aicedosh and I'm going to merge it in a while.

mrwasp commented 7 years ago

I think brighter green is better than red, red associated with 'something wrong'.