Status: Confirmed
        Labels: Component-DiffViewer EasyFix Milestone-Release2.0.x

Comment #1 on issue 3437 by chip...@gmail.com: Double-clicking to Add a Comment in the Diff View Discards Any Comments Made

Looked into a bit. Just leaving some notes for later:

It's not an issue with a double-click event, but rather that a mouse down is sees a comment flag in that position and triggers a click event on it, in order to bring back the (presumably closed) comment dialog. This is in textCommentRowSelector.js in _end(), where it calls click(). We can probably just solve this by keeping some state as to which comment flag is currently open, and not calling click() if it's already open.

You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:

You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.

Reply via email to