Bug 803909

Summary: right-click should select editor row
Product: [Retired] Zanata Reporter: Hedda Peters <hpeters>
Component: Component-UIAssignee: Patrick Huang <pahuang>
Status: CLOSED CURRENTRELEASE QA Contact: Ding-Yi Chen <dchen>
Severity: medium Docs Contact:
Priority: high    
Version: 1.5CC: damason, mkim, pahuang, sflaniga, zanata-bugs
Target Milestone: ---   
Target Release: 2.1   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: 2.0.3-SNAPSHOT (20121128-1507) Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-02-26 04:06:25 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Editor/Focus mismatch, with TM match copied into wrong message none

Description Hedda Peters 2012-03-15 23:11:31 UTC
User-Agent:       Mozilla/5.0 (X11; Linux x86_64; rv:9.0.1) Gecko/20100101 Firefox/9.0.1

When the editor is open, then right-clicking into a new message won't open the editor for that message - however, the focus changes to that message and TM matches are displayed for it.


Reproducible: Always

Steps to Reproduce:
1. Left-click into a message A to open it in the editor
2. Now right-click into message B

Actual Results:  

The editor for message A stays open. However, the focus changes to message B. TM matches are displayed for message B.

Expected Results:  
The editor as well as the focus should shift to message B (or neither should react to right-click at all)

The dangerous thing is that this makes it possibile to copy a TM match into the wrong message - I've done it... 

This is not specific to 1.5, it happened in previous versions, too.

Comment 1 Hedda Peters 2012-03-15 23:13:54 UTC
Created attachment 570451 [details]
Editor/Focus mismatch, with TM match copied into wrong message

Comment 2 Sean Flanigan 2012-07-03 05:49:40 UTC
I don't think Zanata should react to right-click at all.  Apart from the issue found by Hedda, it makes using web browsers' inspector features quite painful.

Comment 3 Runa Bhattacharjee 2012-07-30 13:29:27 UTC
Request for technical evaluation, if this is still a problem.

Comment 4 Michelle Kim 2012-11-06 07:22:43 UTC
I can not reproduce this bug in 2.0 
Can anyone verify and see if this bug is now fixed in 2.0 onwards?

Thanks
Michelle

Comment 5 David Mason 2012-11-08 03:58:51 UTC
The problem with the editor is no longer present, however right-clicking in another editor will select the editor without showing the row as selected and without updating TM matches.
It is no longer possible to copy a TM match to the wrong row, but copying from TM applies to the selected row, rather than the row with the editor that has focus.
The focused row should always also be the selected row.

Steps to Reproduce:
1. Left-click in the editor of a message A to select and focus the message
2. Type some characters, do not save
3. Right-click in the editor of a different message B
4. Press Esc or left-click in editor B to close the right-click menu
5. Type some characters, do not save
6. Left-click in another editor C

Actual Results:
3-4. editor B has focus but message A is still selected and does not save. TM results for message A still shown.
5. both editors A and B are highlighted as editing
6. message A saves, message B does not save

Expected Results:
3-4. message A auto-saves and is no longer selected, message B is selected and has TM results; OR message B is not focused.
5. only a single message is ever highlighted
6. (message A already saved) message B saves.

Comment 6 Patrick Huang 2012-11-19 03:59:51 UTC
committed into master:
https://github.com/zanata/zanata/commit/ef120a01913519f113bf16b8257e155a63366989

Comment 7 Ding-Yi Chen 2012-11-20 03:43:09 UTC
Tested with Zanata version 2.1-SNAPSHOT (20121120-1004)
and Zanata version 2.0.3-SNAPSHOT (20121119-0019).

This bug is fixed in 2.1 but not 2.0.3. Please commit the change to Release branch.

Comment 8 Patrick Huang 2012-11-20 03:51:37 UTC
Committed into release already.

Comment 9 Ding-Yi Chen 2012-11-29 04:03:42 UTC
VERIFIED with Zanata version 2.0.3-SNAPSHOT (20121128-1507)
and Zanata version 2.1-SNAPSHOT (20121128-1048)

with Google Chrome 22.0.1229.94
and Firefox 16.0.2