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

Themed default speed signs (redux) #1404

Merged
merged 9 commits into from
Feb 15, 2022
Merged

Conversation

originalfoo
Copy link
Member

@originalfoo originalfoo commented Feb 14, 2022

Compiled mod for testing: TMPE.zip

This is a redux of #1320

image

Changes to speed limits overlay:

  • Segment and lane modes now always use themed icons (defaults mode still always shows unthemed icons).
  • If user customises a speed to value other than network default, a green sub-icon is appended to the themed speed sign.

Example:

image image

That sub-icon is only shown in interactive mode, when used has customised the speed to non-default value, but only if a mod option is enabled in Overlays tab:

image

Todo:

  • Apply same thing to lane mode
  • Persist option to savegame
  • Fix size of lane mode icons (similar to Themed default speed signs #1320)
  • Test with monorail in lane mode (which adds passenger train sub-icon)

Updates: #1221 (comment)

- In normal view, always show themed signs
- Show green sub-icon if speed = default
@originalfoo originalfoo added Usability Make mod easier to use UI User interface updates SPEED LIMITS Feature: Speed limits Overlays Overlays, data vis, etc. labels Feb 14, 2022
@originalfoo originalfoo added this to the 11.6.5 milestone Feb 14, 2022
@originalfoo originalfoo self-assigned this Feb 14, 2022
@originalfoo originalfoo marked this pull request as draft February 14, 2022 04:52
@originalfoo originalfoo mentioned this pull request Feb 14, 2022
1 task
@originalfoo
Copy link
Member Author

An alternative is to show the green sub-icon (complete with speed value) on non-default speeds, and maybe have mod option to toggle whether it's shown at all?

I'm thinking tht would be much better approach acutally. And have the option disabled by default.

Now actually going to get some sleep.

- only in interactive mode
- only if speed is different to default
- only if new mod option is true
@kvakvs
Copy link
Collaborator

kvakvs commented Feb 14, 2022

I would say it does not matter where and how you show the default green icon, this is a case of bikeshedding. Just show the default somewhere. Need an option to hide it - fine, you already had that implemented.

@originalfoo originalfoo marked this pull request as ready for review February 14, 2022 19:34
@originalfoo
Copy link
Member Author

ready for review

@krzychu124
Copy link
Member

Some themes does not play well with default 😕

image

image

@originalfoo
Copy link
Member Author

Ah, good catch, it's currently using same aspect ratio as the themed sign. Hopefully easy to fix, brb...

- Adds dedicated method on sign renderer for drawing sub icon
@originalfoo
Copy link
Member Author

Ok, try that :)

Copy link
Member

@krzychu124 krzychu124 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks ok, tested in-game.
image

@originalfoo originalfoo added Settings Road config, mod options, config xml Localisation Localised text and features labels Feb 15, 2022
@originalfoo
Copy link
Member Author

@kvakvs any feedback?

Copy link
Collaborator

@kvakvs kvakvs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Me likey

@originalfoo originalfoo merged commit dac1e80 into master Feb 15, 2022
@originalfoo originalfoo deleted the themed-defaults-redux branch February 15, 2022 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Localisation Localised text and features Overlays Overlays, data vis, etc. Settings Road config, mod options, config xml SPEED LIMITS Feature: Speed limits UI User interface updates Usability Make mod easier to use
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants