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

fix: Gradient not visible when homework is truncated #690

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

godetremy
Copy link
Contributor

🐛 [BUG] Gradient not visible when homework is truncated

Fixes of truncating the homework text doesn't show the gradient to see when the homework is too long.

@godetremy godetremy linked an issue Jan 25, 2025 that may be closed by this pull request
Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link
Contributor

@Clmnnt Clmnnt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼

@NathanBnm
Copy link
Contributor

NathanBnm commented Jan 26, 2025

@godetremy du coup ça entre en conflit avec #672

Faudrait se mettre d'accord : Soit on garde uniquement les points de suspension, soit uniquement le dégradé

Perso je préfère le dégradé mais faut pas faire les deux à la fois et faut que ce soit la même chose sur la page d'accueil et sur la page des devoirs

@codeuriii
Copy link
Contributor

Bah le dégradé marche pas chez moi donc faite comme bon vous semble, si vous arrivez a fix le dégradé tant mieux, sinon faudra garder le truncate

@NathanBnm
Copy link
Contributor

Y'a conflit aussi avec #688

@NathanBnm
Copy link
Contributor

Bah le dégradé marche pas chez moi donc faite comme bon vous semble, si vous arrivez a fix le dégradé tant mieux, sinon faudra garder le truncate

Tu as testé sur cette PR ? T'as quel OS ? Thème sombre ou thème clair ?

@codeuriii
Copy link
Contributor

normal vu que elle utilise aussi le truncate (enfin c plutot un fix du truncate car ca truncaté aussi dans l'expend)

@codeuriii
Copy link
Contributor

Bah le dégradé marche pas chez moi donc faite comme bon vous semble, si vous arrivez a fix le dégradé tant mieux, sinon faudra garder le truncate

Tu as testé sur cette PR ? T'as quel OS ? Thème sombre ou thème clair ?

je sais pas comment testé sur une pr, j'ai android et thème clair

@NathanBnm
Copy link
Contributor

Bah le dégradé marche pas chez moi donc faite comme bon vous semble, si vous arrivez a fix le dégradé tant mieux, sinon faudra garder le truncate

Tu as testé sur cette PR ? T'as quel OS ? Thème sombre ou thème clair ?

je sais pas comment testé sur une pr, j'ai android et thème clair

Du coup c'est normal que le dégradé fonctionne mal pour toi c'est justement le but de cette PR

@codeuriii
Copy link
Contributor

Ah bah il faut enlever le truncate alors

@ecnivtwelve ecnivtwelve merged commit 4d122b5 into main Jan 27, 2025
1 check failed
@godetremy
Copy link
Contributor Author

@godetremy du coup ça entre en conflit avec #672

Faudrait se mettre d'accord : Soit on garde uniquement les points de suspension, soit uniquement le dégradé

Perso je préfère le dégradé mais faut pas faire les deux à la fois et faut que ce soit la même chose sur la page d'accueil et sur la page des devoirs

C'est surtout qu'il faut pas être golmon, ça y était déjà mais ça fonctionnait juste pas ! On va pas faire 20 fonctions différentes pour chaque truc qui marche pas !

@NathanBnm
Copy link
Contributor

C'est surtout qu'il faut pas être golmon, ça y était déjà mais ça fonctionnait juste pas ! On va pas faire 20 fonctions différentes pour chaque truc qui marche pas !

Oui c'est moi qui l'avait développé et ça ne fonctionnait pas comme je voulais.

Je suis d'accord mais y'a pas de suivi chacun fait son truc de son côté c'est pour ça que j'ai précisé. Je suis d'avis de ne conserver que le dégradé maintenant que tu as corrigé le souci.

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.

[Bug]: devoirs coupés
6 participants