The 'click' part of your code seems to work fine. I couldn't test the
rest, but prevAll (you've got a typo there) and nextAll + andSelf are
always returning the correct elements. Use Firebug, set console.log( $
(this).preAll('a').andSelf().addClass('star_hover') ) inside the
handler and you'll see that the elements are there. My guess is
something is going wrong when you trigger the mouseout.
- ricardo
On Dec 13, 12:32 am, Reepsy <mre...@gmail.com> wrote:
> Ricardo,
>
> Sorry, but it is a development stage I am not permitted to share. But
> the markup is this (basically):
>
> HTML (this was a select menu that gets converted to a hidden field and
> 5 <a> tags in an earlier step)
>
> <input name="rating" id="the-rating" value="" type="hidden">
> <a href="" id="star1" class="star" title="1 Star">1</a>
> <a href="" id="star2" class="star" title="2 Stars">2</a>
> <a href="" id="star3" class="star" title="3 Stars">3</a>
> <a href="" id="star4" class="star" title="4 Stars">4</a>
> <a href="" id="star5" class="star" title="5 Stars">5</a>
>
> JS:
>
> var r = $('#the-rating'); /* THE HIDDEN FIELD STORES RATING */
> $('a.star').click(function(){
> $(r).val($(this).text()); /* RECORD RATING */
> $(this).nextAll('a').andSelf().removeClass
> ('star_selected').triggerHandler('mouseout');
> return false;})
>
> .hover(
> function() {
> $(this).preAll('a').andSelf().addClass('star_hover')
> $(this).nextAll('a').removeClass('star_selected');
> },
> function() {
> $(this).prevAll('a.star').andSelf().removeClass('star_hover');
> $('#star' + $(r).val()).prevAll('a.star').andSelf().addClass
> ('star_selected');
> }
> );
>
> This worked exactly as expected on stars 2-4. But on #5 the click
> failed to add the class and fire the mouseout. I presume it is because
> there is no nextAll. Likewise, on star #1 the hover failed remove the
> class. Again, I presume because there is no prevAll.
>
> Michael
>
> On Dec 12, 11:50 am, ricardobeat <ricardob...@gmail.com> wrote:
>
> > Do you have a test page we can look at?
>
> > nextAllreturns an empty object if there is no 'next', but it doesn't
> > interrupt thechain, there may be something else going on. I couldn't
> > reproduce your situation here,nextAll().andSelf() returns me the
> > original element.
>
> > - ricardo
>
> > On Dec 12, 10:39 am, Reepsy <mre...@gmail.com> wrote:
>
> > > This might sound naive, but I expected this to work:
>
> > > $(this).nextAll('a').andSelf().removeClass
> > > ('star_selected').triggerHandler('mouseout');
>
> > > It's from a star rating I wrote, where I have 5 <a> tags in a row. If
> > > you click on one it removes a class from it and all that follow it,
> > > and then fires the mouseout event. This works perfectly for stars 1-4,
> > > but fails on #5, because there is no next. But I did not expect it to
> > > ignore the rest of thechain. Everything after .nextAllis ignored. If
> > > I break this into two lines, it works fine:
>
> > > $(this).nextAll('a').removeClass('star_selected');
> > > $(this).removeClass('star_selected').triggerHandler('mouseout');
>
> > > But I am repeating myself with the removeClass. Can anyone see a way
> > > to combine these back into one statement? The mouseout has to go last.
>
> > > Michael
