[solution 5.8.x] admin article.php : erreur de toggles chapo & tag toujours fermés au démarrage
Sudwebdesign
Member
Bonjour a tou(te)s,
Merci aux équipes, superbe surprise il est en 5.8.2 et il tourne bien.
Lorsque l'on édite un article, il y a une légère régression au niveau des bascules (toggles) pour Chapô et Mot-clés.
Dans les moutures précédentes (..., 5.6, 5.7) si le chapô a du texte, il s'affiche, maintenant (5.8.x) non, il sont toujours masqués (en fait depuis le 28 avril 2019, #207 Afficher/masquer le chapeau et les tags d'un article en CSS3).
Comme la remarqué Bazooka07 l'attribut unchecked est inexistant en html, les erreurs sont là*.
Solution pour que cela tombe en marche :
[fichier : core/admin/article.php]
Il suffit de remplacer les deux ' unchecked' :
<?php echo (empty($_GET['a']) || ! empty(trim($chapo))) ? ' unchecked' : ''; ?>
···
<?php echo (empty($_GET['a']) || ! empty(trim($tags))) ? ' unchecked' : ''; ?>
par ' checked' :
<?php echo (empty($_GET['a']) || ! empty(trim($chapo))) ? ' checked' : ''; ?>
···
<?php echo (empty($_GET['a']) || ! empty(trim($tags))) ? ' checked' : ''; ?>
Un grand merci @bazooka07 car; comme prévus par le sieur; (et avec les correctifs), le chapô est ouvert pour un nouvel article 🤩 et a l'ancienne lorsque le chapô est peuplé (idem pour les tags).
Ps : La méthode employé me rappelle beaucoup celle du menu admin en mode mobile.
Ps² : Mea-culpa, c'est passé a l'as à l'époque 😯
*L’attribut attendu ici est checked et comme il est absent; AMHA, remplacé par unchecked pour test; les élément censés être ouvert ne le sont pas, car pour qu'une case à cocher soit décoché il suffit d'omettre checked 😉
Réponses
Hello,
Merci pour la PR, je regarde ça dès que possible.