Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Menu - update drilldown inline filter example markup #9777

Open
thatblindgeye opened this issue Oct 27, 2023 · 1 comment
Open

Menu - update drilldown inline filter example markup #9777

thatblindgeye opened this issue Oct 27, 2023 · 1 comment

Comments

@thatblindgeye
Copy link
Contributor

Describe the enhancement or change
Currently the Menu with drilldown and inline filter example is rendering the search filter (drilled into level 2) as a div inside a ul element, causing an axe error to be flagged (since only li element should be direct children). Either the search input should be moved outside of the menu list or should be rendered within an li as well.

Is this request originating from a Red Hat product team? If so, which ones and is there any sort of deadline for this enhancement?

Any other information?

@mcoker
Copy link
Contributor

mcoker commented Nov 7, 2023

should be rendered within an li as well.

If we want to keep it in the list like that, one option could be to create a generic li that does not have any sort of hover or other treatment and could be used for putting a non-interactive type of thing in. Then the <MenuSearch> (or whatever) could be placed inside of it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Backlog
Development

No branches or pull requests

5 participants