[Résolu] Fermer un pop-up par script externe

Coucou,

Pourquoi y a t-il deux liens pour fermer une seule fenêtre ?

document.querySelectorAll('p a,') Il y a une virgule en trop 'p a'

function (e) // Il faut mettre un argument pour l'événement
{
   window.close ();
   e.preventDefault(); // et Ici il faut s'en servir
}

Pour l'ouverture et la fermeture de la fenêtre t'as un exemple là
https://developer.mozilla.org/fr/docs/Web/API/Window/close

Bonjour,

Merci de ton suivi.

J'ai fait tes corrections et cela marche.

En plus, j'avais oublié d'appeler le script dans la page html...

Accessoirement je passe le lien dans body, tout à la fin de body, donc juste au dessus de </body>

<script src="dp/pop_ferme.js"></script>

C'est la bonne place, non ?

const ferme = document.querySelectorAll('p.lome a:last-child, p.lom a:last-child, div.sep a');
for (let i = 0; i < ferme.length; i++)
{
ferme[i].addEventListener ('click', function (e)
{
window.close ();
e.preventDefault();
}
);
}

PS : il y a plusieurs liens Fermer car le document comprend de nombreuses rubriques, un peu comme une FAQ.
Le bouton Fermer est répété à la fin de chaque rubrique.
Je pourrais mettre un seul lien Fermer en position : fixed ou avec une Flexbox mais c'est une conception ancienne qui a son charme.
Modifié par boteha_2 (13 Jul 2020 - 19:27)

OK c'est cool
Modifié par Dev-Web-06 (13 Jul 2020 - 20:11)

Ha pardon j'avais pas bien lu.

Oui avant </body>c'est la bonne place pour charger ton script.

Dev-Web-06 a écrit :
Oui avant </body> c'est la bonne place pour charger ton script.

Merci.

Je pense que je vais remplacer mes <a href="#"> par des balises <button> plus sémantiques.

Ensuite je coche Résolu.
Modifié par boteha_2 (14 Jul 2020 - 11:40)

Bonjour,

J'ai remplacé les <a href="#">Fermer</a> par <button type="button">Fermer</button>

Pour donner à button l'apparence d'un lien :

button {background: none; border: 0; padding: 0; margin: 0; font-size: 1em; text-decoration: underline; cursor: pointer; color: #votre-choix}

button:hover {votre-choix}

Le javascript devient encore plus simple.

const ferme = document.querySelectorAll('button');
for (let i = 0; i < ferme.length; i++)
{
ferme[i].addEventListener ('click', function ()
{
window.close ();
}
);
}

Je n'ai plus besoin de la méthode preventdefault () car il n'y a pas de comportement par défaut.

Merci de ton aide, comme je suis prudent j'attends un peu avant de cocher Résolu.

Je coche Résolu.
Merci de ton aide.