rave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Cooper, Sean D." <secoo...@mitre.org>
Subject RE: Review Request: Rave-359 (WidgetRating UI: Initial load of page allows a double click for rating)
Date Mon, 05 Dec 2011 18:30:06 GMT
Try abandoning the pre-wiring of the events.  Instead of marking the radio buttons as checked/unchecked
see if using the jquery event model to check/uncheck the buttons resolves the problem.


From: Ankur Goyal [mailto:asgoyal@umail.iu.edu]
Sent: Thursday, December 01, 2011 10:32 AM
To: rave
Subject: Fwd: Review Request: Rave-359 (WidgetRating UI: Initial load of page allows a double
click for rating)

Sending to the entire list.
---------- Forwarded message ----------
From: Ankur Goyal <asgoyal@umail.iu.edu<mailto:asgoyal@umail.iu.edu>>
Date: Wed, Nov 30, 2011 at 4:22 PM
Subject: Re: Review Request: Rave-359 (WidgetRating UI: Initial load of page allows a double
click for rating)
To: "Cooper, Sean D." <secooper@mitre.org<mailto:secooper@mitre.org>>
Cc: Marlon Pierce <mpierce@cs.indiana.edu<mailto:mpierce@cs.indiana.edu>>

Hi Sean,

I did some modifications which solved the problem you mentioned in your first email but there
is a small problem and I am not able to understand it.

When both like and dislike ratings are 0 and then if you click anyone of them (say like button)
it updates properly but when you click on the other (dislike button), it shows no effect no
matter how many times you click on it. But if you click once more on the first one which got
updated (i.e. like button), it wont update and remain as 1 (as needed) and then try to click
on the other (dislike button), the other one then updates to 1 with the first one updates
to 0.

The problem that i noticed was that change function in rave_store.js file is not been called
for the first hit for the other button (dislike). I am unable to find a way out of it.

like: 0 dislike: 0
first hit on like button
like: 1 dislike: 0
second hit on dislike button
like: 1 dislike: 0 (no change no matter how much you click on dislike button)
click on like button again
no change
now click on dislike button
like: 0 dislike: 1 (as expected)
here after it works fine.

I have attached my patch file with this. It will have some alert messages just to know the

Waiting for your reply,

Ankur Goyal

On Tue, Nov 29, 2011 at 4:25 PM, Cooper, Sean D. <secooper@mitre.org<mailto:secooper@mitre.org>>
You might try creating a third radio button for the 'unset' state and have it be a hidden
element on the page.  That way you can correctly set the state of the radio button to 'positive',
'negative' , and 'unset'


>-----Original Message-----
>From: Ankur Goyal [mailto:asgoyal@umail.iu.edu<mailto:asgoyal@umail.iu.edu>]
>Sent: Tuesday, November 29, 2011 4:18 PM
>To: Sean Cooper
>Cc: Marlon Pierce; Ankur Goyal; rave
>Subject: Re: Review Request: Rave-359 (WidgetRating UI: Initial load of page
>allows a double click for rating)
>Hi Sean,
>I was looking into the problem you mentioned.
>There is some property called as "aria-pressed" which is always set as
>"true" for like button and "false" for dislike button after every load of
>widget store page.
>I am unable to find where this effect is brought from so thought to ask you
>about this. :)
>Waiting for your reply,
>Ankur Goyal
>On Mon, Nov 28, 2011 at 2:14 PM, Sean Cooper <secooper@gmail.com<mailto:secooper@gmail.com>>
>>    This is an automatically generated e-mail. To reply, visit:
>> https://reviews.apache.org/r/2917/
>> This patch is forcing the button state to highlight the 'negative' option when
>using chrome or firefox regardless of whether or not the user has already
>selected one of the options (e.g. If the user has selected the 'positive' option
>the 'negative' option is showing up as being selected)
>> - Sean
>> On November 22nd, 2011, 10:05 p.m., Ankur Goyal wrote:
>>   Review request for rave and Marlon Pierce.
>> By Ankur Goyal.
>> *Updated 2011-11-22 22:05:24*
>> Description
>> WidgetRating UI: Initial load of page allows a double click for rating
>> Files modified:
>> rave_store.js
>> store.jsp
>>   Diffs
>>    - trunk/rave-portal-resources/src/main/webapp/WEB-
>>    (1205187)
>>    - trunk/rave-portal-resources/src/main/webapp/script/rave_store.js
>>    (1205187)
>> View Diff <https://reviews.apache.org/r/2917/diff/>

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message