Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 369426 - In compare editor 'Copy current change from right to left' is too close to 'Next difference'
Summary: In compare editor 'Copy current change from right to left' is too close to 'N...
Status: RESOLVED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Compare (show other bugs)
Version: 4.2   Edit
Hardware: PC All
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Platform-Compare-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-23 12:27 EST by Iouri Goussev CLA
Modified: 2012-02-17 04:50 EST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Iouri Goussev CLA 2012-01-23 12:27:25 EST 
Build Identifier: M20110909-1335

'Copy current change from right to left' button is dangerously close to 'Next difference'.

When user use 'Next difference' button to navigate change set, he may move cursor slightly and accidentally undo his changes. 

More space is needed between those two buttons. 

Reproducible: Always

Steps to Reproduce:
1. Click next difference
2. Move your cursor couple of pixels to the left.
3. Click, and your changes are lost.
Comment 1 Iouri Goussev CLA 2012-01-23 12:29:28 EST 
See http://help.eclipse.org/helios/topic/org.eclipse.platform.doc.user/images/Image80_compare_editor.png

4'th button in change editor - Move current change fron right to left
5'th button in change editor - Next difference.
Comment 2 Malgorzata Janczarska CLA 2012-01-25 04:18:42 EST 
I've been using compare editor for quite a while and I never had this problem. Buttons are separated with an extra separator and even if you click 'Copy current change from right to left' by accident your changes are not saved and you get a notice about changes closing the compare editor. A simple Ctrl+Z reverts the change. A minor issue for me.
Comment 3 Tomasz Zarna CLA 2012-02-17 04:50:59 EST 
See comment 2.