🤖 Meticulous spotted visual differences in 287 of 372 screens tested: view and approve differences detected.

_{Last updated for commit 1de698d. This comment will update as new commits are pushed.}

Would it be possible to adjust the height and the padding between the bars to match the actual Flamegraph closely?

Same with the font of the headers in the top table loader:

Just a feedback on the overall approach: Based on my previous experiences with the skeleton loader, it's better if the skeleton loaders are handled by the actual components themselves (by taking in a loading prop) instead of creating new components with the matching design.

1. It helps to reduce the jumps in the UI when the app switches between the loaders and the actual content.
2. It is easier to maintain the loader designs when they are handled by the actually component, otherwise any design change/additions to actual components have to be replicated with the skeleton components as well, becomes a pain in the longer run.

But this is almost like a rewrite of this whole PR, so I guess we can proceed with this implementation but keep this in mind when we plan any work on the loaders in the future so we can transition towards this.

Hmm i agree with the skeleton loaders being ideally handled by the actual components and taking in a loading prop. The latter part of that is actually already included in the PR, but its the former that might be a bit tricky.

Because out of all of the visualizations, the table component is probably the only one in which we can ideally have an idea of what it's going to be like after the data is done loading. For the icicle graph and metrics graph (and to some extent the source view) we can't be sure of how it's going be drawn/rendered on the browser until we've successfully fetched the data. Which is why an SVG matching the design was approach used.

Would it be possible to adjust the height and the padding between the bars to match the actual Flamegraph closely?

Same with the font of the headers in the top table loader:

Thanks for spotting that, I need to do some more responsive work for the skeletons so it's consistent across different screens.