Open okmttdhr opened 21 hours ago
https://calypso.live?image=registry.a8c.com/calypso/app:build-125247&env=jetpack |
https://calypso.live?image=registry.a8c.com/calypso/app:build-125247&env=a8c-for-agencies |
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser:
Sections (~2 bytes removed 📉 [gzipped])
Legend
Generated by performance advisor bot at iscalypsofastyet.com.
This PR modifies the release build for the following Calypso Apps:
For info about this notification, see here: PCYsg-OT6-p2
To test WordPress.com changes, run install-plugin.sh $pluginSlug fix/hide-actions-on-deleted-site
on your sandbox.
Related to https://github.com/Automattic/dotcom-forge/issues/9903
Proposed Changes
This PR hides actions except "Restore" on a deleted site.
before
https://github.com/user-attachments/assets/3f7686d0-b6ea-4e9f-8bd5-f593779a5b59
after
https://github.com/user-attachments/assets/35f8bf5a-ba7d-467f-b9fb-27456b7ae2e8
I retained the loading spinner in the status column, as loading status in actions doesn’t seem to be natively supported. This might be worth revisiting in a follow-up. CC: @youknowriad (in case you have insight)
Why are these changes being made?
https://github.com/Automattic/dotcom-forge/issues/9903
Testing Instructions
Pre-merge Checklist