Skip to content

Conversation

Merkur39
Copy link
Contributor

And no longer display the FabButton in Desktop mode.
You can either hide the tooltip permanently or temporarily.
image

@Merkur39 Merkur39 force-pushed the feat/front-1313 branch 2 times, most recently from df87428 to eabafe7 Compare January 28, 2025 15:57
Copy link

bundlemon bot commented Jan 28, 2025

BundleMon

Files updated (4)
Status Path Size Limits
intents/drive.(hash).js
86.62KB (+705B +0.8%) -
public/drive.(hash).js
2.09MB (+551B +0.03%) -
app/drive.(hash).js
160.75KB (+546B +0.33%) -
vendors/drive.(hash).js
2.32MB (+234B +0.01%) -
Unchanged files (14)
Status Path Size Limits
public/pdf.worker.entry.(hash).worker.js
345.35KB -
services/qualificationMigration/drive.js
283.72KB -
services/dacc/drive.js
263.3KB -
public/cozy-client-js.js
159.28KB -
public/drive.(hash).min.css
39.53KB -
onlyOffice/slide.pptx
24.83KB -
app-drive.(hash).min.css
10.16KB -
intents-drive.(hash).min.css
8.43KB -
onlyOffice/text.docx
5.85KB -
onlyOffice/spreadsheet.xlsx
5.02KB -
manifest.webapp
1.92KB -
index.html
598B -
intents/index.html
509B -
manifest.json
185B -

Total files change +1.99KB +0.03%

Groups updated (4)
Status Path Size Limits
intents/**
87.12KB (+705B +0.8%) -
public/**
2.63MB (+552B +0.02%) -
app/**
160.75KB (+546B +0.33%) -
vendors/**
2.32MB (+234B +0.01%) -
Unchanged groups (3)
Status Path Size Limits
services/**
547.03KB -
onlyOffice/**
35.7KB -
img/**
5.85KB -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

Copy link
Contributor

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 ?

Copy link
Contributor

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 ?

Copy link
Contributor Author

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

Copy link
Contributor Author

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

Copy link
Contributor

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 ?

Copy link
Contributor Author

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 ?

Copy link
Contributor

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
@Merkur39 Merkur39 merged commit 150d88c into master Jan 29, 2025
4 checks passed
@Merkur39 Merkur39 deleted the feat/front-1313 branch January 29, 2025 09:48
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.

2 participants