Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > comp.lang.javascript > #34162

Re: click even doesn't fire

Show all headers | View raw

JRough <janis.rough@gmail.com> wrote on 21 Oct 2017 in
comp.lang.javascript: 

> This should work. 

No, it should not. [and the proof of the pudding..]

> There are no errors but the click event doesn't
> happen?   What is supposed to happen is all the cards are face down.  If
> the user clicks one <TD>  then the card turns UP.  Any other card that
> was UP should turn DOWN.  Only one UP at a time.  tnx, 
> 
> <!DOCTYPE html>
> <html>
> 
>      <head>
>           <style type="text/css">
>           td{height:150px;width:150px;background-color:gray;border: 3px
>           solid black; } section{margin:0 auto;width:50%;}
>         </style>
>      </head>
>      <body>
>      <section>
>           <table  >
>                <tbody>
>                     <tr>
>                          <td class="card"> down </td>
>                          <td class="card"> down </td>
>                     </tr>
>                     <tr>
>                          <td class="card"> down </td>
>                          <td class="card">down </td>
>                     </tr>
>                </tbody>
>           </table>
>      </section>
>           <script type="text/javascript">
>                "use strict";
>                var myNodeList = document.getElementsByClassName("card");
>                     myNodeList.addEventListener('click',
>                     setClickHandlers, false); 
>                function setClickHandlers(e , myNodeList) {
>                 for (var i=0; i< myNodeList.length; i++){
>                     if ( e.myNodeList[i].innerHtml=="up" ){
>                               myNodeList[i].innerHtml= "down";
>                       }
>                  }
>                e.target.td.innerHTML = 'up';
>              }
>      
>           </script>
>      </body>
> </html>

Next to Michael's correction of the addEventListener,
innerHTML must be innerHTML,
'this' is far easier than the strange e.target.td, imho,
downing all cards is easier than just the up one.

Try this for 'this':

<script type="text/javascript">
	"use strict";

	let myNodeList = document.getElementsByClassName("card");

	for (let i=0; i< myNodeList.length; i++)
		myNodeList[i].addEventListener('click', doClick, false);

	function doClick() {
		for (let i=0; i< myNodeList.length; i++)
			myNodeList[i].innerHTML = 'down';
		this.innerHTML = 'up';
	};

</script>



-- 
Evertjan.
The Netherlands.
(Please change the x'es to dots in my emailaddress)

Back to comp.lang.javascript | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

click even doesn't fire JRough <janis.rough@gmail.com> - 2017-10-20 18:05 -0700
  Re: click even doesn't fire "Michael Haufe (TNO)" <tno@thenewobjective.com> - 2017-10-21 09:28 -0700
  Re: click even doesn't fire "Evertjan." <exxjxw.hannivoort@inter.nl.net> - 2017-10-22 21:12 +0200
    Re: click even doesn't fire JRough <janis.rough@gmail.com> - 2017-10-22 13:04 -0700
      Re: click even doesn't fire "Evertjan." <exxjxw.hannivoort@inter.nl.net> - 2017-10-22 22:22 +0200
    Re: click even doesn't fire JRough <janis.rough@gmail.com> - 2017-10-22 13:05 -0700
  Re: click even doesn't fire JRough <janis.rough@gmail.com> - 2017-10-22 13:10 -0700
  Re: click even doesn't fire Joao Rodrigues <groups_jr-1@yahoo.com.br> - 2017-10-22 19:23 -0700
    Re: click even doesn't fire Joao Rodrigues <groups_jr-1@yahoo.com.br> - 2017-10-24 13:15 -0700
      Re: click even doesn't fire "Michael Haufe (TNO)" <tno@thenewobjective.com> - 2017-10-24 14:55 -0700
        Re: click even doesn't fire Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2017-10-25 00:16 +0200
        Re: click even doesn't fire Joao Rodrigues <groups_jr-1@yahoo.com.br> - 2017-10-24 17:51 -0700

csiph-web