555: multiple-line selection in a diff doesn't work when selecting upwards

eyal****@gmai***** (Google Code) (Is this you? Claim this profile.)
chipx86
chipx86
Dec. 27, 2008
What's the URL of the page containing the problem?
http://reviews.review-board.org/r/464/diff/

What steps will reproduce the problem?
1. Go to the "View diff" page of a review request.
2. Click and drag from line 372 to line 377, and click "cancel" in the
pop-up window.
3. Click and drag from line 377 to line 372, and click "cancel" in the
pop-up window.

What is the expected output? What do you see instead?
In 2, the comment should be for 6 lines (and it is).
In 3, the comment should be for 6 lines (it's only for one line - 377).

What operating system are you using? What browser?
Windows XP, Firefox 3
david
#1 david
  • +Confirmed
chipx86
#2 chipx86
This is fixed in my UI rewrite.
  • -Confirmed
    +Started
  • +chipx86
chipx86
#3 chipx86
  • +Milestone-Release1.0
    +Usability
    +Component-DiffViewer
chipx86
#4 chipx86
Fixed in r1644.
  • -Started
    +Fixed