-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dashboard: Remove elements dependency #12237
Conversation
Plugin builds for 1475613 are ready 🛎️!
|
Size Change: +1.4 kB (0%) Total Size: 2.68 MB
ℹ️ View Unchanged
|
Just looking at this myself. I think it because |
assets/js/wp-story-editor.js being 1.34MB seems like very big. I wonder if we should look into breaking this up. I am little bit worried about the editor loading. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good to me. But I would like to follow up and look into the bundle size of the editor, which at 1.34mb seems a little large.
Dashboard and editor have a common bundle for shared code. The main reason is the stickers package, which is a dependency of With this PR, stickers are no longer included for the dashboard and thus no longer in the common bundle but only in the editor bundle. So that all makes sense. There is already a ticket to optimize the way stickers are included, see #11973 See also #11972 for the masks package |
Context
We do not render stories or templates in the dashboard anymore. We just display preview images. So there's no need for us to actually register elements.
Summary
Removes dependencies on
elements
andelement-library
from the dashboard package.This should improve bundles a bit.
Relevant Technical Choices
To-do
User-facing changes
N/A
Testing Instructions
This PR can be tested by following these steps:
Reviews
Does this PR have a security-related impact?
No
Does this PR change what data or activity we track or use?
No
Does this PR have a legal-related impact?
No
Checklist
Type: XYZ
label to the PRFixes #