Pagefind 1.5#1480
Conversation
djLintLinted 849 files, found 0 errors. HugolintDirectories are perfect ✅Partials calls are perfect ✅Too many lines (1)Files should not be too long, it's a sign of mess and a difficulty for overrides. 0 ❌
Complexity is perfect ✅ |
| for (attribut in window.osuny.pagefindOptions) { | ||
| options[attribut] = window.osuny.pagefindOptions[attribut]; | ||
| if (this.elements.buttonOpen) { | ||
| this.elements.buttonOpen.addEventListener('click', this.toggleDetails.bind(this)); |
There was a problem hiding this comment.
c'est préférable d'avoir open / close plutôt que toggle dans le cas où ce sont 2 boutons je trouve
| title.setAttribute('aria-level', '2'); | ||
| window.osuny.Search.prototype.resize = function () { | ||
| if (isMobile()) { | ||
| this.openDetails(false); |
There was a problem hiding this comment.
oui vraiment close() est beaucoup plus clair
|
|
||
| if (this.elements.buttonOpen) { | ||
| this.elements.buttonOpen.addEventListener('click', this.toggleDetails.bind(this)); | ||
| this.elements.buttonOpen.addEventListener('click', this.openDetails.bind(this)); |
There was a problem hiding this comment.
c'est quoi details ? on peut pas juste open ?
There was a problem hiding this comment.
j'ai pensé qu'il fallait préciser mais tu as raison, on pourrait s'en servir pour autre chose
| }.bind(this)); | ||
| } | ||
|
|
||
| window.osuny.Search.prototype.updateDocumentAccessibility = function (isOpen) { |
There was a problem hiding this comment.
on est dans une logique objet, on a déjà l'info sur l'ouverture, il faut la prendre dans this et pas la passer à la fonction il me semble
| window.osuny.Search.prototype.updateDocumentAccessibility = function (isOpen) { | ||
| console.log(isOpen) | ||
| document.body.style.overflow = isOpen ? 'hidden' : 'auto'; | ||
| ariaHideBodyChildren(this.elements.root, isOpen); |
There was a problem hiding this comment.
idem, il faut s'appuyer sur l'état de l'objet, pas faire du fonctionnel avec des paramètres
A11YProblèmes de pagefind :
Changements par rapport à la version actuelle
|
3 participants
Type
Description
Niveau d'incidence