-
Notifications
You must be signed in to change notification settings - Fork 68
feat(OO): Add an edit button to the toolbar in place of the FabButton #3310
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
Conversation
df87428
to
eabafe7
Compare
BundleMonFiles updated (4)
Unchanged files (14)
Total files change +1.99KB +0.03% Groups updated (4)
Unchanged groups (3)
Final result: ✅ View report in BundleMon website ➡️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je trouve que ça fait beaucoup de style et de custom 🤔 Est-ce qu'on ne pourrait pas mettre un Alert ou un PointerAlert dans un Tooltip pour remplacer tout le custom ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Et pour la couleur bleu, ne faut-il pas plutôt utiliser l'inverted (et donc wrapper dans un CozyTheme variant="inverted"
) plutôt que changer toutes les couleurs ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ce genre de tooltip est une première, à voir si le besoin se représente
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Et pour la couleur bleu, ne faut-il pas plutôt utiliser l'inverted (et donc wrapper dans un
CozyTheme variant="inverted"
) plutôt que changer toutes les couleurs ?
Ah pas vu ce message avant...
Non le tout n'est pas sujet aux changements de couleur
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tu utilises var(--primaryColor) qui va changer de couleur selon le thème si je dis pas de bêtise. En revanche tu forces un white pour le texte... Est-ce que tu as essayé en dark/light pour voir le rendu ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oui via l'OS, pas vu de changement, même si il est vrai que le BG devrait être le même que le bouton (thème dark sur le screen).
Le Tooltip serait en dehors du CozyTheme ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je ne sais pas il faut vérifier dans le DOM, c'est possible car ça doit utiliser popper. Mais de toute façon, comme je disais, mettre une couleur de fond primary et forcer du texte blanc n'est pas la bonne approche. Ici on veut un "inverted" à savoir du blanc sur fond bleu. Donc la manière de faire ça c'est de wrapper dans un cozyTheme avec la prop inverted, et non pas changer le style de chaque élément.
And no longer display the FabButton in Desktop mode
eabafe7
to
13ac994
Compare
And no longer display the FabButton in Desktop mode.

You can either hide the tooltip permanently or temporarily.