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

Improve the browse page's UI #303

Merged
merged 9 commits into from
Aug 3, 2023
Merged

Improve the browse page's UI #303

merged 9 commits into from
Aug 3, 2023

Conversation

dabreegster
Copy link
Contributor

A bunch of things:

  • Line up the help buttons on the right, and remove the odd background
  • Group layer controls, add scrolling to the right sidebar, and make sure it doesn't cover up the bottom-right map controls
  • Move the color controls for interventions to the left sidebar. The left is for scheme data, the right is for extra layers -- hopefully that's more clear.
  • Change cursors to indicate clickable layers
  • Fix a bug, so that clicking on multiple layers only does the topmost one
  • Reorder layers on the browse page, so scheme data is on top
  • Start the browse page focused on England, even before loading data
screencast.mp4

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved this to lib/common and renamed to avoid clashing with the built-in JS Map

@dabreegster dabreegster merged commit 3c844ae into main Aug 3, 2023
2 checks passed
@dabreegster dabreegster deleted the browse_ui_fixes branch August 3, 2023 14:18
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