Skip to content

Breadcrumbs: Remove wrapper div and use HtmlRenderer for dynamic content rendering #74274

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.

Sign up for GitHub

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

Jump to bottom
Closed
prasadkarmalkar wants to merge 1 commit into from
Closed

Breadcrumbs: Remove wrapper div and use HtmlRenderer for dynamic content rendering #74274

prasadkarmalkar wants to merge 1 commit into from

Conversation

@prasadkarmalkar
Copy link
Contributor

@prasadkarmalkar prasadkarmalkar commented Dec 30, 2025

What?

Part of Issue- #74248

Why?

  • Remove extra divs and eliminate the extra CSS required to match the frontend and editor UI

How?

  • Uses the HTMLRenderer component and useServerSideRender() hook to eliminate these extra divs and make the HTML on the front end and editor match for Breadcrumbs Block.

Testing Instructions

  1. Edit a Post/Page
  2. Add the Breadcrumbs block
  3. Save and Check UI on both frontend and editor

Screenshots or screencast

Screen.Recording.2025-12-30.at.3.31.51.PM.mov

@github-actions github-actions bot added the [Package] Block library /packages/block-library label Dec 30, 2025
@hbhalodia
Copy link
Contributor

hbhalodia commented Dec 31, 2025

@hbhalodia hbhalodia closed this Dec 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ajitbohra ajitbohra Awaiting requested review from ajitbohra ajitbohra will be requested when the pull request is marked ready for review ajitbohra is a code owner

@fabiankaegy fabiankaegy Awaiting requested review from fabiankaegy fabiankaegy will be requested when the pull request is marked ready for review fabiankaegy is a code owner

Assignees

No one assigned

Labels

[Package] Block library /packages/block-library

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@prasadkarmalkar @hbhalodia