datepicker.1.7.css not loaded as default if jQuery UI 1.7 in use [#947752]

Comment	File	Size	Author
#7	jquery-date-popup.png	22.11 KB	geerlingguy
	date_popup_default.patch	673 bytes	slip

Comment #1

slip commented 20 October 2010 at 20:50

note: accidentally made the patch in date_popup/

Log in or register to post comments

Comment #2

ericduran commented 21 October 2010 at 21:55

Status:

Active

» Reviewed & tested by the community

I tested this and it fixed my issues with the wrong css.

Log in or register to post comments

Comment #3

slip commented 22 October 2010 at 13:15

Title:

datepicker.1.7.css not loaded as default if jQuery 1.7 in use

» datepicker.1.7.css not loaded as default if jQuery UI 1.7 in use

jquery ui, not jquery.

Log in or register to post comments

We might consider adding module_exists('jquery_ui') in this patch because right now date_popup_init will attempt to load the default css file even if jquery_ui is disabled and that causes an error when jquery_ui_get_version() is called.

Log in or register to post comments

Comment #5

karens commented 20 November 2010 at 11:33

Status:

Reviewed & tested by the community

» Fixed

Committed, thanks! I added a test for jquery_ui also.

Log in or register to post comments

Comment #6

geerlingguy commented 1 December 2010 at 11:56

I just downloaded the latest -dev release of date.module, and I'm still seeing unstyled popups - they work better, but no styling is applied. I'm using jQuery UI 1.7.x...

Log in or register to post comments

Comment #7

geerlingguy commented 14 December 2010 at 21:44

Status:

Fixed

» Active

Status	File	Size
new	jquery-date-popup.png	22.11 KB

Setting back to active to ask: is the attached how the datepicker is supposed to look? I just installed the latest -dev release of date.module (Dec. 13), and the date popup doesn't look anything like it used to before I installed jQuery UI 1.7.x (with jQuery Update 1.4.x).