#13328 closed enhancement (fixed)
[patch] AMD refactoring of dojox.form.BusyButton
Reported by: | Kitson Kelly | Owned by: | Rawld Gill |
---|---|---|---|
Priority: | high | Milestone: | 1.7 |
Component: | DojoX Form | Version: | 1.7.0b1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description (last modified by )
This is an AMD refactoring of the dojox.form.BusyButton. Tested the patch against test_BusyButton.html. CCLA on file.
Attachments (1)
Change History (4)
Changed 10 years ago by
Attachment: | BusyButton.js.patch added |
---|
comment:1 Changed 10 years ago by
Description: | modified (diff) |
---|---|
Owner: | changed from dante to Rawld Gill |
Rawld, can you take a look? I tested the patch against test_BusyButton.html too, but it didn't work for me. It gives a loader error:
Object { src="dojoLoader", id="multipleDefine"} Object { pid="dojo", mid="dojo/parser", more...}
There's one obvious mistake in the patch, that it's missing all of the dojo core dependencies except for dojo/_base/kernel, but even after fixing (by changing dojo/_base/kernel to dojo/main) I still get the same error (this is testing over HTTP on FF5/mac).
comment:2 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Tried test_BusyButton.html as of [25945] and it works for me.
comment:3 Changed 10 years ago by
Milestone: | tbd → 1.7 |
---|
Note: See
TracTickets for help on using
tickets.
AMD Refactoring for BusyButton?