conditional javascript

rubyonrails-talk
1

onclick="if(this.style.background='#F9CF00')this.style.background='#ffffff' else this.style.background='#F9CF00';

How do I implement a conditional if else statement in the onclick?

The <tr> is white by default, when a user clicks it, and it is white, I want it to change to orange. And vice versa,

Thanks

2

onclick ="if(this.style.background='#F9CF00')this.style.background='#ffffff' else this.style.background='#F9CF00';

How do I implement a conditional if else statement in the onclick?

The <tr> is white by default, when a user clicks it, and it is white, I want it to change to orange. And vice versa,

this.style.background = (this.style.background == '#f9cf00' ? '#fffff' : '#f9cf00')

That last part works like this:

(some expression yielding true of false ? return this if true : return this if false)

3

I think there was a simple typo -

onclick="this.style.background = (this.style.background == '#F9CF00')? '#FF0000' : '#F9CF00';"

You may also want to look into attaching this behind the scenes removing your behavior from your display completely, best practice - but that's just the geek in me.

4

Jeff.Corcoran wrote:

I think there was a simple typo -

onclick="this.style.background = (this.style.background == '#F9CF00')? '#FF0000' : '#F9CF00';"

Not working D=

You may also want to look into attaching this behind the scenes removing your behavior from your display completely, best practice - but that's just the geek in me.

On Jul 14, 10:17�pm, Justin To <rails-mailing-l...@andreas-s.net>

Yah, once I get it to work I'll stick it somewhere else =)

5

Try this: onclick="style.backgroundColor=(style.backgroundColor == 'rgb(249, 207, 0)' ? '#FF0000' : '#F9CF00')"

Even when you assign a string like '#ffffff' internally is converted to 'rgb(255, 255, 255)'

Works for me in Firefox 3.0 and Safari 3.1.2

Regards.

6

If that is still not working - can you drop an example of the code and an error? The above example I gave you worked for me.

7

Jeff.Corcoran wrote:

If that is still not working - can you drop an example of the code and an error? The above example I gave you worked for me.

Jeff, it works great in Firefox 3.0, but not in IE 7, any suggestions on that?

Thanks!!

8

Also, I have

onclick = Element.show('<%= user.email -%>') => show <tr>

How do I apply the same logic as the bg color to say, "<tr> is showing, so onlcick hide it, and vice versa":

Element.hide('<%= user.email -%>

Thanks!!

9

If you can give me an example of the HTML that is coming out I may be able to help you better.

10

Jeff.Corcoran wrote:

If you can give me an example of the HTML that is coming out I may be able to help you better.

On Jul 15, 10:53�am, Justin To <rails-mailing-l...@andreas-s.net>

How do I get that?

11

Justin To wrote:

How do I get that?

Just look at the source and then cut and paste

12

But there is no source for that since it's using ajax w/o refreshing the page. It shows the table, but the source does not reflect the contents

13

Use firefox. Select the generated html and right click. There is an option for “Show Selection Source”

Or, you can get firebug and either inspect the element or see what the ajax request returned

14

Element.show('<%= user.email -%>') means that user.email is the id of the tr you want to hide, right? Like this:

  <tr id='<%= user.email %>'> .... </tr>

(which fails btw. if two users on that page have the same email)

Use toggle:   <span onclick='Element.toggle(<%= user.email %>)'>Show/hide it!</

Of course you can't put that into the tr itself because there's no clicking on something invisible.

15

Oops, forgot the quotes

<span onclick='Element.toggle("<%= user.email %>")'>Show/hide it!</