Skip to content

Fix duplicate breadcrumb block declaration in partial#204

Merged
Codencode merged 2 commits into
PrestaShop:developfrom
Codencode:fix/breadcrumb-block-inheritance
Feb 24, 2026
Merged

Fix duplicate breadcrumb block declaration in partial#204
Codencode merged 2 commits into
PrestaShop:developfrom
Codencode:fix/breadcrumb-block-inheritance

Conversation

@Codencode

Copy link
Copy Markdown
Contributor
Questions Answers
Description? This PR fixes the breadcrumb block structure to avoid declaring the same {block name='breadcrumb'} in both layout-both-columns.tpl and breadcrumb.tpl.
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? -
Sponsor company Codencode snc
How to test?

@Codencode Codencode requested review from a team and tblivet February 23, 2026 16:12
ga-devfront
ga-devfront previously approved these changes Feb 23, 2026
@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Feb 23, 2026
tblivet
tblivet previously approved these changes Feb 23, 2026
@tblivet tblivet added this to the 3.0.7 milestone Feb 23, 2026
jolelievre
jolelievre previously approved these changes Feb 23, 2026
Touxten
Touxten previously approved these changes Feb 23, 2026

@tblivet tblivet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @Codencode, that’s good 👍 Just a small naming change regarding my suggestion: I don’t think _wrapper is needed.

Comment thread templates/_partials/breadcrumb.tpl Outdated
@ps-jarvis ps-jarvis moved this from Ready for review to Waiting for author in PR Dashboard Feb 24, 2026
@Codencode Codencode force-pushed the fix/breadcrumb-block-inheritance branch from 07e5783 to 26cbf0d Compare February 24, 2026 08:21
@ps-jarvis ps-jarvis moved this from Waiting for author to To be tested in PR Dashboard Feb 24, 2026
@ingridusta ingridusta self-assigned this Feb 24, 2026

@ingridusta ingridusta left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi @Codencode,

Your pr is tested and no regression detected. keyboard nav + links are still functional in the breadcrumb.

Image

It's QA approved ! ✅

@ingridusta ingridusta added the QA ✔️ Status: Check done, Code approved label Feb 24, 2026
@Codencode

Copy link
Copy Markdown
Contributor Author

Thank you @ingridusta

@Codencode Codencode merged commit 7614d56 into PrestaShop:develop Feb 24, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from To be tested to Merged in PR Dashboard Feb 24, 2026
@Codencode Codencode deleted the fix/breadcrumb-block-inheritance branch February 24, 2026 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

QA ✔️ Status: Check done, Code approved

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

7 participants