Opened 21 months ago
Last modified 16 months ago
#57537 new enhancement
Site editor theme export: remove site specific details
Reported by: | jffng | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | has-patch |
Focuses: | Cc: |
Description
Currently when you use the Site Editor's theme export functionality, there are some block attributes that are specific to the site, including:
- refs id on the Navigation block
- queryId on the query block
It would be better to not include these attributes on theme export, since they do not have any use / may cause incorrect behavior if the theme is installed on a different site.
See https://github.com/WordPress/gutenberg/issues/42730 for more context.
Change History (4)
This ticket was mentioned in PR #3899 on WordPress/wordpress-develop by @jffng.
21 months ago
#1
- Keywords has-patch added
@mikachan commented on PR #3899:
21 months ago
#2
Thanks so much for working on this, it looks great!
Do you think we should also remove the id
attributes from the image and cover blocks? Both these blocks also include wp-image-[id]
classes, which I think would be good to remove at the same time.
21 months ago
#3
Do you think we should also remove the id attributes from the image and cover blocks?
Good idea, done in 4def221. I also added a filter for the media & text block.
@madhudollu commented on PR #3899:
16 months ago
#4
Lets remove taxQuery
?
Here is a reference about issue with this attribute p1686029927912499-slack-CRWCHQGUB
Trac ticket: https://core.trac.wordpress.org/ticket/57537