Skip to content

Fix mobile breadcrumb html safe#13870

Closed
rsdeus wants to merge 0 commit intodecidim:developfrom
octree-gva:fix/mobile_breadcrumb_html_safe
Closed

Fix mobile breadcrumb html safe#13870
rsdeus wants to merge 0 commit intodecidim:developfrom
octree-gva:fix/mobile_breadcrumb_html_safe

Conversation

@rsdeus
Copy link
Copy Markdown
Contributor

@rsdeus rsdeus commented Jan 15, 2025

🎩 What? Why?

Please describe your pull request.

📌 Related Issues

#13869

Testing

Create an proposal named, for example: "Partage d'idées"

📷 Screenshots

image

♥️ Thank you!

@github-actions github-actions bot added module: core type: fix PRs that implement a fix for a bug labels Jan 15, 2025
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

👍

@alecslupu alecslupu added release: v0.28 Issues or PRs that need to be tackled for v0.28 release: v0.29 Issues or PRs that need to be tackled for v0.29 labels Jan 15, 2025
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

oh, i saw there are pipeline failing.

maybe worth reverting theese changes and patch decidim-core/app/views/layouts/decidim/header/_menu_breadcrumb_mobile_tablet.html.erb ? ( ex do something with that decidim_escape_translated helper that is being used)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: core release: v0.28 Issues or PRs that need to be tackled for v0.28 release: v0.29 Issues or PRs that need to be tackled for v0.29 type: fix PRs that implement a fix for a bug

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants