jQuery / Coffeescript - if conditional

Question

Using jQuery and coffeescript, I want to update a html element when user presses enter. My code looks like this:

$ ->
  $('p span').live 'keypress', (e) ->      
    if e.keyCode == 13        
      $('div.sidebar-nav ul li a.active').html $(this).attr 'value'

Coffeescript compiles without any errors, but the code is not working. For some reason, if I add change the code to following it works:

$ ->
  $('p span').live 'keypress', (e) ->      
    if e.keyCode == 13
      alert 'some string'
      $('.sidebar-nav a.active').html $(this).attr 'value'

I spend last few hours trying to solve this and still nothing. Any help would be deeply appreciated.

@gdoron: Like in JavaScript, $ is just a function. -> introduces an anonymous function. CoffeeScript also adds calling functions without parentheses. Put the pieces together, and you get $(function() { /* ... */ }); — icktoofay
– icktoofay, Commented Jun 2, 2012 at 22:04
@gdoron: It doesn't remove the ability to use parentheses; it just makes them optional. If you're asking about whether 'value' is another argument, it's not, because there's no comma. — icktoofay
– icktoofay, Commented Jun 2, 2012 at 22:08
@icktoofay. Thanks for this one minute tutorial, Now I'm sure I don't want to use it. YACK! — gdoron
– gdoron, Commented Jun 2, 2012 at 22:09
@gdoron Do you have some kind of fixation on bashing CoffeeScript? :S — epidemian
– epidemian, Commented Jun 3, 2012 at 0:05

BenMorel · Accepted Answer · 2013-10-31 20:39:17Z

3

$('.sidebar-nav a.active')

is different than...

$('div.sidebar-nav ul li a.active')

in the first case, a.active is a descendant of .sidebar-nav, and the structure under .sidebar-nav could be anything (list, article, spans, marquees, doesn't matter, as long as its in there somewhere). In the second, it is a great-grandchild, within a list. We'd need to see your HTML to be sure, but it looks like bad selectors...

edited Oct 31, 2013 at 20:39

BenMorel

37k52 gold badges208 silver badges339 bronze badges

answered Jun 2, 2012 at 22:05

Kyle Macey

8,1543 gold badges40 silver badges78 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

jQuery / Coffeescript - if conditional

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related