8000 Be able to set custom properties on trigger element · Issue #146 · glennflanagan/react-collapsible · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content 8000
Be able to set custom properties on trigger element #146
@MattBodey

Description

@MattBodey

Describe the bug

When running accessibility audits using axe pro and enabling tab index, it was noted that there is no role on the Trigger element.

This lead to us realising that there is no way of passing props to the trigger element. In our case, this would be for aria roles and labels.

To Reproduce

Try and pass properties to the trigger element

Expected behaviour
There should be some way of setting additional properties on the trigger element. As there is already a containerElementProps prop, it would be nice to have a triggerElementProps prop on the Collapsible. This would allow the passing of an object that would set props on the TriggerElement

I'm happy to raise a PR for this if you would like

Metadata

Metadata

Assignees

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions

    0