Discussion :

[sebgac]

Bonjour, je suis en train de refactoriser mon script, et j'essaie d'éviter certaines redondances.

J'ai remarqué que

Code :

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

1
2
3
4
5
 
$('#voyages').on('click', function () {
            if (continentOuvert == false) {
                $('#voyages i.fas.fa-arrow-up').show(500);
                $('#voyages i.fas.fa-arrow-down').hide();

Revenait souvent pour pas mal de mes catégories. L'idée c'est de mettre

Code :

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

1
2
$('#voyages i.fas.fa-arrow-up').show(500);
$('#voyages i.fas.fa-arrow-down').hide();

dans une fonction, et d'utiliser le sélecteur écoutés pour déterminer dans quelle id seront affichés/masqués les icones " i.fas.fa " (dans ce cas dans l'id #voyage, mais après il y a l'id #continent etc ...)

J'essaie de le faire comme ceci :

Code :

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

1
2
3
4
5
 
            function arrowUp() {
            $('this i.fas.fa-arrow-up').show(500);
            $('this i.fas.fa-arrow-down').hide();
        }

ou bien :

Code :

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

1
2
3
4
5
 
            function arrowUp() {
            $(this + ' i.fas.fa-arrow-up').show(500);
            $(this + ' i.fas.fa-arrow-down').hide();
        }

Où le this doit prendre la valeur du sélecteur dans $('#voyages').on('click', function ()

Cela ne fonctionne pas, je pense que je m'y prends mal, kkun pourrait me donner une piste aider à me débloquer s'il vous plait ?

Merci d'avance

[SpaceFrog]

si les elements sont contenu dans this ...

Code :

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

1
2
 $(this).find(' i.fas.fa-arrow-up').show(500);
 $(this).find(' i.fas.fa-arrow-down').hide();

[Invité]

Bonjour,

le plus malin ici serait de passer l'id en paramètre de la fonction :

Code :

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

1
2
3
4
5
function arrowUp( id )
{
    $(id+' i.fas.fa-arrow-up').show(500);
    $(id+' i.fas.fa-arrow-down').hide();
}

et pour l'appel :

Code :

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

arrowUp( '#voyages' );

ou plus généralement :

Code :

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

arrowUp( '#'+$(this).id ); // (si tu es sûr que c'est toujours un id, et pas une classe)

[sebgac]

Merci pour vos réponses !

j'ai finalement utilisé ce code

Code :

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

1
2
3
4
function arrowUp(id) {
            $(id + ' i.fas.fa-arrow-up').show(500);
            $(id + ' i.fas.fa-arrow-down').hide();
        }

puis appel avec

Code :

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

arrowUp('#'+$(this).attr('id'));

je sais pas pourquoi ça marchait pas avec find()

[SpaceFrog]

ça fontionnerait sans doute avec

Code :

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

1
2
$(this).find('.fas.fa-arrow-up').show(500);
 $(this).find('.fas.fa-arrow-down').hide();

[Invité]

@SpaceFrog

Oui, s'il le laisse dans le cours du script.

Mais NON, puisqu'il le met dans une fonction (arrowUp), où $(this) ne sera pas défini.