Ensure DashCanvas layout updated if window resized while hidden

[amcclain]

• DashCanvas now properly updates its layout when shown if the browser window had been resized while the component was hidden (e.g. in an inactive tab).
• Fixes DashCanvas does not update layout when window resized while not visible #3215

Hoist P/R Checklist

Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

• Caught up with develop branch as of last change.
• Added CHANGELOG entry, or determined not required.
• Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
• Updated doc comments / prop-types, or determined not required.
• Reviewed and tested on Mobile, or determined not required.
• Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

[amcclain]

Two things I wasn't 100% sure about:

• Update to use composeRefs to compose model.ref, the useOnVisibleChange hook, and (this is the part I wasn't clear on) a forwarded ref. I was looking at the pattern in DashContainer and went with that:

hoist-react/desktop/cmp/dash/container/DashContainer.ts

Line 34 in 8ee88de

ref = composeRefs(

• In the ticket @haynesjm42 filed, he has a delay in his workaround - in local development I was seeing the canvas resize OK without it. Might be a timing difference introduced by the use of the hook? Happy to start without it, but wanted to call that out.

[haynesjm42]

Re: the timing, I bet you are correct and that now that we are using the hook and reacting on the visibility of the DashCanvas itself instead of the activeTabId that it is no longer needed