Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 349227 - [Client]Compare editor: Remove border style for diff blocks
Summary: [Client]Compare editor: Remove border style for diff blocks
Status: CLOSED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Client (show other bugs)
Version: 0.2   Edit
Hardware: PC Windows 7
: P2 normal (vote)
Target Milestone: 0.2   Edit
Assignee: libing wang CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-13 16:00 EDT by libing wang CLA
Modified: 2011-09-01 11:41 EDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description libing wang CLA 2011-06-13 16:00:01 EDT
Currently we are using BorderStyle to render the diff blocks in side by side compare editor.This affects the line height in the Orion standard editor.
We found some issues :
1.Clicking on inside a diff block does not put the text input cursor properly
2.Range selecting text is flashing inside a diff block.
3.Ruler does not align well if there are a few numbers of diff blocks
The reason of all above is because the line height on a bordered line  is slightly bigger than a normal line.
A temporary solution in 0.2 is to remove the border and use slightly darker background color for the diff blocks.
For the current diff , we will use darker background color than the normal diff block.
For the conflicting block , we will use light red color.
Comment 1 Simon Kaegi CLA 2011-06-13 17:03:59 EDT
+1 for doing something about this in RC2
Comment 2 libing wang CLA 2011-06-14 11:30:05 EDT
fixed with 789bbd90a7e22a95d3bb6e13c98b9ce1fee7d654.
Code reviewed by Felipe and Silenio.