-
Notifications
You must be signed in to change notification settings - Fork 24
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: add new way to display peaks #2562
Conversation
Deploying with Cloudflare Pages
|
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #2562 +/- ##
=======================================
Coverage 51.87% 51.87%
=======================================
Files 52 52
Lines 2562 2562
Branches 85 85
=======================================
Hits 1329 1329
Misses 1231 1231
Partials 2 2 ☔ View full report in Codecov by Sentry. |
You may invert the logic so that by default it displays the peak values on the top and when you click on the toggle it would display the value just over the peak. |
Is the edit mode still necessary in the new display mode? It appears that you intend to set it as the primary display mode |
The tests are not passing but the functionality looks ok to me. |
dde59db
to
2e8867e
Compare
No description provided.