Skip to content

Mobile header refactoring - #946

Merged
tchoutri merged 1 commit into
flora-pm:developmentfrom
TixieSalander:development
Sep 18, 2025
Merged

Mobile header refactoring#946
tchoutri merged 1 commit into
flora-pm:developmentfrom
TixieSalander:development

Conversation

@TixieSalander

@TixieSalander TixieSalander commented Sep 18, 2025

Copy link
Copy Markdown
Member

Proposed changes

  • Use native popover API for navigation menu (not requiring JS + better out-of-the-box accessibility)
  • Popover polyfill for non-compatible browsers
  • ESM output + code splitting configured on ESbuild to be able to conditionally load polyfills
  • Header's search directly visible at the headers bottom
b6d72cc785f3b657.mp4

@TixieSalander TixieSalander added the frontend Frontend concerns label Sep 18, 2025
tchoutri
tchoutri reviewed Sep 18, 2025
View reviewed changes
Comment thread assets/css/1-core/1-reset.css
/*
Hide polyfilled popovers
*/
@supports not selector(:popover-open) {

@tchoutri tchoutri Sep 18, 2025

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.

Amazing

@tchoutri
tchoutri merged commit b6a87c5 into flora-pm:development Sep 18, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tchoutri tchoutri tchoutri approved these changes

Assignees

No one assigned

Labels

frontend Frontend concerns

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@TixieSalander @tchoutri