fix: incorrect padding offset on health chart#121
Conversation
✅ Deploy Preview for agentscan ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughGlobalEventsEvolution.vue now computes responsive left/right padding based on chart width and series count using a new ChangesChart Layout and Dependency
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This fixes the incorrect offset calculation on the health chart, which was overflowing the first and last datapoints more and more after each day (with the current number of days to this date, the first and last datapoints were basically impossible to hover). The offset now adapts to hardcoded internals of the chart component.
Other: bump vue-data-ui to latest (no impact on our usage here, just staying fresh)
Summary by CodeRabbit
Bug Fixes
Chores