-
-
Notifications
You must be signed in to change notification settings - Fork 229
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
✨ (entity selector) remove slide-in settings drawer #3353
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @sophiamersmann and the rest of your teammates on |
f24450a
to
c194fb2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks and works great, thank you!
c194fb2
to
7a4907a
Compare
d7d70b6
to
8370b41
Compare
8370b41
to
ea83cd6
Compare
ea83cd6
to
c6e5829
Compare
c6e5829
to
492f6b2
Compare
WalkthroughThe recent updates across various files in the Our World in Data Grapher codebase mainly focus on refining the user interface, specifically transitioning from a drawer to a popup model for settings and map projections. This involved adjustments in z-index values for proper layering, simplification of toggle and visibility logic in the settings menu, and the removal of constants and references associated with the previous drawer implementation. These changes aim to streamline the user experience and code efficiency. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
a5593fb
to
c015073
Compare
c015073
to
72b12ee
Compare
72b12ee
to
c91b5b5
Compare
3386c69
to
70c24cf
Compare
70c24cf
to
badf14f
Compare
Merge activity
|
Cycle 2024.2: Entity selector (prep work)
Removes the slide-in settings drawer and always renders settings into a popup instead.
We will be using the slide-in drawer for the entity selector instead (at certain screen sizes). I removed the slide-in drawer code in this PR, but I will be re-adding it for the entity selector in subsequent PRs.