Opened 12 years ago

Closed 12 years ago

#120 closed enhancement (fixed)

add loading spinner as extension

Reported by: sheppard Owned by: theduckylittle
Priority: major Milestone: 2.6.1
Component: UserExtensions Version: 2.6
Keywords: Cc:

Description

Attachments (2)

dojox.patch (796 bytes ) - added by sheppard 12 years ago.
Add dojox.widget.Standby to build
Loading.js (2.2 KB ) - added by sheppard 12 years ago.
Loading spinner widget

Download all attachments as: .zip

Change History (10)

comment:1 by sheppard, 12 years ago

I (re-)implemented this by utilizing the dojox Standby widget.

The changes I made to trunk were as follows:

  1. Add dojox.widget.Standby to dojo build (see attached patch)
  1. Copy libs/dojo-1.6.1/dojox/widget/Standby/images/loading.gif to build/dojox/widget/Standby/images/loading.gif (adding directories as needed). This assumes an updated build directory structure, see #121.
  1. Create/update the LoadingSpinner extension (attached).

If it is undesirable to add dojox.widget.Standby to the build I can come up with a different process. The advantage of this approach is that anyone can use the extension just by adding the <script> without any additional configuration.

by sheppard, 12 years ago

Attachment: dojox.patch added

Add dojox.widget.Standby to build

by sheppard, 12 years ago

Attachment: Loading.js added

Loading spinner widget

comment:2 by sheppard, 12 years ago

Like #124, this extension won't work in IE unless standards mode is enabled (see #125).

comment:3 by theduckylittle, 12 years ago

Owner: changed from sheppard to theduckylittle
Status: newassigned

comment:4 by theduckylittle, 12 years ago

Status: assignedtesting

comment:5 by sheppard, 12 years ago

Status: testingneeds_work

Looks like the code is working, but still needs build/dojox/widget/Standby/images/loading.gif to be checked in.

(Wondering if #128 might provide a better long-term solution).

comment:6 by theduckylittle, 12 years ago

Status: needs_workassigned

#128 is the right answer, I'm going to work on *an* answer for a minute here...

comment:7 by theduckylittle, 12 years ago

Status: assignedtesting

r892

Localized the loading image. I think that is a better demonstration for those looking to customize anyway.

comment:8 by bfischer, 12 years ago

Resolution: fixed
Status: testingclosed
Note: See TracTickets for help on using tickets.