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

Fixing legend #70

Open
wants to merge 4 commits into
base: Development
Choose a base branch
from
Open

Fixing legend #70

wants to merge 4 commits into from

Conversation

davidreneuw
Copy link
Contributor

This initial setup implements a hardcoded range on heatmap versions of the map and dynamic range on the scatter version. In addition, a couple altitude chart bugs were fixed: sometimes the altitude chart would not show the correct range, if any, and some other times it would show values like latitude and longitude on the y axis.

This initial setup implements a hardcoded range on heatmap versions of the map and dynamic range on the scatter version. In addition, a couple altitude chart bugs were fixed: sometimes the altitude chart would not show the correct range, if any, and some other times it would show values like latitude and longitude on the y axis.
@davidreneuw davidreneuw self-assigned this Mar 18, 2021
@davidreneuw davidreneuw linked an issue Mar 18, 2021 that may be closed by this pull request
Removed hardcoded bounds for legend range and made it from 0 to the max value that is not NaN
Added a checkbox that allows the user to decide if the colorbar for the geo map will have fixed range or dynamic range. Added a paragraph explaining the function as well as a special indication on the legend when it is not fixed.

Added translations as well.
@davidreneuw
Copy link
Contributor Author

This branch has implemented all the features and fixed all the issues linked to it. This is ready to be merged.

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.

Add Dynamic legend checkbox
1 participant