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

Overhauled icons & adjusted UI #1761

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

Conversation

mrmaffen
Copy link

I have noticed that the icons used in TMPE are a bit all over the place. Some had thin white outlines, others didn't. Some were quite low-res, others weren't. The overall style was also not consistent imo. White icons with black outlines, black icons with white outlines etc.
The button background textures also didn't match the C:S ones. I have tried to match the vanilla style as closely as possible, however I haven't yet added proper "active"-states. Normally the button foreground icons should switch to a blue posterized version. This could be done in the future, if somebody really cares to do it. However I think it already looks quite neat.
I have also discarded the less saturated "normal"-states, since all they did was make the buttons feel washed out and again decrease legibility. "active" and "normal" states are identical fore foreground pngs right now. The background png is enough to distinguish between button states. However it would make sense to get rid of those active pngs since it's just duplicated data, but simply removing them resulted in a compilation error, which I didn't want to hunt down. Maybe somebody else knows what to do.

I have also changed the main menu button. The background has lost the crown, which wasn't legible. Instead I have given the crown to the traffic light itself, while removing the v11 text. The result is a recognizable, clean icon. At least in my opinion.

I have removed the version labels, since most users don't really care about the version number, except for when they've just updated maybe, or when checking if they're actually up-to-date. This can still easily be done in the Mod Manager or in Skyve. I think this approach is cleaner and matches user expectations.

Screenshots

MainMenu before:
cities_tmpe_before_main
MainMenu after:
cities_tmpe_after_main


RoadSelectionPanel before:
cities_tmpe_before_road
RoadSelectionPanel after:
cities_tmpe_after_road

Just wanted to add: This mod project is an absolute A+. I literally cannot play C:S without it and haven't for a long time. Good job. I hope I can contribute my part with this PR.

The version label doesn't have to be this prominent. Most users don't care about it during normal usage. On the few occasions where it is important, the user can easily look it up in their mod manager or Skyve.
The freed up screen space can now be used to spell out TM as Traffic Manager. This is clearer and avoids confusion, especially for new users. However I'd argue, that it even would make sense to get rid of the title label entirely.
Other than that, the "?"-button has been moved to the top-right corner and decreased in size a bit.
@kvakvs
Copy link
Collaborator

kvakvs commented Dec 19, 2023

Have a look at #1525
There was so much discussion and so many problems found on the way, before i lost steam.
Also i stored original textures in vector format, SVG (from Inkscape), where have you stored the originals? Its not there.
P.S. Users do care about version label it just can be very very small, barely legible.

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.

2 participants