Remove axis padding from equal-aspect image plot

[JamesDoingStuff]

No description provided.

[PeterC-DLS]

Looks fine apart from my comments

[VictoriaBeilsten-Edmands]

Would you add an example of a plot in different sized containers to a tab in client/example/src/App.tsx

[PeterC-DLS]

Please add the new attribute to ImagePlot.stories

[JamesDoingStuff]

Updated to add story. I wasn't able to get the useMeasure hook to work without causing infinite re-renders - the ref from that hook seems to behave slightly differently to the one from useRef in a straight swap.

[PeterC-DLS]

Does setting tightAxes=true always mean that the axes are drawn wholly to the image extent?

In the screenshot, I squeezed the browser window horizontally. The left plot has space visible at the bottom, and the top right plot has space on the left and right side:.

[JamesDoingStuff]

Sorry about that, I hadn't noticed - it seemed to revert back to the correct aspect when expanding the window at all, so not sure what exactly wasn't triggering when shrinking it. I've changed it so that instead of re-triggering the effect, it uses a resize observer callback. I think it's fixed now and the animation seems smoother as a bonus

[PeterC-DLS]

I think there is still an issue when reducing the window height. The bottom right plot has space on the left and right:

[JamesDoingStuff]

Damn, back to the drawing board... Thanks for letting me know

[JamesDoingStuff]

Actually I think that that might be a result of the plot's toolbar reaching it's minimum size - the space only starts appearing once the toolbar hits the left boundary of the plot. This is what happens if you turn it off:

I could get the plot to shrink smaller than the toolbar if you'd like, but at that point the title formatting is going to be messed up anyway and there's not much I can do about that

[PeterC-DLS]

How easy is it to move the title to its own box so it isn't constrained by the plot size?

[JamesDoingStuff]

The title (as it currently exists) gets handled way down in the h5web VisCanvas component, so I wouldn't want to mess with that. Adding in a new title box would be possible I suppose, but would probably add more complexity to this feature

[PeterC-DLS]

Ok, if you allow the plot to be smaller than the toolbar width, does the items on the toolbar flow into the drop-down menu?

[JamesDoingStuff]

Apologies for not getting this finished before going on leave. The issue was that I'd put the toolbar it its own div - setting minWidth: 0 for that div lets the toolbar shrink again

[PeterC-DLS]

The example app behaves fine when I squeeze the window horizontally: plots seem to have minimum widths and the toolbar narrows and items overflow to dropdown menu properly.

[PeterC-DLS]

LGTM