Skip to content

Fix header redesign found issues#15434

Merged
alecslupu merged 11 commits intodecidim:feature/remove-menu-entriesfrom
i-need-another-coffee:fix/fix-header-issues
Nov 11, 2025
Merged

Fix header redesign found issues#15434
alecslupu merged 11 commits intodecidim:feature/remove-menu-entriesfrom
i-need-another-coffee:fix/fix-header-issues

Conversation

@MariaDascaluPublicis
Copy link
Copy Markdown
Contributor

@MariaDascaluPublicis MariaDascaluPublicis commented Oct 28, 2025

🎩 What? Why?

Fixed some issues listed here https://decidim.notion.site/Review-menu-and-navigation-268dea4aebbe804baa92df64eadf83fc

📌 Related Issues

Testing

  • Visit the platform on a desktop screen, when the header loads, you should see a hamburger menu icon, the language chooser and the search input
  • Click the hamburger icon, then a menu panel opens displaying the full list of navigation links.
  • Click outside the panel or click close, then the panel closes.
  • The menu is closed, then you can open it again using the same icon.
  • Navigate through the links inside the hamburger menu, you should be redirected to the corresponding section.
  • Click on the language chooser and the available languages should be listed in a vertical list
  • On desktop, when you navigate through a participatory space, then you see a breadcrumb showing the full path from the homepage to the current page.
  • You click on any item in the breadcrumb on desktop, then you are redirected to that corresponding level.
  • On mobile, when you navigate through a participatory space, then you see a compact breadcrumb showing only the immediate previous level.
  • Tap the compact breadcrumb on mobile, then you redirected to the previous level in the hierarchy.

📷 Screenshots

image image image image

♥️ Thank you!

@github-actions github-actions bot added module: core type: fix PRs that implement a fix for a bug labels Oct 28, 2025
alecslupu
alecslupu approved these changes Nov 10, 2025
View reviewed changes
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 the no-backport Pull Requests that should not be backported label Nov 10, 2025
@alecslupu alecslupu changed the title Fix header redesigne found issues Fix header redesign found issues Nov 10, 2025
@alecslupu alecslupu merged commit 23cc295 into decidim:feature/remove-menu-entries Nov 11, 2025
74 of 75 checks passed
@alecslupu alecslupu deleted the fix/fix-header-issues branch November 11, 2025 07:23
andreslucena added a commit that referenced this pull request Nov 18, 2025
@alecslupu @MariaDascaluPublicis
@greenwoodt @andreslucena
Implement new menu in frontend (#15134)
7346577 
* Remove Help, Meetings, and Activity from the header navigation

Remove unused locale

Patch the menu

* Implement breadcrumb navigation (#15141)

Co-authored-by: MariaDascaluPublicis <166501615+MariaDascaluPublicis@users.noreply.github.com>

* Remove the "Global menu" content block (#15135)

Co-authored-by: Tom Greenwood <tom.t.greenwood@gmail.com>

* Implement hamburger menu  (#15132)

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
Co-authored-by: MariaDascaluPublicis <maria.dascalu@publicissapient.com>

* Add language chooser to header (#15365)

Co-authored-by: MariaDascaluPublicis <166501615+MariaDascaluPublicis@users.noreply.github.com>
Co-authored-by: MariaDascaluPublicis <maria.dascalu@publicissapient.com>

* Fix header redesign found issues (#15434)

Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>

* Add breadcrumb to menu (#15507)

* Add breadcrumb to accountability

* Add breadcrumb to blogs

* Add breadcrumb to debates

* Add breadcrumb to collaborative_texts

* Add breadcrumb to elections

* Add breadcrumb to surveys

* Add breadcrumb to sortitions

* Add breadcrumb to proposals

* Add breadcrumb to meetings

* Fix routing errors

* Revert changes on sortitions to avoid conflicts

---------

Co-authored-by: MariaDascaluPublicis <166501615+MariaDascaluPublicis@users.noreply.github.com>
Co-authored-by: Tom Greenwood <tom.t.greenwood@gmail.com>
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
Co-authored-by: MariaDascaluPublicis <maria.dascalu@publicissapient.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

+1 more reviewer

@alecslupu alecslupu alecslupu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@MariaDascaluPublicis MariaDascaluPublicis

Labels

module: core module: proposals no-backport Pull Requests that should not be backported type: fix PRs that implement a fix for a bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@MariaDascaluPublicis @alecslupu