vtex / shoreline

VTEX Design System for back-office experiences. Currently available for internal usage at VTEX.
https://shoreline.vtex.com
24 stars 1 forks source link

Review Variant prop in the Menu component #1561

Open beatrizmilhomem opened 5 months ago

beatrizmilhomem commented 5 months ago

In the Menu documentation, it includes all Button variants as props of the Menu. In Figma, the Menu only has a primary and tertiary variants.

davicostalf commented 1 month ago

It's missing the More actions variant, with the three dots icon on the left of the menu button.

@beatrizmilhomem This is already covered in #1741. I'm changing this issue to focus on the other problem.

davicostalf commented 1 month ago

Relevant discussion happening in #1847

lafray commented 1 month ago

It's better remove the status from backlog in this case, no?

lucasaarcoverde commented 1 month ago

@davicostalf o que acha de fecharmos essa #1847 e trazer a discussão dela pra ca? Já que é a mesma discussão e um ponto de contra argumento a ter apenas primary e tertiary no Figma

Em relação ao Menu ter toda a capacidade de variant do Button no código é porque é melhor que o componente seja extensível e evite breaking changes caso novos comportamentos surjam, como por exemplo o caso que o @davicostalf trouxe na outra issue, que provavelmente não estava mapeado quando especificamos o Menu

beatrizmilhomem commented 1 month ago

@lucasaarcoverde mesmo que o componente tenha essa capacidade, seria possível limitar a implementação de alguma forma? Ou isso teria que ficar apenas a nível de documentação mesmo?