patternfly / patternfly/patternfly-react
Menu - update drilldown inline filter example markup
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 862
- Forks
- 392
- Avg merge
- 4d 8h
- Merged PRs (30d)
- 9
Description
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?
Jira Issue: PF-112
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start from the PatternFly Menu documentation example, “with drilldown and inline filter,” and locate its React/TypeScript example source. Run the relevant accessibility checks or axe test, then adjust the filter markup so the menu’s ul contains only valid li children or the filter is outside the list; done means the example renders correctly without the reported axe error.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- accessibility, frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 62/100