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

Couleur de texte modifiée, mais couleur de fond par défaut #2805

Open
jonathanpoelen opened this issue Jun 7, 2015 · 11 comments · May be fixed by #6697
Open

Couleur de texte modifiée, mais couleur de fond par défaut #2805

jonathanpoelen opened this issue Jun 7, 2015 · 11 comments · May be fixed by #6697
Labels
C-Front Concerne l'interface du site Facile Bon ticket pour débuter pour rejoindre le développement !

Comments

@jonathanpoelen
Copy link

Dans le forum, après demande d'aperçu, si javascript désactivé et sur une configuration de couleur sombre, le texte est noir sur fond gris foncé pour le textarea. Pareil pour les inputs de connexion.

Le gris foncé vient de ma couleur système.

@Eskimon Eskimon added Evolution C-Front Concerne l'interface du site labels Jun 15, 2015
@jonathanpoelen
Copy link
Author

Plus gênant, cela se fait aussi avec les messages édités.

@Situphen
Copy link
Member

Donc si je résume, tu as changé la couleur de fond des textareas et inputs en gris ?

@jonathanpoelen
Copy link
Author

Pas directement: j'ai changé le thème de mon bureau.
Cela impact les couleurs par défauts du navigateur.

@Situphen
Copy link
Member

Pour moi ce n'est pas le problème de ZdS si les paramètres de ton thème affectent les couleurs par défaut du navigateur ! Je ne pense pas que ZdS puissent y faire quelque chose. Quel est ton OS ? Quel est ton thème ? :)

@jonathanpoelen
Copy link
Author

Kubuntu, thème Wonton Soup.

Le problème vient surtout que ZdS ne modifie qu'une partie des couleurs par défaut.
Sur le css actuel (main.min.1155581.ceecc.css, ligne 687 (avec les outils de dev de firefox)) on peut voir une règle pour body,button,html,input,select,textarea avec color: #222 sans background: ... associé.

@Situphen
Copy link
Member

Du coup là effectivement on devrait pouvoir mettre un background: ...; !

@gustavi gustavi changed the title Couleur de texte modifier, mais couleur de fond par défaut Couleur de texte modifiée, mais couleur de fond par défaut Dec 16, 2015
@GerardPaligot
Copy link
Member

@jonathanpoelen C'est uniquement quand le JS est désactivé ? Parce que ZdS ne supporte plus le JS désactivé.

@jonathanpoelen
Copy link
Author

@GerardPaligot: Avec JS activé, il n'y a le problème au niveau de l'aperçu dans le forum. Par contre, tous les autres inputs/textarea l'ont toujours (création d'article/sujet, modification de message dans le forum, recherche, connexion, etc).

@artragis artragis added the Facile Bon ticket pour débuter pour rejoindre le développement ! label Aug 22, 2018
@Vanadiae
Copy link
Contributor

je confirme (pour avoir eu ce bug dans une précédente version de mon navigateur et avec thème système sombre), ce background est bien défectueux.

@Arnaud-D Arnaud-D removed the Facile Bon ticket pour débuter pour rejoindre le développement ! label Apr 21, 2020
@Situphen
Copy link
Member

Je vois deux solutions pour corriger ce soucis :

  • soit assumer que tout le site est en thème clair et donc mettre un fond blanc sur les champs de texte textarea et input ;
  • soit faire en sorte que ces champs de texte s'adaptent au thème clair ou sombre utilisé et donc ne pas forcer notre couleur de texte (qui est très proche du noir complet donc visuellement il n'y aura pas grand différence).

Quelle est selon vous la meilleure option ?

@Situphen
Copy link
Member

Après en avoir discuté avec @AmauryCarrade, je pense que la première option est la moins risquée donc on va partir sur celle-ci. Il sera toujours temps plus tard de mettre en œuvre la deuxième option si envie.

Instructions :

Supprimer la ligne suivante :

background-color: transparent;

Dans le fichier assets/scss/base/_base.scss vers la ligne 79, ajouter le code suivant :

textarea,
input[type="text"],
input[type="password"] {
    background-color: $white;
}

Cela devrait faire l'affaire.

amined159 pushed a commit to amined159/zds-site that referenced this issue Jan 16, 2025
@amined159 amined159 linked a pull request Jan 16, 2025 that will close this issue
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 Facile Bon ticket pour débuter pour rejoindre le développement !
Projects
Status: À traiter
Development

Successfully merging a pull request may close this issue.

7 participants