Mike Perrotti

Results 115 comments of Mike Perrotti

@siddharthkp - that doesn't seem right... 🤔 I'll investigate

@siddharthkp - I'm seeing that behavior in the preview deployment

We should talk to a11y about how this should be implemented.

> I believe it's likely design will agree with this proposal as I think this is already supported in PVC This is in fact what PVC (and PCSS) currently do....

@lesliecdubs - if the group agrees with this change, it would fit great in this PR: https://github.com/primer/react/pull/3582 If you prefer we do it in a separate PR, I'm cool with...

@megan-arellano - can we get a screenshot of what you're building so we can have more context?

Ok, I think I understand the issue now. I think we just need to update the [Rails implementation](https://view-components-storybook.eastus.cloudapp.azure.com/view-components/lookbook/inspect/primer/beta/details/playground) to align with the [progressive disclosure guidelines](https://primer.style/ui-patterns/progressive-disclosure). What do you think @jonrohan...

Closing because [NavList](https://primer.style/design/components/nav-list) supports this

@tallys - I'm already in touch with @joshblack about this. I don't think it needs a Primer patterns session.

Added @keithamus as a reviewer since he's familiar with [@github/tab-container-element](https://github.com/github/tab-container-element), which does a lot of the heavy-lifting here.