Problem/Motivation

🐛 AmĂ©liore l'affichage des Ă©lĂ©ments de l'en-tĂȘte et de la navigation en mobile #1367 (+ docs translate #1431)

Mostly CSS, so nothing to change for that part. Three things to check on our side, probably small or nothing:

  • Upstream now wraps a lone shortcut button in <div class="fr-btns-group"> . tools_links is a free slot in header.twig: check our stories/doc show the wrapper.
  • aria-current is now styled on expandable N1 .fr-nav__btn at all breakpoints. nav_menu.twig already sets aria-current on active-trail buttons: verify rendering after the update.
  • Translate doc says no <nav> wrapper anymore: translate.twig already uses a div. Nothing to change.

Issue fork ui_suite_dsfr-3615329

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

just_like_good_vibes created an issue. See original summary.

just_like_good_vibes opened merge request !218

just_like_good_vibes’s picture

just_like_good_vibes
Location PARIS
commented
Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.