Bugzilla (bugzilla.redhat.com) will be under maintenance for infrastructure upgrades and will not be available on July 31st between 12:30 AM - 05:30 AM UTC. We appreciate your understanding and patience. You can follow status.redhat.com for details.
Bug 728854 - Save notification closes the editor unnecessarily
Summary: Save notification closes the editor unnecessarily
Alias: None
Product: Zanata
Classification: Retired
Component: Component-UI
Version: 1.3
Hardware: Unspecified
OS: Linux
Target Milestone: ---
: 1.4.2
Assignee: Alex Eng
QA Contact: Ding-Yi Chen
Depends On:
Blocks: 746870
TreeView+ depends on / blocked
Reported: 2011-08-08 07:08 UTC by Sean Flanigan
Modified: 2011-10-28 07:02 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2011-10-28 07:02:35 UTC

Attachments (Terms of Use)

Description Sean Flanigan 2011-08-08 07:08:40 UTC
User-Agent:       Mozilla/5.0 (X11; Linux i686 (x86_64)) AppleWebKit/534.30 (KHTML, like Gecko) Chrome/12.0.742.100 Safari/534.30

The user's own save notifications can cause the editor to close (cancel edit).  

This problem can happen when clicking up and down in the table, depending on processing speed, but the most consistent trigger is based on key shortcuts.

Reproducible: Always

Steps to Reproduce:
1. Enter some text in a target cell (ie change the content) to force a Save upon exit.
2. Press Alt+J+K (ie Alt+J and Alt+K virtually simultaneusly).
Actual Results:  
Cell content saved, but the save notification for that cell causes the editor to close.  If the notification is for the text content as is currently in the editor, there is no need to cancel editing.

Expected Results:  
Cell content saved; editor is open on the original target cell.

Comment 1 Sean Flanigan 2011-08-08 07:17:12 UTC
Note that this problem is currently exacerbated by bug 728858's extra save notifications.

Comment 2 David Mason 2011-08-08 07:19:49 UTC
This bug is probably related to bug 728851

Comment 3 Sean Flanigan 2011-08-10 05:37:13 UTC
If we fix bug 728858, I suppose this one won't need a high priority.

Comment 4 Sean Flanigan 2011-09-07 04:33:20 UTC
Assigning to Scrum product owner for prioritisation.

Comment 5 Runa Bhattacharjee 2011-09-08 13:15:09 UTC
This is a serious loss of functionality of the editor. Moving this back to high.

Comment 7 Ding-Yi Chen 2011-10-20 04:25:03 UTC
VERIFIED with Zanata version 1.4.2-SNAPSHOT (20111020-1402)

Note You need to log in before you can comment on or make changes to this bug.