2016-04-27 17 views
-1

hallo jungs ich mache einen toggle-effekt in meiner seite, aber ich habe fehler, wenn ich den knopf für andere teile der seite schließen. Wenn der Knopf in einem Teil von HTML funktioniert, wenn zum Beispiel in anderem Div der Knopf nicht funktioniert. Kannst du Kerl herausfinden, was los ist? Kannst du auch sagen, wenn das meine jQuery sauber ist? oder müssen verbessert werden?jquery toggle klasse funktioniert nicht, wenn in anderen div ist

html:

<a href="#menu-toggle" class="btn btn-sidebar-close" id="close"> 
    <i class="fa fa-times" aria-hidden="true"></i> 
</a> 

<a href="#menu-toggle" id="menu-toggle"data-toggle="tooltip> 
    <i class="fa fa-search fa-2x" aria-hidden="true"></i> 
</a> 

js:

$('#close').click(function(e) { 
    e.preventDefault(); 
    $('#wrapper').toggleClass('toggled'); 
}); 

$('#menu-toggle').click(function(e) { 
    e.preventDefault(); 
    $('#wrapper').toggleClass('toggled'); 
}); 
+0

besser, wenn man eine Geige zu machen, und die Situationen replizieren, weil die Code oben sind nicht genug, um zu zeigen, was das eigentliche Problem ist, – mmativ

Antwort

1

könnte eine Verbesserung sein:

$('#close').click(function(e) { 
     e.preventDefault(); 
     $('#wrapper').toggleClass('toggled'); 
}); 

$('#menu-toggle').click(function(e) { 
     e.preventDefault(); 
     $('#wrapper').toggleClass('toggled'); 
}); 

in einer einzigen Funktion als sie beide die gleiche Funktionalität enthalten:

$('#close, #menu-toggle').click(function(e) { 
     e.preventDefault(); 
     $('#wrapper').toggleClass('toggled'); 
}); 
+0

danke für die Hilfe, es ist immer gut lernen gute Standards für den Code, vielen Dank. – Raduken

+0

@Raduken kein Problem Mann, Generalisierung ist sehr wichtig und es ist gut, es früh zu erfassen. –

+0

Ich verstehe nur einige Leute hier nicht, nur um jemanden Code zu kritisieren, nur weil die Person kein Wissen hat, ich sage nicht auf diesem Thread, aber trotzdem danke nochmal. – Raduken

1

Sie haben vergessen, Zitat nach "Tooltip" hier zu schließen:

<a href="#menu-toggle" id="menu-toggle"data-toggle="tooltip"> 
    <i class="fa fa-search fa-2x" aria-hidden="true"></i> 
</a> 

Andernfalls Ihr Code funktioniert:

See this Fiddle

Verwandte Themen