Opened 8 years ago

Closed 8 years ago

#14386 closed enhancement (fixed)

Refactor dojox.mobile.common

Reported by: ykami Owned by: ykami
Priority: high Milestone: 1.8
Component: DojoX Mobile Version: 1.7.0
Keywords: Cc:
Blocked By: Blocking:

Description

dojox/mobile/common.js has bunch of static functions. Some of them should be in separate files so that the mobile widgets could remove dependency on common.js. For example, icon handling functions and lazy-load support functions can be separated out from common.js.

Change History (8)

comment:1 Changed 8 years ago by ykami

In [27247]:

Refs #14386 !strict Added iconUtils.js and lazyLoadUtils.js, which contain some utility functions in common.js. The functions in common.js remain until all the references are removed.

comment:2 Changed 8 years ago by ykami

In [27432]:

Refs #14386 !strict Inline doc updates. Use dojox.mobile object to access the current theme.

comment:3 Changed 8 years ago by ykami

In [27597]:

Refs #14386 !strict Destroys the existing icon node before creating a new one.

comment:4 Changed 8 years ago by ykami

Status: new → assigned

comment:5 Changed 8 years ago by ykami

In [27607]:

Refs #14386 !strict. Removed setupIcon(), createIcon(), and createDomButton().

comment:6 Changed 8 years ago by ykami

In [27609]:

Refs #14386 !strict. Removed the code for lazy-loadable feature, which was moved to lazyLoadUtils.js.

comment:7 Changed 8 years ago by ykami

In [27611]:

Refs #14386 !strict. Removed redefinition of getEnclosingWidget, which is no longer necessary.

comment:8 Changed 8 years ago by ykami

Resolution: → fixed
Status: assigned → closed
Note: See TracTickets for help on using tickets.