search

milieuinfo / webcomponent-vl-ui-wizard

1 stars 0 forks source link

[BUG] - Knoppen plakken aan elkaar als ze onder elkaar komen te staan #30

Closed driesva closed 4 years ago

driesva commented 4 years ago

Omschrijf het probleem Wanneer de navigatie slots niet op 1 lijn passen, komen die onder elkaar en plakken ze vast aan elkaar. Wanneer de standaard vl-button-link wordt gebruikt is dit niet echt een issue, maar wel als dit een ander button is.

Hoe te reproduceren

                      <button is="vl-button" type="button" slot="next-action" secondary>
                            Naar de vorige stap gaan
                        </button>
                        <button is="vl-button" type="button" slot="next-action">
                            Naar de volgende stap gaan
                        </button>

Gewenst gedrag Ruimte ertussen

Screenshots In onze app: image

Even gesimuleerd met demo: image

Omgeving (gelieve aan te vullen):

coemans commented 4 years ago

@driesva het is eigenlijk de bedoeling dat de vl-button-link gebruikt wordt, dan is er geen probleem.

driesva commented 4 years ago

Wel ja, maar in afwachting van de stappen te beheren via code hadden wij momenteel 1 slot "misbruikt" en ik heb dit issue zo opgemerkt.

Is minor, maar als je de afnemer de optie geeft om daar iets willekeurig te zetten, lijkt een goede weergave ook nodig.

coemans commented 4 years ago

Het doel van die vrijheid is hopelijk niet om af te kunnen wijken van de huisstijl. Er wordt geadviseerd om een vl-button-link te gebruiken om terug te gaan naar een vorige stap en een vl-ui-button primary variant om verder te gaan naar de volgende stap. Het doel van #26 is om in uitzonderlijke gevallen de rendering applicatief over te nemen of programmatisch te kunnen navigeren tussen stappen. Als die richtlijnen gevolgd worden, lijkt het me niet nodig om dit issue op te lossen.

Aan @kevcraey om alsnog in te plannen.

driesva commented 4 years ago

In ons geval gaat het over de 1ste stap, waar er dus geen "vorige" is maar enkel afmelden/ "volgende". In plaats van het slot te misbruiken zullen we de knoppen zelf definiëren conform de huisstijl.

Aan jullie de beslissing of jullie dergelijke zaken al dan niet aanpakken, ik meld gewoon maar.

coemans commented 4 years ago

Als het puur over de knoppen gaat los van de context van een vl-ui-wizard, is het eerder een bug voor vl-ui-action-group. Ik merk dat AIV ook last heeft van dit gedrag, maar het issue kan op dat niveau zeker aangekaart en opgelost worden.