UX/UI: mise a jour du theme vers v2.5.6#5340
Conversation
hellodeloo
changed the title
hellodeloo
added
the
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
| <link rel="shortcut icon" href="{% static_theme_images "favicon.ico" %}" type="image/ico"> | ||
| <link rel="stylesheet" href="{% static "vendor/theme-inclusion/stylesheets/app.css" %}"> | ||
| <link rel="stylesheet" href="{% static "vendor/easymde/easymde.min.css" %}"> | ||
| <link rel="stylesheet" href="{% static "vendor/theme-inclusion/stylesheets/app.css" %}"> | ||
| <link rel="stylesheet" href="{% static "css/itou.css" %}"> |
There was a problem hiding this comment.
Le changement d’ordre ne me semble pas indispensable ? Est-ce qu’il pourrait être fait dans une autre PR pour éviter de mélanger les choux et les carottes ?
(je pense que l’ordre proposé est plus logique : upstream < theme < itou overrides, mais ça ne me semble pas lié à la mise à jour du thème)
There was a problem hiding this comment.
Yes, avec l'aide de quelques !important pour la surcharge, je peux laisser l'ordre d'origine
francoisfreitag
left a comment
francoisfreitag
left a comment
There was a problem hiding this comment.
Ça m’a l’air bien autrement.
En testant, je viens de remarquer que easyMDE capture Tab, ce qui rend difficile la navigation au clavier...
hellodeloo
force-pushed
the
deloo/update-theme-to-v.2.5.6
branch
from
418e5bf to
ac7a73e
Compare
Yes, le focus bleu n'est qu'une aide visuelle, car effectivement le fake textarea n'est pas vraiment :focus et la capture du tab qui force le "focus" dans le fake textarea crée un "piège au clavier" et rend impossible la sortie au clavier :/ |
2 participants
🤔 Pourquoi ?
Pour harmoniser les styles des textarea EasyMDE avec les textarea du thème et ajouter une aide visuelle au focus
💻 Captures d'écran
Avant
Apres
