[Erledigt] jquery hasClass()

chunky · 02.05.2011, 11:32

Hi. Warum wird bei folgendem Code-Schnipsel denn immer "is closed" ausgegeben?

Code:

<script type="text/javascript">
$(document).ready(function(){

$('.expand').live('click', function() {

		if ( $(this).hasClass('row_closed') )
		{

			alert('is closed');

			/* Open */
			this.removeClass( 'row_closed' );
			this.addClass( 'row_open' );
		}
		else
		{

			alert('is open');

			/* already open - close it */
			this.removeClass( 'row_open' );
			this.addClass( 'row_closed' );
		}
});
});
</script>
<img src="/img/icons/img.png" alt="" class="expand row_closed">

MaiKaY · 02.05.2011, 11:43

probier es doch mit einem else if

Code:

}else if ( $(this).hasClass('row_open') ){

Chriz · 02.05.2011, 11:45

this => $(this)

chunky · 02.05.2011, 12:05

Zitat:

Zitat von Chriz

this => $(this)

Oh Mist. Hab's nicht gesehen.

Danke!

nikosch · 02.05.2011, 15:31

(ungetestet)

PHP-Code:

  
$(document).ready(function(){

$('.expand').live('click', function() {
    $(this)
      .toggleClass ('row_closed')
      .toggleClass ('row_open'   , ! $(this).hasClass ('row_closed'));
});

});

Noch besser wäre es, sich auf eine Klasse zu beschränken. DRY Prinzip beachten!

[edit]
oder kurz:

PHP-Code:

  $('.expand')
  .addClass ('row_closed') // Initialwert, nicht vergessen
  .live('click', function() {
    $(this).toggleClass ('row_closed row_open');
});

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
jQuery Post in iFrame	Atikos	JavaScript, Ajax und mehr	9	13.04.2011 16:45
Einfaches JQuery resize und scoll läuft nicht	smilla	JavaScript, Ajax und mehr	5	08.02.2011 20:44
jQuery, einem DOM-Knoten eine Payload anhängen	Chriz	JavaScript, Ajax und mehr	6	22.12.2010 15:31
jQuery - Text an Cursorposition einfügen	Koala	JavaScript, Ajax und mehr	5	11.11.2010 21:28
Bin noch ein Anfänger mit jquery bitte um hilfe	bomber007	JavaScript, Ajax und mehr	4	19.08.2010 23:34
[Erledigt] jQuery Slider - In eine Richtung sperren	Trainmaster	JavaScript, Ajax und mehr	2	08.08.2010 16:06
[Erledigt] Statistik mit jQuery	DJ_RhoxxZ	JavaScript, Ajax und mehr	6	25.07.2010 17:47
jquery: dialogbox funktioniert nicht	Dreamwatcher	JavaScript, Ajax und mehr	16	22.07.2010 18:22
jQuery findet Element nicht	ByStones	JavaScript, Ajax und mehr	8	11.06.2010 14:03
jQuery Plug scroll to - wie einbetten?	mxxxx	JavaScript, Ajax und mehr	5	24.04.2010 18:12
[Erledigt] [Erledigt] jQuery Endlos Scrolling Umlaute	kn0wledge	JavaScript, Ajax und mehr	0	18.10.2009 19:29
jQuery mit AJAX ohne ActiceX im IE?	krush	JavaScript, Ajax und mehr	2	11.08.2009 08:54
[Erledigt] jQuery Dateien nachladen	Phoscur	JavaScript, Ajax und mehr	7	14.03.2009 13:44
jQuery Plugin - so richtig?	stayInside	JavaScript, Ajax und mehr	5	23.02.2009 12:34
For Schleife mit jQuery	phpbeginner	JavaScript, Ajax und mehr	3	22.11.2008 22:35


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

02.05.2011, 11:43
MaiKaY Erfahrener Benutzer Registriert seit: 17.09.2009 Beiträge: 633 PHP-Kenntnisse: Anfänger	probier es doch mit einem else if Code: }else if ( $(this).hasClass('row_open') ){ __________________ ACHTUNG!!! SIGNATUR!!! PHP-Code: `var_dump($gehirn); exit;` 0 ??? WTF ? freutag.net

02.05.2011, 11:45
Chriz Moderator Registriert seit: 11.05.2008 Beiträge: 6.266	this => $(this) __________________ "Nuschel ich?" - "Was?"

02.05.2011, 15:31
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.987 PHP-Kenntnisse: Fortgeschritten	(ungetestet) PHP-Code: `$(document).ready(function(){ $('.expand').live('click', function() { $(this) .toggleClass ('row_closed') .toggleClass ('row_open' , ! $(this).hasClass ('row_closed')); }); });` Noch besser wäre es, sich auf eine Klasse zu beschränken. DRY Prinzip beachten! [edit] oder kurz: PHP-Code: `$('.expand') .addClass ('row_closed') // Initialwert, nicht vergessen .live('click', function() { $(this).toggleClass ('row_closed row_open'); });` __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --