-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
feat(discover): Added world map mini graph to Discover saved query list #29493
Conversation
size-limit report
|
Drafting this for now, still deciding on what it should visually look like |
71750ca
to
0504234
Compare
Increase query card render height to 150 px to match the Dashboard cards and to get a better aspect ratio on world maps.
…er-world-map-mini-chart
Reopened, we decided to move forward with the world map preview |
… into feat/discover-world-map-mini-chart
zoom: fromDiscover ? 1.1 : fromDiscoverQueryList ? 0.9 : 1.3, | ||
center: [10.97, fromDiscoverQueryList ? -12 : 9.71], |
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.
Should we move some of these magic numbers to constants?
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.
👍 updated!
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.
LGTM
…er-world-map-mini-chart
Based off
feat/discover-bar-chart