Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#13871 closed defect (fixed)

Exception in SpinWheelSlot when hidden/invisible

Reported by: Javier Pedemonte Owned by: ykami
Priority: high Milestone: 1.7
Component: DojoX Mobile Version: 1.7.0b1
Keywords: 1.7-mobile Cc: Adam Peller
Blocked By: Blocking:

Description

If SpinWheel? is not visible (for example, if it is placed in a secondary View that is not currently visible), results in an error.

If the SpinWheel? is visible, then in SpinWheelSlot?.getCenterPanel(), the properties offsetTop/offsetHeight have non-zero values, and the function returns a panel instance. If SpinWheel? is hidden/invisible (for example, in a secondary View that isn't currently selected), then the offset* properties are always zero and getCenterPanel() returns null, causing exception.

Attachments (1)

spinWheel_2_917.html (885 bytes) - added by Javier Pedemonte 8 years ago.
testcase

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by Javier Pedemonte

Attachment: spinWheel_2_917.html added

testcase

comment:1 Changed 8 years ago by Adam Peller

Cc: Adam Peller added

comment:2 Changed 8 years ago by ykami

Status: newassigned

comment:3 Changed 8 years ago by ykami

Resolution: fixed
Status: assignedclosed

In [26572]:

Fixes #13871 !strict Removed setTimeout. If the SpinWheel? widget is initialized when it is on a hidden pane, the initialization fails because correct dom element dimension is not available. If that is the case, keep the initial value in a widget property, and apply it when resize() is called next time.

comment:4 Changed 8 years ago by ykami

Keywords: 1.7-mobile added
Milestone: tbd1.7

comment:5 Changed 8 years ago by ykami

In [26573]:

Fixes #13871 !strict scrollable#_speed had to be initialized in init().

comment:6 Changed 8 years ago by ykami

In [26584]:

Fixes #13871 !strict Change the spin duration to zero only at start up time to reduce flickers especially on android.

Note: See TracTickets for help on using tickets.