297: Show selected diff chunk in a better way

giri*****@gmai***** (Google Code) (Is this you? Claim this profile.)
chipx86
 chipx86
Dec. 27, 2008 
What's the URL of the page containing the problem?
Any diff

What steps will reproduce the problem?
 when you press 'n' or 'p' to look at the next or previous diff, I dont 
visually see which text is currently selected. There is a tiny arrow on 
the left screen which tells me that this text is selected, but there 
should be a better way. 

What is the expected output? What do you see instead?
You could display a red box or something around the selected chunk of 
text. This way, the user can clearly know that this is the piece of code 
I'm looking at. 

What operating system are you using? What browser?
Any.

Please provide any additional information below.
None.
chipx86
#1 chipx86
  • -Type-Defect
    -Priority-Medium
    +Type-Enhancement
    +Priority-Low
    +Component-DiffViewer
chipx86
#2 chipx86 
I'd like to find a better visual effect, but borders sadly won't be it. Different
browsers are very picky about how they display borders on collapsed tables, and we'd
need to put a border around the whole tbody (something IE doesn't support).
#3 giri*****@gmai***** (Google Code) (Is this you? Claim this profile.) 
Is it possible to do something like p5 diffhtml does ?
david
#4 david
  • +Show selected diff chunk in a better way
chipx86
#5 chipx86 
Got this working in my UI rewrite.
  • +Started
  • +chipx86
chipx86
#6 chipx86
  • +Milestone-Release1.0
chipx86
#7 chipx86 
Much improved in r1644.
  • -Started
    +Fixed