Welcome-Seite fixen - Githubissues

kickstartDS / kickstartDS-storybook

Our own, kickstartDS branded Storybook instance. Have a look for inspiration!

https://www.kickstartDS.com/storybook/

5 stars 2 forks source link

Welcome-Seite fixen #20

Closed julrich closed 3 years ago

julrich commented 3 years ago

Hier gibt es noch ein paar Dinge zu bereinigen:

[x] In den Teaser Box-Elementen habe ich ziemlich hässliche Links verwendet (e.g. /?path=/story/content-visual--box-hell), die lösen z.B. einen Reload aus. Das sollte mit den korrekten, internen Storybook-Links ersetzt werden
[x] ~~Das Layout lässt sich hier irgendwie nicht auf 100% stellen, das sieht sonst leider etwas komprimiert aus~~
[ ] Ein paar kleine visuelle Bugs (treten irgendwie nicht immer auf?)
- Manchmal sind Buttons nicht small
- Teilweise hat Visual kein Layout für die Text Box mehr

grafik

julrich commented 3 years ago

Layout geht bestimmt auch insgesamt schöner. Können bestimmt @DanielLeyUX und @fleven-kds Input liefern. Wollte nur erstmal einen minimalen Inhalt als Startpunkt erzeugen!

lmestel commented 3 years ago

Die Links konnte ich fixen. War aber etwas umständlich, weil die Docs eigentlich nicht dazu gedacht sind, komplexe React-"Seiten" zu bauen, sondern Markdown-Fließtext mit eingebetteten Stories bzw. Komponenten.

Daher kann man auch nicht das Layout so einfach umstellen :/

Die Button-Option "small" funktioniert in Content-Boxen nicht. Das ist KDS-Bug: https://github.com/kickstartDS/kickstartDS/issues/401

julrich commented 3 years ago

Dann sollten wir den Bug in kickstartDS noch fixen, das taucht auch an ein, zwei anderen Stellen noch auf (z.B. die Boxen ganz unten auf der Landingpage, die auch eigentlich kleine Buttons haben sollten).

Beim Layout bleiben wir dann einfach bei der aktuellen Breite. Vielleicht können @fleven-kds und @DanielLeyUX hier den Inhalt / das Aussehen noch ein wenig optimieren.

Hier auch noch mal der Link zu der Seite im Storybook: https://preview.kickstartds.com/storybook/?path=/story/welcome--page

lmestel commented 3 years ago

Button size ist gefixt.

julrich commented 3 years ago

Muss das Storybook dafür noch mal neu gebaut und deployed werden? Ich glaube aktuell ist es noch nicht sichtbar :thinking:

Danach kannst du das Ticket gerne auf @DanielLeyUX stellen!

julrich commented 3 years ago

Nachdem ich mit @lmestel noch mal drauf geschaut habe, der kleine Button ist da und funktioniert! Also wirklich vielleicht noch mal den Inhalt reviewen / anpassen vom Design.

julrich commented 3 years ago

Für mich passt jetzt auch alles, bis auf ein Detail. Tatsächlich taucht jetzt der letzte offene Task aus dem Ticket oben wieder auf: Teilweise hat Visual kein Layout für die Text Box mehr

Kann auch gut sein, dass ich da etwas falsch interpretiere, aber sollte die Box nicht schmaler sein? Wenn sie statt 100% Breite eher 2/3 Breite hätte, sähe es wesentlich stimmiger aus!

Vielleicht kannst du das noch fixen, @fleven-kds

julrich commented 3 years ago

Für mich passt jetzt auch alles, bis auf ein Detail. Tatsächlich taucht jetzt der letzte offene Task aus dem Ticket oben wieder auf: Teilweise hat Visual kein Layout für die Text Box mehr

Kann auch gut sein, dass ich da etwas falsch interpretiere, aber sollte die Box nicht schmaler sein? Wenn sie statt 100% Breite eher 2/3 Breite hätte, sähe es wesentlich stimmiger aus!

Und mobil sollte wir wohl den Button noch etwas optimieren, im Zweifel schreiben wir einfach eine gezielte CSS-Klasse für genau den Button!

Vielleicht kannst du das noch fixen, @fleven-kds

DanielLeyUX commented 3 years ago

im Browser sind die hover der buttons übrigens "Anti-A11Y :) Also zb so sieht das onHover aus oder so:

julrich commented 3 years ago

Jetzt sieht für mich alles gut aus, ich schliesse mal!