#5638 closed bug (fixed)

Opened May 20, 2010 02:21PM UTC

Closed May 21, 2010 05:27PM UTC

Last modified May 31, 2010 03:44PM UTC

Download builder: theme missing new file jquery.ui.selectable.css on ui-dev.jquery.com 1.8.2

Reported by: rdworth Owned by: jdsharp
Priority: blocker Milestone: 1.8.2
Component: [meta] ui.downloadbuilder Version: 1.8.1
Keywords: Cc:
Blocked by: Blocking:
Description

1.8.2 is now on the test server ui-dev.jquery.com

When downloading a zip, the new file jquery.ui.selectable.css is missing from the theme folders and there's also no evidence of it in jquery.ui.base.css in the theme folders.

Attachments (0)
Change History (4)

Changed May 20, 2010 02:56PM UTC by jzaefferer comment:1

Not tested, but could be enough to just add ", theme" here, in site svn:

Index: config.inc.php
===================================================================
--- config.inc.php	(revision 1277)
+++ config.inc.php	(working copy)
@@ -74,7 +74,7 @@
 		'ui.resizable.js' 	=> array('hasDemo' => true, 'hasCss' => true, 
 			'name' => 'Resizable', 'desc' => 'Makes any element on the page resizable.', 'requires' => 'ui.core.js, ui.widget.js, ui.mouse.js, theme'),
 		'ui.selectable.js' 	=> array('hasDemo' => true, 
-			'name' => 'Selectable', 'desc' => 'Makes a list of elements mouse selectable by dragging a box or clicking on them.', 'requires' => 'ui.core.js, ui.widget.js, ui.mouse.js'),
+			'name' => 'Selectable', 'desc' => 'Makes a list of elements mouse selectable by dragging a box or clicking on them.', 'requires' => 'ui.core.js, ui.widget.js, ui.mouse.js, theme'),
 		'ui.sortable.js' 	=> array('hasDemo' => true, 
 			'name' => 'Sortable', 'desc' => 'Makes a list of items sortable', 'requires' => 'ui.core.js, ui.widget.js, ui.mouse.js'),
 	),

Changed May 21, 2010 05:23PM UTC by jdsharp comment:2

owner: → jdsharp
status: new → assigned

Changed May 21, 2010 05:27PM UTC by jdsharp comment:3

resolution: → fixed
status: assigned → closed

This has been resolved on ui-dev.com. In addition to adding the theme requirement, the " 'hasCss' => true" pair was added to the ui-selectable.js array.

Changed May 31, 2010 03:44PM UTC by rdworth comment:4

milestone: 1.9 → 1.8.2

Fixed in ui website rev 1280