Ajout des Échos #190
Ajout des Échos #190
Conversation
Lire sur europresse
Fix css + fix a weird loading issue
| function extractKeywords() { | ||
| return extractKeywordsFromTitle() || extractKeywordsFromUrl(window.location); | ||
| } | ||
|
|
||
| function extractKeywordsFromTitle() { | ||
| const titleElem = document.querySelector("h1").childNodes[0]; | ||
| return titleElem && titleElem.textContent; | ||
| } | ||
|
|
||
| function extractKeywordsFromUrl(url) { | ||
| /*const source_url = new URL(url); | ||
| const lemonde_match = source_url.pathname.match(/([^/.]+)(_\d*_\d*\.html)?$/); | ||
| if (!lemonde_match) throw new Error("Could not find keywords in lemonde url"); | ||
| return lemonde_match[1];*/ | ||
| throw new Error("not implemented"); | ||
| } |
There was a problem hiding this comment.
Vous pouvez enlever extractKeywordsFromUrl du coup
|
|
||
| async function onLoad() { | ||
| /* Weird reloading after load */ | ||
| await new Promise(r => setTimeout(r, 1000)); |
There was a problem hiding this comment.
Le 1000 semble un peu aléatoire, non ? Il ne faudrait pas un mutationObserver ici, pour être sûr que l'on a bien l'élément que l'on attend ?
| const article = document.querySelectorAll("article")[0]; | ||
| const elt = article.children[1].children[1].children[0].children[0].children[1].children[0] |
There was a problem hiding this comment.
ça semble très très fragile, comme ça... On ne peut pas trouver un sélecteur plus précis?
There was a problem hiding this comment.
Pas vraiment, toutes les classes sont obfusquées, de type "sc-kla0ai-1". Après un rechargement de la page elles semblent être toujours identiques mais je pense qu'elles peuvent changer n'importe quand. Une alternative serait de mettre le lien directement dans le titre, au moins il n'y a qu'un seul h1.
There was a problem hiding this comment.
Est-ce que l'on peut faire un query selector qui contient les tags des éléments, au moins?
Quelque chose comme article div[some_attr*=some_value] span
There was a problem hiding this comment.
En fait j'avais déplacé le bouton pour le mettre à côté des boutons "Commenter", "Partager", etc, ce qui donne :
document.querySelector("button[aria-label=Commenter]")?.parentElement?.parentElement
ce qui me semble plus robuste.
On regarde 3 événements différents : - changement dans <title> (nouvelle sous-page via un routeur javascript) - changement dans <main> (fin du chargement de la sous-page) - apparition d'une iframe (fin du chargement de la page initiale)
|
Nouveau commit avec des changements dans la logique de détection du chargement initial avec un mutationobserver. |
|
Super, c'est intégré, merci pour votre contribution ! |
Support for it had been added in lovasoa#190.
Support for it had been added in #190.
No description provided.