Kate Higa
Kate Higa
1. Based on what you know, can we rely on keyboard map API at this point? Are there polyfills we could use or would we need to write our own?...
We have the opposite of what you're suggesting right now where we generate previews from the YARD doc examples in the component files, then run tests against those. 😆 If...
👋 I'm not sure familiar with the state of dialog is currently but wanted to bring this [audit comment from @jscholes ](https://github.com/github/accessibility-audits/issues/275#issuecomment-1228870632) to your attention in case it hasn't been...
This was specifically observed in this design repo in a `.mdx` file. Here's a [related commit ](https://github.com/primer/design/pull/239/commits/5a443b46111e95a7f52012fb689b6839701bfcfe )that fixed it.
@andrialexandrou sorry I totally missed this notification! ~~It looks like this error will also need to be resolved here in [primer/experimental/navigation_list/item.html.erb](https://github.com/github/github/blob/master/app/components/primer/experimental/navigation_list/item.html.erb#L31) which seems to be actively in use. `@primer/rails-reviewers` where...
@dio5 Unfortunately not, I ended up abandoning the effort. I recommend joining the Discord group if you haven't already. There might be someone who can help now.
I'd love to see something like this too! We use Primer React where some components support an `as` prop so it'd be great if we can map a prop value...
@ljharb I don't believe there's currently an option to configure enable component mappings on a per rule basis. I'm thinking of this more as an enhancement of the [existing component...
Same as @jrdn91, I don't have a list of rules in mind. But take for instance, [heading-has-content](https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/main/docs/rules/heading-has-content.md) rule which currently does accept `components` like `Heading`. It'd be great if we...
Another rule that comes to mind is [role-supports-aria-props](https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/main/src/rules/role-supports-aria-props.js). Our team would benefit a lot from being able to discourage people from using `aria-label` on `` for instance.