Move ErrorBoundary so that tracks/views that have crashed can be closed more easily

[cmdcolin]

Currently if a track has something that hits the track level error boundary, then the track label is not displayed so the track is more difficult to close (have to use tracklist maybe)

Note: It is a little uncommon to even have this ErrorBoundary show up as there are other error handling mechanisms related to data fetching and whatnot, but this ErrorBoundary can be useful catching unexpected hard crashes in the component code and avoiding the whole session, or in this case, the whole view, from being crashed.

[cmdcolin]

some cases where the "tracklabel itself will crash the view" that will not be caught by this but probably not a large concern

[codecov]

Codecov Report

Merging #3317 (ee34e5e) into main (d54632d) will decrease coverage by 0.00%.
The diff coverage is 62.50%.

❗ Current head ee34e5e differs from pull request most recent head f4e3e6e. Consider uploading reports for the commit f4e3e6e to get more accurate results

@@            Coverage Diff             @@
##             main    #3317      +/-   ##
==========================================
- Coverage   59.44%   59.44%   -0.01%     
==========================================
  Files         736      736              
  Lines       28997    28996       -1     
  Branches     7017     7017              
==========================================
- Hits        17238    17236       -2     
- Misses      11513    11514       +1     
  Partials      246      246              

Impacted Files	Coverage Δ
packages/core/ui/App.tsx	76.71% <50.00%> (ø)
...src/LinearGenomeView/components/TrackContainer.tsx	88.57% <60.00%> (-2.61%)	⬇️
...c/LinearGenomeView/components/LinearGenomeView.tsx	84.00% <100.00%> (+2.51%)	⬆️
...rative-view/src/ServerSideRenderedBlockContent.tsx	62.50% <0.00%> (-4.17%)	⬇️
products/jbrowse-web/src/util.ts	48.33% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more