Skip to content

Conversation

@amined159
Copy link

@amined159 amined159 commented Jan 16, 2025

Forcer un fond blanc pour les champs textarea et input et supprimer la ligne qui applique un fond transparent aux champs de texte.

Fix #2805

Contrôle qualité

Vérifier que dans le forum, après demande d'aperçu, ainsi que sur les inputs de connexion :

  • Si JavaScript est désactivé dans le navigateur
  • Si le mode sombre est activé sur le système
  • Si les deux conditions sont réunie

Alors, le texte doit rester noir sur fond blanc dans les champs de texte (textarea et input).

@coveralls
Copy link

coveralls commented Jan 16, 2025

Coverage Status

coverage: 89.305%. remained the same
when pulling cff6d10 on amined159:ticket0
into 8fcacfc on zestedesavoir:dev.

@amined159 amined159 changed the title Fixed Prémâchés #2805 Mettre un fond blanc sur les champs de texte textarea et input Feb 5, 2025
@Convolutio
Copy link
Contributor

Le fond blanc est bien forcé dans les champs d'écriture de réponses des forums et dans les champs d'entrée des informations de connexion. Ceci a été testé par vérification du css rendu dans Mozilla en localhost (je n'ai pas pu constater visuellement le problème spécifié par le ticket, même sans la modification de cette PR).

Voici les instructions de QA détaillées que j'ai appliquées :

Contrôle qualité

  1. Sourcer l'environnment de développement : source zdsenv/bin/activate
  2. Exécuter la commande python manage.py loaddata fixtures/users.yaml (permet de charger dans la base de données de test un faux utilisateur pour tenter de poster un message dans le forum et avoir donc un champ textarea d'affiché)
  3. Construire le frontend et le servir : make build-front && make run-back
  4. Sur le navigateur, se rendre sur localhost:8000
  5. S'assurer que le JavaScript est désactivé sur ce domaine local. Par exemple, sur Mozilla, il est possible d'installer l'extension NoScript et de mettre l'onglet ouvert en mode Untrusted.
  6. Mettre le système en apparence sombre. Le site peut également être mis en sombre de manière forcée avec l'extension DarkReader pour tester rapidement l'affichage dans les deux modes d'éclairages.
  7. Aller sur la page d'identification : http://localhost:8000/membres/connexion/
  8. Inspecter l'élément d'input pour entrer son identifiant. Vérifier avec l'inspecteur de style que le style a été corrigé comme il faut.
  9. Se connecter avec l'identifiant user et le mot de passe user.
  10. Se rendre dans une page de forum indexée dans http://localhost:8000/forums/
  11. Étant connecté, un champ en bas de la page pour ajouter une réponse devrait apparaître. Il s'agit d'un élément HTML de balise textarea. L'inspecter.
  12. Dans l'inspecteur de style du navigateur, l'élément devrait posséder la couleur de background (lire le champ CSS background-color) suivante :
    • rgb(250,250,250) en mode clair (soit du gris et non du transparent)
    • une couleur sombre créée par la mise en mode sombre de l'outil extérieur

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Puisque je n'arrive pas à reproduire le problème initial, j'ai juste regardé si l'élément input du formulaire de connexion avait bien un background-color: $white (OK) et sur le textarea du formulaire pour ajouter un message à un sujet sur le forum (pas OK, cf mon commentaire).


textarea {
margin: $length-10 0 $length-10 -#{$length-1};
background-color: transparent;
Copy link
Member

Choose a reason for hiding this comment

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

On dirait qu'il faut en réalité garder cette ligne, sinon le textarea pour ajouter un message dans un sujet sur le forum a un fond un peu gris au lieu de blanc. Si je rajoute cette ligne, on obtient bien un fond bien blanc.

Copy link
Member

Choose a reason for hiding this comment

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

Committé.

@github-project-automation github-project-automation bot moved this from En attente de QA to Modification demandée in Suivi des PR Feb 25, 2025
Notamment pour que l'ancien éditeur ait bien un fond blanc et non gris.
@philippemilink
Copy link
Member

J'ai committé la ligne qu'il fallait conserver. Le problème se posait avec l'ancien éditeur qui avait un fond gris au lieu de blanc sans cette ligne.

@philippemilink philippemilink moved this from Modification demandée to En attente de QA in Suivi des PR Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: En attente de QA

Development

Successfully merging this pull request may close these issues.

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

5 participants