-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[Block Library - Post Excerpt]: Add attribute to show the read more
link
#38662
base: trunk
Are you sure you want to change the base?
[Block Library - Post Excerpt]: Add attribute to show the read more
link
#38662
Conversation
Size Change: +123 B (0%) Total Size: 1.31 MB
ℹ️ View Unchanged
|
I think it can be confusing that there is both a setting (even if needs to be toggled) and a separate block. |
That could be a good idea, in the form of a Both approaches though would still keep legacy code which I don't like. Maybe we could use this approach but 'hide' better these settings under advanced? 🤔 |
Flaky tests detected in 7bd7665. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4102291297
|
This is POC for adding a new attribute to
Post Excerpt
block to hide by default theread more
link. This is because recently a new standalone Read More block link has been added, which provides more flexibility and enables more designs.Testing instructions
Post Excerpt
blocks should work as before(both editor and front end)read more
text set should migrate and hide the editable field in the editor. In frontend we didn't show anything anyway if it wasn't set.Post Excerpt
blocks and observe that the field is not visible at the insertion and is displayed if we toggle the attribute at the Inspector controls.