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

User selectable map route line color and switchable route line out & back opacity #4030

Merged
merged 2 commits into from Jun 10, 2022

Conversation

paulj49457
Copy link
Contributor

@paulj49457 paulj49457 commented Aug 30, 2021

To try and address comments 1&2 raised in : https://groups.google.com/g/golden-cheetah-users/c/LQ1bq3zGTeE/m/5AvoGKX6AQAJ

1) Map tiles layer should be able to be dimmed or desaturated to make the ride's path more visible - The map details are great, but often visually overpower and make it hard to see the ride's path (map details have many colours and saturated colours).. especially if multicoloured "shaded zone" (which I find useful is on)
2) The colour and width of yellow outline of ride path should be configurable - depending on map tiles other colours might be preferable (don't overlap colours of map tile details).. increasing width (if highlight colour is transparent) make path more visible

Trying to keep things simple and hopefully provide sufficient configurability I have proposed:

  1. the option to switch off the out & back route line opacity, this can improve the visibility of the route line, the default is the standard GC 50% opacity display.

Capture1

  1. and I have also added the option to change the route line colour (see below), note: this is only displayed for activities with no power information or when the hide shaded zones is selected (checkbox ticked), and again defaults to the standard GC red:

Capture2

Note the power zone line colours are already configurable, the difficulty is trying to choose them so they appear clear on some tile server displays :

Capture3

@paulj49457
Copy link
Contributor Author

Any chance this can get merged in 3.6 ?

…e to the addition of two new colurs (different PR) incorrectly merged
@amtriathlon amtriathlon merged commit dd5bc51 into GoldenCheetah:master Jun 10, 2022
@amtriathlon
Copy link
Member

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants