[#70573] Fix AXe Accessibility error: invalid list structure

[myabc]

Ticket

https://community.openproject.org/wp/70573

What are you trying to accomplish?

• Fixes invalid HTML in top menu.
• Renders top and left menu inside semantic navigation sections (<nav> elements).
• Replaces headings with accessible names (aria-labels) - headings (particularly <h1> may confuse screen-readers about the structure of the page)

Screenshots

No visual changes.

Merge checklist

• Added/updated tests
• Added/updated documentation in Lookbook (patterns, previews, etc)
• Tested major browsers (Chrome, Firefox, Edge, ...)

[github-actions]

Caution

The provided work package version does not match the core version

Details:

• Work package URL: https://community.openproject.org/wp/70573
• Work package version: 17.0.1
• Core version: 17.1.0

Please make sure that:

• The work package version OR your pull request target branch is correct

[Copilot]

Pull request overview

This pull request fixes AXe accessibility errors related to invalid list structure in the top menu and main menu. The changes replace semantically incorrect HTML with proper navigation elements and ARIA labels.

Changes:

• Changed top menu wrapper from <ul> to <nav> with proper aria-label
• Removed redundant <h1 class="sr-only"> elements for top menu and main menu
• Changed main menu wrapper from <div> to <nav> with aria-label for proper semantic structure

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
lib/redmine/menu_manager/top_menu_helper.rb	Changed render_top_menu_left to use <nav> instead of <ul> and added aria-label for accessibility
app/views/layouts/base.html.erb	Removed redundant screen-reader-only headings and changed main menu wrapper to semantic <nav> element with aria-label