Skip to content
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

Fix: Term Description block should only be available in the site editor #51053

Merged
merged 3 commits into from
Jun 28, 2023

Conversation

carolinan
Copy link
Contributor

What?

The term description block displays the description of taxonomies in archives, and does not output anything on the front
if it is used in any other context.

Closes #38552

Why?

It can be confusing if you add a block to the post content, that is visible in the editor but not the front.

How?

Adds __experimental": "fse", to block.json

Testing Instructions

Create a new post using the block editor.
Try to add a term description block. Confirm that the block is not available.

Go to the Site Editor and edit any template.
Try to add a term description block. Confirm that the block is available.
(if you view the front, it still wont show unless you are on an archive page for a taxonomy that has a description to show)

@carolinan carolinan added [Block] Term Description Affects the Term Description Block - used for displaying the description of a tag, catergory or othe [Type] Enhancement A suggestion for improvement. labels May 29, 2023
@github-actions
Copy link

Flaky tests detected in c988f9d.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/5265203395
📝 Reported issues:

Copy link
Member

@aristath aristath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me! It's a minimal and simple change, fixes the issue, and gets the job done the right way.

Thank you for working on this @carolinan! 👍

@aristath aristath merged commit 16486bd into trunk Jun 28, 2023
50 checks passed
@aristath aristath deleted the update/hide-term-description branch June 28, 2023 08:37
@github-actions github-actions bot added this to the Gutenberg 16.2 milestone Jun 28, 2023
sethrubenstein pushed a commit to pewresearch/gutenberg that referenced this pull request Jul 13, 2023
@femkreations femkreations added the Needs User Documentation Needs new user documentation label Sep 28, 2023
@danieldudzic
Copy link

I'm not sure if the __experimental": "fse" was meant for hiding blocks in the block editor.

This change has introduced a regression, as the Term Description block now is not registered without an active Gutenberg plugin running.

https://core.trac.wordpress.org/ticket/59755

SiobhyB pushed a commit that referenced this pull request Oct 27, 2023
Reverts changes in #51053, to address a regression in which the Term Description block was only available in core when the Gutenberg plugin was active.
SiobhyB pushed a commit that referenced this pull request Oct 27, 2023
Reverts changes in #51053, to address a regression in which the Term Description block was only available in core when the Gutenberg plugin was active.
SiobhyB pushed a commit that referenced this pull request Oct 27, 2023
Reverts changes in #51053, to address a regression in which the Term Description block was only available in core when the Gutenberg plugin was active.
SiobhyB pushed a commit that referenced this pull request Oct 28, 2023
* Patterns: set context of pattern category api call to view so authors and contributors can access (#55553)

* Query Loop: Disallow "enhanced pagination" with core blocks that may contain third-party blocks (#55539)

* Update useContainsThirdPartyBlocks and texts

* Improve texts a bit

* Show the list of unsupported blocks

* Use the BlockTitle component instead

* Simplify modal text

* Revert "Query Loop: Disallow "enhanced pagination" with core blocks that may contain third-party blocks (#55539)"

This reverts commit f2dd6de.

* Query Loop: Disallow "enhanced pagination" with core blocks that may contain third-party blocks (#55539)

* Update useContainsThirdPartyBlocks and texts

* Improve texts a bit

* Show the list of unsupported blocks

* Use the BlockTitle component instead

* Simplify modal text

* Revert CSS changes to show/hide embedded pdf (#55667)

* Ensure Term Description block is available in core (#55669)

Reverts changes in #51053, to address a regression in which the Term Description block was only available in core when the Gutenberg plugin was active.

---------

Co-authored-by: Glen Davies <[email protected]>
Co-authored-by: David Arenas <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Term Description Affects the Term Description Block - used for displaying the description of a tag, catergory or othe Needs User Documentation Needs new user documentation [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Term Description block should only be available in the FSE template editor.
4 participants