FLUID-1607: Quick clicking the undo/redo link breaks its functionality in Opera

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
  1. 0.5beta1
  2. 0.5
  3. 0.6beta1
Fixed Versions
N/A
Component
  1. 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)