Issue/409 chart zoom v1

[rachel-labri-tipton]

Pull Request

Description

This adds the zoom feature to the chart. Currently you can highlight the chart you want to inspect and the main y-axis domain is adjusted based on the maximum y value in the selected set of data.

The Recharts library allows you to highlight a reference area and then redraw the chart boundaries.

I'm wondering if the RemixLine could use another prop that would be the zoomYMax of the chart when a chart is in a zoomed state.

Here are a few screenshots of the feature.

Fixes #409

How Has This Been Tested?

I ran the code locally.

Checklist:

• [ ] My code follows OCF's coding style guidelines
• [ ] I have performed a self-review of my own code
• [ ] I have made corresponding changes to the documentation
• [ ] I have added tests that prove my fix is effective or that my feature works
• [ ] I have checked my code and corrected any misspellings

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
nowcasting-app	✅ Ready (Inspect)	Visit Preview	💬 3 unresolved	Jan 17, 2024 10:17am

[peterdudfield]

Looks really good i like it, Lets get this in with out the yaxis zoom for the moment

1. Looks great on national
2. GSP needed some y max buffer I think
3. Site level zoom went a bit funny, not sure why?
4. Delta national looked good
5. Detla GSP needed a bit of extra y mas

[rachel-labri-tipton]

thanks for having a look, @peterdudfield! All of these issues are the same for me. I have a couple ideas for fixing the ymax issue.

[braddf]

Hey @rachel-labri-tipton – looking really good on the y-axis zoom here ⚡ I think I agree with Dan's comments on Vercel, an X to close also makes sense to me, maybe on the top right, but can see how it looks. And getting same bugs with site/GSP as Peter and you have mentioned above – happy to have a look at these together tomorrow maybe if you fancy?

[braddf]

Hey @rachel-labri-tipton, is this PR superseded by the other zoom one now, and if so can we close this to tidy up pending?