Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Storybook] Suppression du header bleu uniquement lors du build de storybook #898

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arnaudforaison
Copy link
Contributor

Before

image

After - proposition 1

Toolkit-Storybook-header-logo

After - proposition 2

Toolkit-Storybook-header

Donner moi votre avis sur les propositions, ainsi que vos retours de PR.
But ici: supprimer le script inject-version

jforatier
jforatier previously approved these changes Mar 17, 2021
@jforatier
Copy link
Contributor

Pas de préférences pour ma part. Légère tendance au minimalisme mais bon le logo a du sens ici.

bernard-arnaud
bernard-arnaud previously approved these changes Mar 19, 2021
@sisko59
Copy link
Contributor

sisko59 commented Mar 23, 2021

Préfèrence pour la proposition 2 afin d'avoir l'information de suite, ce qui me semble important pour une lib.
Pour la 1ère proposition, il faut que les utilisateurs aillent sur l'image et reste dessus pour "deviner" que la version s'affiche.

Une autre solution serait d'afficher la version en-dessous des logos ?
image
ou tout en bas des composants:
image

@romuleald
Copy link
Contributor

Préférence pour le 2sd thème, surtout que l'on pourrait mettre un logo en favicon

@arnaudforaison
Copy link
Contributor Author

Ajouter logo + titre version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants