Summary: | [Apply Patch] Hunk compare editor needs to provide more information | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Stefan Xenos <sxenos> | ||||
Component: | Compare | Assignee: | Platform-Compare-Inbox <platform-compare-inbox> | ||||
Status: | ASSIGNED --- | QA Contact: | |||||
Severity: | enhancement | ||||||
Priority: | P3 | CC: | jean-michel_lemieux, Michael.Valenta, pawel.pogorzelski1, tomasz.zarna | ||||
Version: | 3.3 | ||||||
Target Milestone: | --- | ||||||
Hardware: | Other | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Stefan Xenos
2007-06-04 12:57:43 EDT
I agree with point 1. Point 2 is already provided (i.e. the ancestor contains the before lines). I think that point 3 is worth investigating. Regarding point 2: Thanks for the tip. I'm now showing the ancestor pane by default. However, it is hard to tell by looking at the editor that the upper quadrent contains the deleted lines, and I'm not sure that most users would know what they're supposed to do when they see the three-paned editor. I would suggest showing the deleted lines in the right pane, in strikethrough. In the event of a modify, show the "before" lines in strikethrough, followed by the "after" lines. The after lines would have the blue background and context lines would be undecorated (as they are now). I suspect that most users will not intuitively understand what the "common ancestor" means in the case of a hunk, but *will* understand that strikethrough text means a deletion. Created attachment 84222 [details]
Mockup that shows how to move the insertion point without additional toolbar buttons
This shows an alternative way to show the buttons for moving the insertion point (the LCS buttons are not shown).
Rather than putting them on the toolbar, they can be attached to the insertion point marker itself along with a grippy that lets you drag it up and down.
Mass update - removing 3.4 target. This was one of the bugs marked for investigation (and potential fixing) in 3.4 but we ran out of time. Please ping on the bug if fixing it would be really important for 3.4, and does not require API changes or feature work. This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. |