Metadata
- Source
- FLUID-1607
- Type
- Bug
- Priority
- Major
- Status
- Closed
- Resolution
- Fixed
- Assignee
- Colin Clark
- Reporter
- Justin Obara
- Created
2008-09-24T13:58:11.000-0400 - Updated
2008-12-12T16:30:54.000-0500 - Versions
-
- 0.5beta1
- 0.5
- 0.6beta1
- Fixed Versions
- N/A
- Component
-
- Inline Edit
Description
Quick clicking the undo/redo link breaks its functionality. It will still change between undo and redo, but the value of the inline edit field will remain the same.
Steps to reproduce:
1) Open the bSpace example from the daily build site:
http://build.fluidproject.org/fluid/sample-code/inline-edit/section-info/section-info.html
2) Edit one of the inline edit fields and press enter to save the value
3) Quickly and repeatedly click the undo/redo link
Notice that soon you get a context sensitive menu and the functionality of the undo/redo button is broken. This context sensitive menu is the same that appears if you double click plain text on the page.
Environments
Opera 9.5 (Win XP)
Comments
-
Justin Obara commented
2008-09-24T13:58:56.000-0400 Bug Parade: 0.5 release
-
Jacob Farber commented
2008-09-24T14:00:58.000-0400 Perhaps it would help if event propogation is stopped immediately after our event, preventing the text selection from happening?
-
Justin Obara commented
2008-12-03T13:52:50.000-0500 Bug Parade 0.6 release
-
Antranig Basman commented
2008-12-12T14:03:44.000-0500 Opera delivers two "click" events "back-to-back" - that is, before reflecting any visibilty changes for button states in the UI. As a general recommendation, the "model" for any component should be resistant to corruption from any combination of events incoming from the outside, even if it believes it issues control requests to destroy or hide the relevant triggering controls.
Fixed at revision 6209
-
Justin Obara commented
2008-12-12T16:30:54.000-0500 The commit was code reviewed by Colin Clark (only needed to be linted)
After linting,
I Verified the fix using
Opera 9.5 (Win XP)