Discussion :

[boteha]

Bonjour,


J'ai deux problèmes.

1)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

var mobile = window.matchMedia ('(min-width: 768px)');

Je veux calculer la valeur au chargement de la page et au resize de la fenêtre.

2)

Selon la valeur dynamique de mobile je veux activer-désactiver un addEventListener ('click', function())

J'ai illustré la question avec un codepen.

Il fonctionne au chargement mais pas au resize.

Merci d'avance.

[SpaceFrog]

essaye en ajoutant un listener sur le resize ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

window.addEventListener('resize', maFonctionSurResize);

[SpaceFrog]

J'ai vu ta tentative sur le codepen ...

mobile est un objet dont il faut tester le matches ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
window.addEventListener ('resize', function()
{
 
  mobile = window.matchMedia ('(min-width: 768px)');
 console.log( mobile.matches? "Desktop":"Mobile"  ); 
}
);

pour la modif de couleur sur le clcik :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
const test = document.getElementById ('test');
 
mobile = window.matchMedia ('(min-width: 768px)');
 
 
window.addEventListener ('resize', function()
{
mobile = window.matchMedia ('(min-width: 768px)');
 
}
);
 
 
if (mobile.matches)
{
test.addEventListener ('click', function()
{
if (test.style.color=='blue') test.style.color='red';
else test.style.color='blue';
});
}

[boteha]

Bonjour SpaceFrog,

Merci pour ton suivi.

je teste ce soir.

Je te tiens informé.

[NoSmoking]

Bonjour,

attention à ce genre de test :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
if (test.style.color=='blue')
  test.style.color='red';
else
  test.style.color='blue';

il ne fonctionnera que si tu as écris :

Code html :

Sélectionner tout - Visualiser dans une fenêtre à part

<element style="color:blue"></element>

... mais pas si tu passes par une feuille de style ou une déclaration entre balise <style></style>.

[boteha]

Bonjour SpaceFrog,

J'ai fait un rapide copier-coller de ton code dans mon codepen mais le résultat n'est pas au rendez-vous.

Bonjour NoSmoking,

J'ai fait pour le test la changement demandé.
Dans la vraie vie mon problème n'est pas de changer une couleur mais de bloquer une fonction qui en gros au click ouvre un pop-up..

[SpaceFrog]

J"ai testé le premier code et j'ai bien mobile ou desktop en console ...

[boteha]

Bonjour SpaceFrog,

D'accord mais tu vois que sur le codepen le résultat n'est pas celui attendu.

J'attends que la fonction de changement de couleur au clic soit désactivée dès que le resize passe en dessous de 768px, et réactivée dès que l'on passe au-dessus.

Actuellement la largeur est reconnue au chargement de la page mais pas au resize.

[SpaceFrog]

Oui simplement parceque le onclick reste attribué ...

Change de stratégie n'essaye pas d'ajouter ou supprimer le onclick modifie un flag sur le onresize
et teste le en debut de fonction ... fais un return false selon le cas pour ne pas

[SpaceFrog]

Dans l'idée un truc comme ça


le fonction de changement de couleur

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
function changecolor(){
  if (!mobile.matches){ return false,} // test du format et sortie de fonction si pas le bon format
  // ici le changement de couleur 
.../...
}

l'attribution du onclick (quelque soit le format )

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
const test = document.getElementById ('test');
test.addEventListener ('click', changecolor)

initialisation de mobile

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
var mobile = window.matchMedia ('(min-width: 768px)');

rafraichissement de mobile sur le resize

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
window.addEventListener ('resize', function(){
  mobile = window.matchMedia ('(min-width: 768px)');    
});

[boteha]

Merci de ton suivi.

Je fais le test ce soir.

Je vous tiens informés.

[boteha]

Bonjour SpaceFrogm

Copier-coller de ton code dans le codepen, j'ai juste un peu changé l'ordre, fonctionnement attendu.

MERCI beaucoup.

J'attends de transposer dans la vie réelle avant de cocher Résolu.

Par ailleurs, je ne comprendrai jamais Javascript.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
function changecolor(){
  if (!mobile.matches) {return false;  }
  else
  {   
    if (test.style.color=='blue') test.style.color='red';
else test.style.color='blue';
}
 }

Comment les deux variables mobile et test définies hors de la fonction peuvent-elles y entrer ?

J'aurais cru devoir faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
function changecolor (test, mobile)
{
 etc...
}

[SpaceFrog]

les variables sont initialisée en dehors de la fonction et sont donc "globales"

d'après un test que je viens de réaliser, le refresh de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

  mobile = window.matchMedia ('(min-width: 768px)');

sur le resize ne serait pas utile:
https://jsfiddle.net/e358bxLw/

Il semble que la variable se rafraichit toute seule l'affectation se fait par référence ?

[boteha]

Bonjour SpaceFrog,

Encore merci pour ton suivi, je regarde cela ce soir.

[boteha]

Hello SpaceFrog,

Tu as raison, dans le codepen j'ai supprimé le window.addEventListener ('resize', function()) et cela continue à marcher comme avant.

Pour ma gouverne la valeur de mobile est recalculée dans la fonction Changecolor ?

if (!mobile.matches) impose un calcul ?

[NoSmoking]

Comme dit, en début de script tu définis :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

const Mobile = window.matchMedia ('(min-width: 768px)');

puis tu ne t'en occupes plus, lorsque la largeur du viewport sera inférieure à 768px Mobile.matches vaudra false, dans le cas contraire ta variable vaudra true sans que tu rien à faire.

Il te suffit donc de prendre cela en compte dans la fonction déclenchée sur le clic.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
element.addEventListener("click", (e) => {
  if (Mobile.matches) {
    // grand écran
  }
  else {
    // petit écran
  }
});

Tu pourrais même mettre un gestionnaire directement sur cette variable

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
Mobile.addEventListener ("change", () => {
  console.log(Mobile.matches);
});

[boteha]

Bonjour NoSmoking,

Merci de tes précisions.

[boteha]

J'ai installé le code en production.

Fonctionnement parfait, fluide, même sous IE, parfaitement synchronisé avec les media-queries CSS.

Merci encore de votre aide.