Skip to content

[Details Block]: Adds anchor support in details block#66734

Merged
t-hamano merged 2 commits into
trunkfrom
adds/anchor-support-details-block
Nov 8, 2024
Merged

[Details Block]: Adds anchor support in details block#66734
t-hamano merged 2 commits into
trunkfrom
adds/anchor-support-details-block

Conversation

@amitraj2203
Copy link
Copy Markdown
Contributor

@amitraj2203 amitraj2203 commented Nov 5, 2024

What?

Closes #66723

Why?

Adds anchor support in details block

Testing Instructions

  1. Create a post or page.
  2. Insert a details block.
  3. On the Inspector control, open the Advanced Panel and see the HTML Anchor option

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 5, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: amitraj2203 <amitraj2203@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>
Co-authored-by: skorasaurus <skorasaurus@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@amitraj2203 amitraj2203 requested a review from t-hamano November 5, 2024 05:46
@amitraj2203 amitraj2203 added [Type] Enhancement A suggestion for improvement. [Block] Details Affects the Details Block - used to display content which can be shown/hidden labels Nov 5, 2024
Copy link
Copy Markdown
Contributor

@t-hamano t-hamano left a comment

Choose a reason for hiding this comment

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

LGTM! I think all static blocks need anchor support, as was done in #23197.

@t-hamano
Copy link
Copy Markdown
Contributor

t-hamano commented Nov 6, 2024

@amitraj2203 Could you rebase this PR? The stalling CI should work (See #66771).

@amitraj2203
Copy link
Copy Markdown
Contributor Author

@t-hamano Done

@t-hamano t-hamano merged commit bee12b1 into trunk Nov 8, 2024
@t-hamano t-hamano deleted the adds/anchor-support-details-block branch November 8, 2024 08:36
@github-actions github-actions Bot added this to the Gutenberg 19.7 milestone Nov 8, 2024
@skorasaurus
Copy link
Copy Markdown
Member

Thank you for your work on this =)

karthick-murugan pushed a commit to karthick-murugan/gutenberg that referenced this pull request Nov 13, 2024
Co-authored-by: amitraj2203 <amitraj2203@git.wordpress.org>
Co-authored-by: t-hamano <wildworks@git.wordpress.org>
Co-authored-by: skorasaurus <skorasaurus@git.wordpress.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Block] Details Affects the Details Block - used to display content which can be shown/hidden [Type] Enhancement A suggestion for improvement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Details Block: Add Anchor Support

3 participants