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] Background pour textinput #5137

Closed
wants to merge 2 commits into from
Closed

[FIX] Background pour textinput #5137

wants to merge 2 commits into from

Conversation

NicolasStr
Copy link

Ajout de background par défaut sur les input afin d'éviter une modification par le thème d'Ubuntu/autre
Exemple: avec un thème sombre le texte s'écrivait noir sur noir

Rafraichissement du cache style requis

Contrôle qualité

Vérifier avec un thème Ubuntu :)

Nicolas Str****** added 2 commits December 10, 2018 22:43
Add Background for textinputs.
Ubuntu dark themes default it to black and render it as black font over blakc box
@coveralls
Copy link

Coverage Status

Coverage remained the same at ?% when pulling 8920aef on NicolasStr:dev into 3589cb6 on zestedesavoir:dev.

@Situphen Situphen added C-Front Concerne l'interface du site S-Évolution Ajoute de nouvelles fonctionnalités labels Dec 10, 2018
@artragis
Copy link
Member

Salut !

bienvenu parmi nous ! Merci beaucoup pour ta participation.

Nous allons tester ton fix dès qu'on aura un peu de temps. Sache néanmoins que vue la manière dont nous organisons nos release, la correction n'apparaîtra que dans la v28.1 (donc pas tout de suite).

@NicolasStr
Copy link
Author

Salut!

Pas de soucis!

@A-312
Copy link
Contributor

A-312 commented Dec 12, 2018

Ca corrige la #5017 ?

@A-312
Copy link
Contributor

A-312 commented Dec 12, 2018

Il faudrait peut être en profiter pour modifier : #2805

Je pourrais QA avec mon ubuntu quand je l'aurais sous la main (si personne le fait avant).

@NicolasStr
Copy link
Author

Je peux faire ça d'ici 3 jours si besoin

@artragis
Copy link
Member

@A-312 tu as pu tester?

@A-312
Copy link
Contributor

A-312 commented Dec 17, 2018

@artragis Si quelqu'un le fait avant ça sera mieux, je n'ai mon ubuntu sous la main (ça sera après les fêtes).

@A-312
Copy link
Contributor

A-312 commented Dec 17, 2018

@Eskimon j'ai vu que tu avais ubuntu, as tu l'espaces de dev aussi ?

@Eskimon
Copy link
Contributor

Eskimon commented Dec 17, 2018

Oui, tout mon dev se fait sur ubuntu, zds inclus

@Eskimon
Copy link
Contributor

Eskimon commented Dec 18, 2018

:D j'ai répondu sans regarder le texte avant. Il faut que je fasse la QA. Entendu, je vais m'en occuper de suite.

@Eskimon
Copy link
Contributor

Eskimon commented Dec 18, 2018

Alors, y a un souci :/

Voilà le résultat que j'obtiens, certains champs sont blancs, d'autres sont gris (et sans le patch, tout les champs sont blancs sur ma config de base).

screenshot from 2018-12-18 10-06-12
screenshot from 2018-12-18 10-06-44
screenshot from 2018-12-18 10-06-58

Aussi, point de vue process, il faudrait que tu travailles sur une branche issue de dev' mais pas sur dev' directement ;) .

@artragis
Copy link
Member

@NicolasStr tu peux prendre en compte le retour de Eskimon stp?

@NicolasStr
Copy link
Author

Alors travailler sur sans branche issue de dev je sais pas trop comment faire mais le patch sera fait d'ici demain, j'ai eu des petits soucis !

@artragis
Copy link
Member

On en est où ici.

@A-312 A-312 removed their assignment Sep 10, 2019
@Situphen Situphen added the S-Zombie Ticket ou PR oubliée label Jan 1, 2020
@Situphen
Copy link
Member

Situphen commented Jan 1, 2020

@NicolasStr Cette PR est inactive depuis un an et tu as supprimé ton fork, donc je ferme. N'hésites pas à nous dire si tu veux la continuer ou faire une autre contribution !

@Situphen Situphen closed this Jan 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-Évolution Ajoute de nouvelles fonctionnalités S-Zombie Ticket ou PR oubliée
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants