Leading and trailing icon position in button link

[aweell]

Permitir indicar la posición del asset icono en los botones:

Conecto con #631 por si tendría sentido juntar las dos en una release como update de éste componente.

Implementación

Usar gap en vez del margin para controlar el espaciado entre elementos.

Tal y como está ahora puede cambiarse la posición del asset en función de la estructura, pero no se respeta el espaciado correctamente:

[yceballost]

Qué casos crees que puede tener sentido para el trailing? En el caso del buttonLink es evidente que es para el chevron que plantea Movistar

[aweell]

Es un buen punto, habría que echarle un ojo a eso (Estaba contemplando el button como una única implementación <Button>) Pero son componentes separados...

Se me ocurre en primera instancia que se quieran usar botones como trigger de un dropdown menu:

Así de paso saco a la palestra otro tema que quería tocar con respecto al trigger del componente menu.

https://media.giphy.com/media/Hj7SSRD2lAeQ0/giphy.gif

[yceballost]

Yo si de momento no existe una necesidad real no lo metería, lo dejaría on hold.

[aweell]

Cambio el scope de la discussion únicamente a buttonLink

[yceballost]

Lo que se queda como discussion entonces es por un lado cómo nombramos a ese trailing / leading? y la implementación?

[aweell]

Se quedaría:

1. Si tiene sentido que existan dos posiciones para el icono en el <ButtonLink> ya que en el ejemplo de Add icon to ButtonLink #631 sólo se muestra una.
2. En el caso de que lo tenga, si leading o trailing son el nombre apropiado para indicar estas posiciones (También es habitual encontrarlo como start/end o before/after)
3. (Dejaría los detalles de implementación para la definición de las specs)

[yceballost]

Finalmente vamos con la opción Leading y Trailing para la nomenclatura de la posición del icono.