search

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 Type prop in the Menu component #1741

Open beatrizmilhomem opened 3 months ago

beatrizmilhomem commented 3 months ago

Problem

Prop is inconsistent with Figma.

Expected behavior

The values should be the same in Figma and code.

Suggested solution

Change: actions → moreActions menu → customLabel

moreActions should be the default.

lafray commented 2 months ago

This change generates a breakchange. How are you dealing with this now that you release 1.x?

lafray commented 2 months ago

The issues, #1561, #1847 and this one, are all breakchanges. Isn't it better to put all of these into a single issue and close the others, to make a well-thought-out breakchange and avoid someone pulling it hastily?

lucasaarcoverde commented 2 months ago

This change generates a breakchange. How are you dealing with this now that you release 1.x?

@lafray nesses casos podemos deprecar o comportamento atual sem gerar uma breaking change e adicionar os novos valores

No entanto, creio que é melhor entrarmos em um acordo e atualizar no Figma ao invés do código já que seria menos conflitante com o uso da lib, o que acha @beatrizmilhomem @davicostalf ?

beatrizmilhomem commented 2 months ago

@lucasaarcoverde estou aberta a atualizar o Figma, mas o Menu ter uma prop value que se chama menu não faz sentido. Concorda? Tem alguma outra proposta?

davicostalf commented 3 weeks ago

Conversamos sobre essa issue hoje na weekly. Chegamos na conclusão que nem a solução atual no código nem a solução do Figma são intuitivas. Portanto, não faria sentido alterar o código para ter match com o Figma. Podemos documentar melhor a prop e continuar a pensar numa solução para comunicar essa variação na API. Vamos levar a solução do código para o Figma por enquanto para evitar a inconsistência.