client/web/src/site-admin: Add history panel

[indradhanush]

👉 Loom: https://www.loom.com/share/d84dfbf54f3c46d99a67ff79910eadbe

Fixes #46032.

Test plan

1. Tested locally
2. Builds should pass

App preview:

• Web

Check out the client app preview documentation to learn more.

[sg-e2e-regression-test-bob]

Bundle size report 📦

Initial size	Total size	Async size	Modules
0.00% (0.00 kb)	0.04% (+6.54 kb)	0.06% (+6.54 kb)	0.00% (0)

Look at the Statoscope report for a full comparison between the commits b249fa6 and ccf9b20 or learn more.

Open explanation

• Initial size is the size of the initial bundle (the one that is loaded when you open the page)
• Total size is the size of the initial bundle + all the async loaded chunks
• Async size is the size of all the async loaded chunks
• Modules is the number of modules in the initial bundle

[sashaostrikov]

Nice improvement, good job :)
Added some comments.

[varsanojidan]

Looks great Indra, I think this is going to be really helpful for Site Admins!

Have you played around with adding colors to the diffs by any chance? Something like this?

I feel like visually it would be kind of nice.

[sashaostrikov]

Did a second round of comments.
Ongoing fixes LGTM, waiting for new commits :)

[indradhanush]

Have you played around with adding colors to the diffs by any chance? Something like this?

@varsanojidan I did not. For the first version we decided to ship it without colors to start with.

[sashaostrikov]

LGTM!
I think you need to add a horizontal line in case of an empty diff too?
Let's fix it and

[indradhanush]

I think you need to add a horizontal line in case of an empty diff too?

@sashaostrikov Diff will technically never be empty any more since we changed the backend to skip redundant entries. The API spec allows it to be null at the moment which needs to be changed.

[sashaostrikov]

Approving to unblock.
Nice job!

[danielmarquespt]

Regarding the avatar:

It would be nice to have the avatars, but not at the expense of performance. If we figure out a way of displaying avatars without a performance hit, then this would be an example of how to show them:

if not:

Nevertheless, I think we could change the title of the section to "Change history" and the layout of each row.

I also think we can be clear about the changes that happen via file instead of via user. I added wording for this, however I'm not entirely sure if it captures the exact scenario.