Opened 14 years ago

Closed 14 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 14 years ago.
Change "jQuery" to $

Download all attachments as: .zip

Change History (4)

Changed 14 years ago by vmx

Attachment: global-jquery-to-dollar-sign.patch​ added

Change "jQuery" to $

comment:1 Changed 14 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 14 years ago by Scott González

Owner: changed from paul to scott.gonzalez
Priority: major → blocker
Status: new → assigned

Already fixed in progressbar.

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

Resolution: → fixed
Status: assigned → closed
Type: enhancement → bug

Fixed in r951.

Note: See TracTickets for help on using tickets.