Opened 9 years ago

Closed 9 years ago

#3596 closed bug (fixed)

Replace global jQuery with dollar sign

Reported by: vmx Owned by: Scott González
Priority: blocker Milestone: 1.7
Component: ui.core Version: 1.6rc2
Keywords: Cc:
Blocked by: Blocking:

Description

Some files use "jQuery" instead of the dollar sign. This is a bad idea, as you loose the possibility to pass a custom jQuery object into the closures that surround the plugins.

It passes the test suite.

Attachments (1)

global-jquery-to-dollar-sign.patch (4.7 KB) - added by vmx 9 years ago.
Change "jQuery" to $

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by vmx

Change "jQuery" to $

comment:1 Changed 9 years ago by vmx

I forgot to mention that I haven't fixed the datepicker as it contains so many references to "jQuery" in onclick attributes. Fixing that would need a major refactoring.

comment:2 Changed 9 years ago by Scott González

Owner: changed from paul to scott.gonzalez
Priority: majorblocker
Status: newassigned

Already fixed in progressbar.

comment:3 Changed 9 years ago by Scott González

Resolution: fixed
Status: assignedclosed
Type: enhancementbug

Fixed in r951.

Note: See TracTickets for help on using tickets.