[DONE] Improve A11y of chapter edition form #938
Conversation
…inputs in chapters.js + add aria-describedby attributes on all channel fields
+ remove useless nbsp;
pod/chapter/static/js/chapters.js
Outdated
| let inputStart = document.getElementById("id_time_start"); | ||
| if (inputStart) { | ||
| inputStart.insertAdjacentHTML( | ||
| "beforebegin", | ||
| " <span class='getfromvideo'><a id='getfromvideo_start' class='btn btn-primary btn-sm'>" + | ||
| gettext("Get time from the player") + | ||
| "</a><span class='timecode'> </span></span>", | ||
| "</a><span class='timecode' id='timecode_start'> </span></span>", |
There was a problem hiding this comment.
je me posais la question sur des id en dur dans les champs si il n'y a pas un risque de doublon. Un id doit etre unique dans un DOM
There was a problem hiding this comment.
Oui, j'ai justement supprimé des ID en doublons dans cette PR ;)
pour id='timecode_start', il ne s'affiche que dans le cas de l'ajout/modification d'un chapitre, mais on pourrait l'appeler "chapter_time_start" pour qu'il soit moins générique.
| @@ -51,6 +51,7 @@ | |||
|
|
|||
|
|
|||
| <!-- quality selector --> | |||
| <!-- NOTE: hls-quality-selector doesn't seem to work on Safari. --> | |||
There was a problem hiding this comment.
il est mis à jour dans ma PR #925 , je suis en train de passer à videojs8
There was a problem hiding this comment.
Tant mieux ! Si ca remarche avec Safari on pourra enlever le commentaire ^^
| @@ -205,7 +204,7 @@ | |||
| {% endif %} | |||
|
|
|||
| {% if video.chapter_set.all %} | |||
| player.videoJsChapters(); | |||
| player.podVideoJsChapters(); | |||
There was a problem hiding this comment.
idem, les chapitres seront remplacés dans ma PR #925
There was a problem hiding this comment.
d'acodac. je laisse ca en attendant sinon y'a des trucs qui ne marchent plus ^^
Badatos
changed the title
There was a problem hiding this comment.
Un console.log n'est pas commenté. J'imagine que c'est fait pour ?
There was a problem hiding this comment.
Oui je me suis dit que ce serait bien d'indiquer dans la console cette info si jamais on a un cas ou les enrichissements ne s'affichent pas. Quand tout va bien il ne devrait rien afficher ^^
There was a problem hiding this comment.
Un détail encore : la ligne 39 pourrait être mise sur plusieurs lignes :)
Merci à toi ! |
* Improve A11y of chapter edition form * Coorect bug on video enrichments * correct bug on chapters.js * Improve chapter validation process * link with aria-describedby displayed "timecode" to "start" and "end" inputs in chapters.js + add aria-describedby attributes on all channel fields * correct some missing podVideoJsChapters + remove useless nbsp; * Replace id="timecode_start" by id='chapter_time_start' (less generic) * QoC (multi lines for attrs loop in customfilewidget)
Work on RGAA 11.04 :
"Dans chaque formulaire, chaque étiquette de champ et son champ associé sont-ils accolés (hors cas particuliers) ?"
P13 (Chapter edition form)
Non-conformité :
Piste de correction :
Champs obligatoires *
comme sur certaines pages (contact par exemple)P11 (Channel edition form)
Non-conformité :
Piste de correction :