Skip to content

NavList.Group should pass all props through to the ActionList.Group #2132

Closed
@mattcosta7

Description

@mattcosta7

Describe the bug

const Group = ({title, children, sx: sxProp = {}}: NavListGroupProps) => {

(as a nit here - using sx = {} in the function definition means every render by default supplies a new sx object. sx = defaultSx, where previously const defaultSx = {} would avoid that

NavList.Group only accepts a handful of props from ActionList.Group, and should probably accept almost/all and pass them through, specifically for things like aria attributes and data props, for test ids, etc

Note: As this is a public repo, please be careful not to include details or screenshots from unreleased GitHub products or features. In most cases, a good bug report should be able to describe the new component without including business logic or feature details, but if you must discuss context relating to an unreleased feature, please open an issue in the private Design Systems repo and link to it here.

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions