-
Notifications
You must be signed in to change notification settings - Fork 85
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
[Draft] High resolution UI textures #1525
base: master
Are you sure you want to change the base?
[Draft] High resolution UI textures #1525
Conversation
…n up to 128px and drawn in vector
Regarding the traffic light icon, details can be found in #71 Maybe in place of the I think icon for priority signs could be simplified to just the Yield icon as it's most globally recognised (even US has Yield signs). Could some of the tools take their icon from the currently selected signage theme?
The lane arrows icon always struck me as being too abstract. Could we maybe have a themeable icon for that (as it varies from country to country)? As the lane arrows IRL are stretched to make them work from driver perspective, we'd need to un-stretch them to fit icon better. For clear traffic & toggle despawn, is it worth taking this opportunity to rethink how we present those to users (particularly new users)? Possible icons for clear traffic (would need color inverting, and border adding):
As for toggle despawn, from user perpsective it's more about control over size of traffic jams. An exclamation mark doesn't really convey anything about traffic jams. Maybe we could use something like:
Additionally, any icons we use should be detailed on the credits page in wiki: https://github.com/CitiesSkylinesMods/TMPE/wiki/Credits |
Let's split your huge comment into things which are in the scope, and things which require extra programming and big brain (note: i have no big brain most of the time) Updating the pull request text with checklist... |
For button hovered background, what about just a blue circle (will be visibly brighter than green default circle, and will create better distinction between hovered vs. active button). |
Adding all other restrictions (ped crossing, enter blocked junction, uturn, left on red, right on red, lane change) to the Fallback theme in #1523 now they can be overridden and used by the main menu, when i write that piece of code. |
Not sure. IMO it would be too much to fit in to an icon and still quite confusing. What about the 'allow despawn' icon being the three cars icon I posted earlier, but with two cars at back greyed out or just outlines? BTW, I think this #1525 largely completes #523 so could possibly mark it as fixing #523 in OP? |
I don't have much to add but I was mentioned and decided to take a glance, this is looking really good! Once this is merged might just convince me to come back to C:S for a bit haha. Looking forward to HQ icons. Finally can use them with my ultrawide 1440p with no blur! 😄 |
Definitely much better than the old button! It still looks a bit brash though... What about something like: https://thenounproject.com/icon/traffic-135499/ And then in |
For |
Nice! Fine tuning... Disable despawn button:
Clear traffic button:
Both buttons:
|
Those look amazing! BTW, would it be worth tying the button background with just tht thin green outline (no fill on normal state) - it looks super clean. |
I'd like buttons to be distinct from the panel. We're not doing minimalist flat "material design" which is criticised by many people for how hard is to find buttons and other interactable elements in it. |
Closes #523
Checklist
v11
with small yellow "update notification dot" -!TMPELifecycle.Instance.WhatsNew.Shown
https://thenounproject.com/icon/remove-873403/
https://thenounproject.com/icon/remove-873413/
https://thenounproject.com/icon/remove-car-621411/
Prevent despawning: https://thenounproject.com/icon/traffic-135499/
Enable despawning: https://thenounproject.com/icon/car-135487/