Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Button UI #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

FireController1847
Copy link

I think most people can agree that right now the button within the road tool seems out of place and odd. Well I'm here to fix that!

I changed the tooltip to just say the mod name. This is for a couple reasons: 1) the tooltip is supposed to be a very very very short (like 3 words short) descriptor of what the button is. For example, the straight road says "Straight Road", curved says "Curved Road," the advisor says "Advisor," and so on. 2) there's no reason to display the mod version in the tooltip. If we need to find out, we should look at the Content Manager where every other mod's version is. 3) it was just really long. We don't need instructions in the tooltip; those can be in the menu itself or on the steam workshop page.

I removed the text of the button and replaced it with the icon of the selected option. This makes sense for various reasons. The major ones being: 1) consistency; it feels way more consistent with the other buttons and really fits in well, 2) appearance; imho, it looks a lot better, and 3) usability; it's quick and easy to identify if you've got any settings enabled the next time you go to build a road. Just glance and you recognize the icon, instead of having to read tiny text.

Hope this gets merged, I'd love to see these improvements implemented :)

P.S. I did not up the version, in case you'd want to do it.

* Changed the tooltip because we want readability. We want to make it quick and easy to know what the button is. We don't care about the version -- that's in the content manager, nor do we care about instructions, that's in the menu itself.
* Improved the UI of the button to be more consistent with the buttons around it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant