Quick clicking the undo/redo link breaks its functionality in Opera
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.
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.
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)
Antranig Basman December 12, 2008 at 7:03 PM
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 December 3, 2008 at 6:52 PM
Bug Parade 0.6 release
Jacob Farber September 24, 2008 at 6:00 PM
Perhaps it would help if event propogation is stopped immediately after our event, preventing the text selection from happening?
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.