Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#15436 closed defect (fixed)

remove dojo stub code from scrollable.js

Reported by: bill Owned by: Eric Wang
Priority: undecided Milestone: 1.8
Component: DojoX App Version: 1.7.2
Keywords: Cc: cjolif
Blocked By: Blocking:

Description (last modified by bill)

The scrollable.js file in dojox/app which is 95% copied from dojox/mobile should be updated to remove the dojo stub code (same as the original file was updated).

I also suggest the modifications in #15438.

(Of course, better than either of these would be to get rid of the duplicated file. Maybe we could move the dojox/mobile file to dijit if that would help.

Change History (4)

comment:1 Changed 7 years ago by bill

Milestone: tbd1.8

Marking for 1.8 since this code will confuse the AMD doc parser.

comment:2 Changed 7 years ago by bill

Cc: cjolif added
Description: modified (diff)

comment:3 Changed 7 years ago by cjolif

Resolution: fixed
Status: newclosed

The stub code has been removed for 1.8. Removing the duplication is part of our plans for next release. This not just as easy as sharing the file as some amount of code has been changed and so there is some refactoring needed.

comment:4 Changed 7 years ago by bill

OK, then I filed #15440 for the other issue.

Note: See TracTickets for help on using tickets.