Hi,
The main purpose of jquery UI is its ability of using different themes for the widgets. At present the jquery ui module is not including any css. When I add any widget the respective theme has to be included automatically. If no theme is specified the base theme should be included. At present we need to include respective theme css files explicitly.
I changed drupal_add_js function to include the respective themes.
Please check the attached patch
Note: You need to apply http://drupal.org/files/issues/jquery_ui_external_compress_type_bug.patch patch for http://drupal.org/node/388356 bug.
After applying the patch you can download any customized theme that you created using the jqueryUI Theme Roller interface and use here. The theme will automatically load when you add the widget.
To use new themes use the following procedure
Download any theme from from the http://jqueryui.com/download link and extracted it. Copy theme directory to jquery_ui/jquery.ui/theme folder with the name of the theme. I downloaded Vader theme
>> unzip jquery-ui-personalized-1.6rc6.zip
>> cp -r theme
/jquery.ui/themes/Vader
In my module I included that theme using the following function call
jquery_ui_add('ui.accordian','Vader')
That's it. The corresponding theme is enabled for that widget.
Comment | File | Size | Author |
---|---|---|---|
#122 | jquery_ui-themeroller-388384-122.patch | 3 KB | achton |
#113 | jquery_themeroller.patch | 2.92 KB | mmilo |
#102 | jquery_ui.09.12.2010.patch | 9.29 KB | mmilo |
#95 | jquery_ui_theme_management.tar_.gz | 10.32 KB | emptyvoid |
#77 | 388384-jquery_ui.patch | 2.78 KB | smk-ka |
Comments
Comment #1
pflame CreditAttribution: pflame commentedComment #2
RobLoachNicely done, but I think it would be nice to have that $theme variable pulled in through a setting in the administration somewhere. Would be nice to add it only for widgets that need the CSS too ;-) .
Comment #3
pflame CreditAttribution: pflame commentedHaving administration setting page to specify theme is a good idea but this theme will apply to whole site. Now you can have different theme on different pages. All jquery ui widgets come with basic css. On single page keeping different css for different widgets is something which I need to check, because I am not sure if it is possible.
Comment #4
pflame CreditAttribution: pflame commentedHaving administration setting page to specify theme for specific page is good idea. I will work on this feature. Thanks Rob Loach for giving me feedback.
Comment #5
pflame CreditAttribution: pflame commentedI implemented admin interface to set the default theme. Attach the patch and go to /admin/settings/jquery_ui page.
Here it will display the themes that are there in jquery_ui/jquery.ui/themes/ directory.
If you want to use default theme just add widgets with
jquery_ui_add('ui.accordian');
If you want to override default theme and want to use different theme then use
jquery_ui_add('ui.accordian','Vader');
Note: Please attach first patch before attaching this patch
Comment #6
pflame CreditAttribution: pflame commentedComment #7
psynaptic CreditAttribution: psynaptic commentedWhere does base in
variable_get('jquery_ui_default_theme', 'base');
come from? I can't seem to locate it in any jQuery UI theme download package.Edit: I downloaded jQuery UI 1.7.1: for jQuery 1.3+ and base was indeed included. This doesn't seem to be included in the jQuery UI 1.6: for jQuery 1.2.6 packages. As far as I can see jquery_ui currently works with jQuery UI 1.6: for jQuery 1.2.6 so there is no base theme.
Am I missing something or do we need to make this work without the base theme?
Comment #8
psynaptic CreditAttribution: psynaptic commentedI've improved the patch and feel it is now ready to be committed.
I have removed the necessity to apply this patch in the original post:
http://drupal.org/files/issues/jquery_ui_external_compress_type_bug.patch
I kept the $theme variable passed into the function so the theme of each widget can be changed in the call to jquery_ui_add(). I also kept the admin page with settings form but cleaned up the way theme folders are discovered. The variable
jquery_ui_theme
is still used to set the theme on a site basis.I have followed coding standards apart from the concatenation operator which follows the 7.x behaviour (this was already used in the file so I thought it would be best to conform to this).
There is still the problem that the 'base' theme is not present in the jQuery UI 1.6 theme packages downloaded from http://jqueryui.com/themeroller/ so I'd like to know if anyone has an idea of how we can deal with this. Maybe we should discover the themes and just enable the first found until another is set in the admin?
There is also extraneous whitespace on lines 17 and 57 but I didn't want to include that with my patch.
Comment #9
psynaptic CreditAttribution: psynaptic commentedIn keeping with the new release system and this post:
http://drupal.org/node/362509
I have changed the "base" theme in
variable_get('jquery_ui_theme', 'base');
to "flora".This is the default theme name when downloading the package from:
http://code.google.com/p/jquery-ui/downloads/list?q=1.5
Comment #10
psynaptic CreditAttribution: psynaptic commentedWhitespace cleanup only.
Comment #11
psynaptic CreditAttribution: psynaptic commentedThe download from http://code.google.com/p/jquery-ui/downloads/list?q=1.5 contains a theme with files arranged as such:
Whereas the Theme Roller has this structure:
How do think we should get around this? It seems the Theme Roller is only distributing packages with ui.all.css rather than individual files.
Comment #12
psynaptic CreditAttribution: psynaptic commentedMy lord, just had a good look at the CSS output from the Theme Roller and it SUCKS!
I think all development must have moved to 1.7 and 1.6 will be all but abandoned. I'm going to make my own CSS for this as what is output from the Theme Roller is no good for this project.
This issue has just been pushed right to the back of my queue unless someone comes up with something I may have missed.
Comment #13
pflame CreditAttribution: pflame commentedThanks for improving my patch. When I am posting this feature. the http://jqueryui.com/themeroller/ is different. They changed the directory structure. It looks like my patch does not work with 1.6 version. I downloaded 1.6 version of jquery_ui custom theme. After extracting, in the css folder there is a custom-theme directory. We can keep this in jquery_ui/jquery.ui/themes directory but it is not having ui.core.css, ui.theme.css.
jquery 1.7 version directory structure is good. I think it is better to use jquery ui 1.7 version. To use jquery-1.3.2 version , we can suggest the users to use jquery_update development code.
Prajwala
Comment #14
pflame CreditAttribution: pflame commentedI did not check http://code.google.com/p/jquery-ui/downloads/list?q=1.5 link.
Now also when I download jquery_ui development bundle from the jquery_ui home page http://jqueryui.com/home , I am getting the theme as base. You can download from http://jquery-ui.googlecode.com/files/jquery-ui-1.7.1.zip location.
Comment #15
psynaptic CreditAttribution: psynaptic commentedSo are you saying that jquery_update dev version now supports jQuery 1.3.2?
I agree jQuery UI 1.7 is much better than 1.5.3.
Edit: Ok, jquery_update 6.x-2.x-dev works with jQuery 1.3.2 so I think we should rebuild this patch to work with only jQuery 1.7 rather than the IMO very poor 1.5.3.
Comment #16
pflame CreditAttribution: pflame commentedyes I used jQuery 1.3.2 and jQuery UI 1.7 in one of my project. It is better to support new version than struggling to support older version. If we use use jQuery UI 1.7 , I hope my original patch is good, there might be small changes required.
Comment #17
psynaptic CreditAttribution: psynaptic commentedCurrent instructions to get Theme Roller themes working with the latest patch in this issue are:
You then need to select custom-theme as the theme for jquery_ui on admin/settings/jquery_ui
Comment #18
pflame CreditAttribution: pflame commentedInstall jquery ui module by following bellow steps.
You then need to select custom-theme as the theme for jquery_ui on admin/settings/jquery_ui
Comment #19
psynaptic CreditAttribution: psynaptic commentedThanks prajwala!
Any testers?
Comment #20
pflame CreditAttribution: pflame commentedTo make testers life easy. I attached a jQuery UI Test page module.
Just install and enable the module. Then you will see a jQuery UI Example link in the navigation. Click on the line then you should able to see Accordian, Tabs, Slider, DatePicker, Progressbar, Dialog, overlay and shadow, framework icons on the page.
This page is having all widgets supported by jQuery UI.
Try checking all widgets.
Try changing the JQuery UI theme from admin > settings > jquery ui page. The theme should effect on the example page.
Comment #21
adrinux CreditAttribution: adrinux commentedPatch from http://drupal.org/node/388384#comment-1530114 applies, works as described. I successfully modified a theme using the theme builder, downloaded and installed it.
Nice work :)
Comment #22
golchi CreditAttribution: golchi commentedI check and used this patch. It works as described. Good job (for a newbie you may need the http://drupal.org/files/issues/jqui_test.zip file to see how to use both the module and patch).
Comment #23
kardave CreditAttribution: kardave commentedI can confirm the above comment. I did the steps in #18, and working correctly.
Thanks,
Dave
Comment #24
sunThis patch slightly overlaps with #266692: Settings for the theme used by jQuery UI - can anyone clarify the difference(s)?
Comment #25
apadernoThe patch should also add the code to change the module priority in the installation file; differently, the module would include the CSS files before other modules have the chance to set a different theme.
As reported by sun in #266692: Settings for the theme used by jQuery UI, the value for the new priority should be the maximum value between 10, and the higher priority used by any enabled module.
Comment #26
pflame CreditAttribution: pflame commentedWith this patch we no need to define any .inc files and get_info functions.
This patch search for the different themes in the /jquery_ui/jquery.ui/themes folder. In the admin>settings>jquery_ui configuration page, It will display the different theme directories in the /jquery_ui/jquery.ui/themes folder. By default it will have 'base' theme.
This patch does not depend on the drupal theme that used. If the user want new theme for the jquery ui widgets, he can download theme from jQuery UI themeroller or can create new jquery UI theme and place in the jquery_ui/jquery.ui/themes folder.
I do not see any need to provide module priority. I used this patch in Accordion Blocks and Facebook Stream module. It works fine.
Comment #27
apadernoI thought the module implemented
hook_init()
to load the CSS files, but I was wrong. In that case, to change the module priority is not necessary at all.Comment #28
ck9 CreditAttribution: ck9 commentedCan someone please post idiot proof instructions on installing a new theme for a datepicker. I see a themes/default directory in my jquery.ui folder for v1.6. I'm assuming all this php / hook stuff is for module developers.
Comment #29
apadernoThe topic of this report is how to change the code to make it possible to change the widget theme basing on the theme used from Drupal. It's not a report for the end user, but for the maintaners of the project.
If you think the instructions are not clear, you should open a new report; attaching a comment to the first report you find is the right way to not get any answer.
Comment #30
ck9 CreditAttribution: ck9 commentedI suddenly get the impression I just walked in on an important meeting. I'll be leaving now. =)
Comment #31
owntheweb CreditAttribution: owntheweb commentedHello,
I'm totally new to using patches, but I gave it a shot. I think I'm going in the right direction, but some errors occurred. Here's what I did:
command line output:
jquery_ui.module.rej:
Any thoughts on how to fix this? Is there a patched file or files I can download and replace? That would make my life wonderful for this 'quick' project I'm working on. I appreciate the assistance as I build on my tiny but growing Drupal muscles. :)
Comment #32
pflame CreditAttribution: pflame commentedHello ultrus,
Thanks for mentioning patch error for the new version of jquery ui.
I created this patch using jquery_ui 6.x-1.2 version. Try downloading jquery_ui 6.x-1.2 from http://drupal.org/node/274037 and apply this patch for now.
I will create new patch file for the latest version of jquery_ui and upload here.
Regards,
Prajwala
Comment #33
pflame CreditAttribution: pflame commentedCreated new patch which support latest version of jquery_ui module. Please use the patch attached to this comment.
Comment #34
dirk negen CreditAttribution: dirk negen commentedHi there, I was wondering where exactly to apply the patch?
Comment #35
pflame CreditAttribution: pflame commentedThe above patch add some more code to jquery_ui.module file. check http://drupal.org/patch/apply to how to apply patch.
Comment #36
bluestarstudios CreditAttribution: bluestarstudios commentedwill this become part of the next jQuery UI module release?
Comment #37
pflame CreditAttribution: pflame commentedLooking for becoming part of jquery ui release.
Comment #38
adrinux CreditAttribution: adrinux commented@azriprajwala please learn how to create patches against a cvs checkout :) You really shouldn't need to specify the file when applying a patch.
Attached is a different version of the latest patch, which should apply as usual:
"patch -p0 < jquery_ui_388384_v5_clean.patch"
Confirmed as working when applied to jquery_ui.module 6.x-1.3.
Comment #39
adrinux CreditAttribution: adrinux commentedOh, and using jQueryUI 1.7.2 with, jquery_update 6.x-2.x-dev (2009-Apr-24) and jQuery 1.3.2.
Comment #40
Tim Saltzman CreditAttribution: Tim Saltzman commentedHi I have my drupal site hosted on godaddy. Can someone tell me where and how to install this patch please? I can't see where I am supposed to enter the commands for this patch. I need to install this patch to be able to use the Accordion Blocks Module.
Comment #41
apaderno@#40: The patch should simply applied to a local copy of the module files, which are then copied to the server through FTP (I think you don't have SSH access to the server where your site is hosted).
See Applying patches for more details.
Comment #42
johnny_up CreditAttribution: johnny_up commentedHi,
I've tried to implement this, and I've had no luck at all. I downloaded/installed jquery update (jquery_update-6.x-1.1) , then jquery_ui (jquery_ui-6.x-1.3), and enabled them ok without any errors. I downloaded the new jquery_ui (jquery-ui-1.7.2), and added it, as well as the new theme from themeroller (jquery-ui-1.7.2.custom). Then came the patch...The patch - jquery_ui_388384_v5.patch - successfully ran (or so it seemed, I used CygWin), but the I now get the error:
Fatal error: Call to undefined function jquery_ui_add() in the accordian blocks module on line 13. Any ideas? I'll attach the resulting new module file after the patch as a txt in case it'll help.
My goal is to use the accordian blocks module...
Thanks in advance!
Comment #43
pflame CreditAttribution: pflame commentedHi johnny,
You should install jquery update 6.x-2.x-dev version because jquery ui 1.7 need jquery 1.3.2.
This error will come because if you do not enable jquery_ui module.
Please follow the instructions specified at http://drupal.org/node/388384#comment-1530114
You have to enable jquery_update, jquery_ui then accordion_blocks module.
Comment #44
johnny_up CreditAttribution: johnny_up commentedThanks for that, azriprajwala - I seriously appreciate it. The problem was that I didn't install the right version of jQuery_update. It works like a charm now.
Comment #45
GiorgosKThis is not a support request
please don't change category - unless you know what you are doing
tried latest patch v5, with jquery update 2.x installed and jquery ui 1.3
the css are inserted correctly
but I can't get datepicker to style at all when using ui-lightness
any ideas
Comment #46
colin_young CreditAttribution: colin_young commentedSo this patch applied fine and appears to be creating the correct css links, but none of the styles are actually appearing.
The css files can all be loaded using the paths found in the header include, so I'm not sure what's going on. I'm going to keep investigating and I'll report back here if I find anything useful.
Comment #47
colin_young CreditAttribution: colin_young commentedThere's something funky going on with the themes. I'm getting all the right JQuery js files included, but the theme is applied inconsistently. On one page I get all the right CSS loading, but on another, none are loaded.
I'm calling jquery_ui_add in template.php ({theme}_preprocess_page hook), but I've found that in order to get the themes to show up at all, I need to call it in e.g. my php for my panels 3 layout file. CSS links are only generated by calling jquery_ui_add in the panels layout file.
I've tried to call jquery_ui_add from page.tpl.php, but it doesn't appear to work from there at all. I think some of my issues may not be related to this patch, but some are.
Comment #48
pflame CreditAttribution: pflame commentedHi colin_young,
jquery_ui_add function is similar to drupal_add_js. Normally I use jquery_ui_add in my init hook or in my menu callback.
Comment #49
colin_young CreditAttribution: colin_young commentedOkay. I've modified my theme's template.php:
That seems to have taken care of things. Theming is working for me (I tested out the DatePicker also, and that's getting themed).
Nice work on this patch. Thanks!
Now I can get back to perverting the Drupal UI in ways that I'm probably going to regret when it comes to upgrading to D7 :)
Comment #50
bluestarstudios CreditAttribution: bluestarstudios commentedSo since this is working, when can we see this being finally released in an official release?
Thanks!
Comment #51
pflame CreditAttribution: pflame commentedI am also looking for this patch to be added to release.
Comment #52
mrfelton CreditAttribution: mrfelton commentedplease lets get this committed.
Comment #53
mrfelton CreditAttribution: mrfelton commentedWhen I downloaded the 'smoothness' theme from jqueryroller, these are the files that were included:
Note the name of the .css file - this is not what the module was expecting, and so the css was not loaded correctly. I renamed the css file to ui.theme.css and then it worked. Either this needs to be added to the installation instructions, or the code needs to be more flexible with the name of the css file it looks for.
Comment #54
pflame CreditAttribution: pflame commentedmrfelton,
What you specified is correct. Jquery UI changed the directory structure of the theme roller download now. When you download any theme from theme roller now u get this directory structure
You can use the directory from sommothness/development-bundle/themes/smoothness.
Comment #55
srobert72 CreditAttribution: srobert72 commentedSubscribing.
Could it be possible to include this patch in DEV version ?
Comment #56
emptyvoid CreditAttribution: emptyvoid commentedSubscribing
This is exactly what I was planning to contribute to this module :)
http://drupal.org/node/672970#comment-2432960
Comment #57
Mac Clemmens CreditAttribution: Mac Clemmens commentedSubscribing
Comment #58
hgltao CreditAttribution: hgltao commentedI installed modules Blocks in Accordion, Jquery_UI 6.x-1.3(with Jquery-ui-1.72). I used the accordion effect. The menus can be openned, when I clicked the menu. But it can not be closed, when I clicked another menu. After I clicked all the menus, all menus will be openned. It seems that the accordion effect doesn't work on my Drupal.
Any ideas? Why does it not work?
Comment #59
pflame CreditAttribution: pflame commentedHi hgltao,
You should have posted in the Blocks in accordion module issue queue. This project is about providing jquery UI. To use jquery-ui-1.72, you should have jquery 1.3.2 and above. Check which version of jquery is available in your instance. If required install jquery-update6.x-2.x version, this version of jquery-update module will provide jquery 1.3.2.
Comment #60
MacMannn CreditAttribution: MacMannn commented@azriprajwala:
On december 5th you said you wanted this patch committed. As far as I can see, that hasn't happened yet. Are there still issues that need to be solved?
Anyways, I appreciate the work that has been done on developping this module / patch. At the moment I'm just wondering if it would be wise to enter into a major tweaking session with this change being committed any time now...
I'd much rather use the default module — maintaining Drupal is hard enough as it is ;-)
Attached, you'll find the datepicker as it is showing up when using the default installation with the currently available production module for D6. Not verry attractive, especially not when compared to what can be created using the themeroller...
I'd love a simple, straightforward way to choose a fitting theme for the datepicker on my sites.
Comment #61
pflame CreditAttribution: pflame commentedHi MacMannn,
There are no issues regarding this patch. My be I am unable to communicate properly what exactly this patch is about.
When I place this patch the default module did not provide loading of default css. That means when we install the module and enable date picker we could not see the out put as it is appearing now, It used to be datepicker without any css and it used to look like this module is not working. So I have created this patch by adding support to load default css and support for the themeroller. I think it would be better I create new ticket by providing the patch about themeroller.
Thanks for making a note and making me think about this patch.
Comment #62
bluestarstudios CreditAttribution: bluestarstudios commentedok. now I'm really confused. If I'd like to benefit from the theme roller and use a theme that I generated with it, what do i have to do? Can anybody give a step by step?
Comment #63
pflame CreditAttribution: pflame commentedI have tested the lated patch file http://drupal.org/files/issues/jquery_ui_388384_v5_clean.patch with the latest development version of this module and it is working good.
@bludstarstudios,
I understand why do you get confused. while writing http://drupal.org/node/388384#comment-2745992 comment I did not test the current development version of the jquery ui module with the patch I provided here, and I thought the module itself was loading the basic css required for jquery ui widgets to appear. But the development module does not loading any css.
Follow bellow steps to use jquey ui theme roller support.
Comment #64
yettyn CreditAttribution: yettyn commentedAnyone tested this with latest jquery.ui-1.8 released stable just a few days ago?
EDIT: Ahhh, I see now it uses jquery-1.4.2 so will probably not work unless also jquery_update module is upgraded.
Comment #65
dapperry CreditAttribution: dapperry commentedDear azriprajwala,
I have followed your instructions in #63 exactly. While I can now pick one of the themes in the JQuery UI, the stylesheet of the selected theme does not show up in my module page, and obviously the css is not working. There seems to be a real problem with the directory structure being different from earlier. I have tried to work around this, but maybe thats the problem. Is there anyway you can provide a download of the entire jquery_ui module (with patched module, jquery and all the standard themes)? I have spent hours trying to get this to work, and I just need a working copy.
Thanks,
David
Comment #66
dapperry CreditAttribution: dapperry commentedBTW,
If I manually add:
to my mymodule-form.tpl.php file it does indeed work.
:) David
Comment #67
pflame CreditAttribution: pflame commentedHi dapperry,
I have attached working jquery_ui module. install jquery_update-6-x-2.x-dev module.
Comment #68
dapperry CreditAttribution: dapperry commentedThanks azriprajwala,
This works great!
:) David
Comment #69
bluestarstudios CreditAttribution: bluestarstudios commentedthanks so much for your help. :) Will try to see how it works and will let you know. :)
Quick question about jQuery Update. The last time I was using it the update would break a lot of the default drupal actions like opening and closing fieldsets on forms. Do you know if that is still the case?
Comment #70
pflame CreditAttribution: pflame commentedHi bluestarstudios,
I used jQuery Update 6.x-2.x version in almost all projects that I developed. I don't see any breakage of drupal action.
I am sure lightbox2 module does not work, because the js code in the lightbox2 used some deprecated selectors of jquery1.2.6 like @. Those deprecated things are removed in jquery1.3.
If you are using lightbox2 module and jquery_update6.x-2.x then lightbox does not work properly and it causes lot of problems like always opening lightbox for all links.
Comment #71
srobert72 CreditAttribution: srobert72 commentedAs I know Lightbox works very well with JQuery 1.3.
I use Lighbox with JQueryUpdate 6.x-2.x (JQuery 1.3.2)
I have also tried with JQueryUpdate upgraded manually with JQuery 1.4.2.
And Lightbox still works.
Comment #72
hewdaily CreditAttribution: hewdaily commentedThis worked for me but I only tested it against the accordian_blocks module and I can't collapse the admin block but...whatever, that's a different issue - my custom styles are showing.
*NOTE*
Double-check your themeroller download before you leave the themeroller site because the downloader is buggy. So download your custom theme, unzip, go into the unzipped directory and goto development-bundle->themes->your_theme_name and make sure you have more than just ui.base.css sitting in there. You should have ui.datepicker.css, ui.dialog.css, and all the others that you left the check marks on at the themeroller download page. If you don't make sure all those files are there then you may end up following all the instructions above and see only partially styled objects, or no styling at all. It will seem like this Drupal jQuery UI thing isn't working when in fact it can't work because there are no files in your custom theme directory to work with.
Comment #73
iparsin CreditAttribution: iparsin commentedTo hgltao #58,
I have got exactly same issue earlier. However, after I uninstalled and reinstalled all modules again(strictly following the instruction with right versions on each module), it is working perfectly.
Thank you azriprajwala, you are awesome!
Comment #74
attheshow CreditAttribution: attheshow commentedsubscribing
Comment #75
victor.gurung CreditAttribution: victor.gurung commentedHi
I need to use this patch but I am confused where exactly i need to keep the code provided in the pacth. can any body help me out
Comment #76
pflame CreditAttribution: pflame commentedHi victor,
Use the patch from http://drupal.org/node/388384#comment-1851354. And you can check http://drupal.org/patch/apply link for how to apply patch.
Comment #77
smk-ka CreditAttribution: smk-ka commentedRe-rolled with some changes:
- Since the files of the 'base' theme that is included with this module differs from custom themeroller styles, we need to differentiate between the two.
- The biggest difference is that themeroller styles come glued together into one ui.all.css file (based on the selection made before downloading), while the included 'base' theme allows adding modular stylesheets based on feature (i.e., datepicker, tabs, etc.).
- The previous patch didn't differentiate and added a lot of files that don't exist when using themeroller.
Comment #78
juroon CreditAttribution: juroon commentedI just followed the steps in http://drupal.org/node/388384#comment-2747704, but used the new patch in http://drupal.org/node/388384#comment-3221462
I'm probably not testing everything there is to test, but it looks good so far. I downloaded the themed version of jQuery UI 1.7.3 (Legacy release, for jQuery 1.3.2) from http://jqueryui.com/download
This resulted in a download file named jquery-ui-1.7.3.custom.zip rather than one named *.personalized.* as mentioned above. The extracted directory contained the development bundle, which I renamed to jquery.ui and uploaded to my jquery_ui module.
I was then able to select my them in the jQuery settings at .../admin/settings/jquery_ui
Will post back if I discover any subsequent problems.
Comment #79
mansspams CreditAttribution: mansspams commentedtested #77. works. some css ugliness visible, but will try fix described here #672970: Theming Modal Frames
Comment #80
mansspams CreditAttribution: mansspams commentedYes. Works.
Comment #81
smk-ka CreditAttribution: smk-ka commentedIt would be cool if we could add the themeroller instructions either to the README or directly on the ui theme selection page. Somebody up for this task?
Comment #82
pflame CreditAttribution: pflame commentedsmk-ka,
Yes, it is good to have the themeroller instructions in README. I will do that and submit the new patch with every thing.
Comment #83
Josh Benner CreditAttribution: Josh Benner commentedsubscribing
Comment #84
drewish CreditAttribution: drewish commentedI'm not quite sure this logic is correct at least as far as the "smoothness" theme for 1.7.3. It has:
jquery-ui-1.7.3.custom.css
ui.accordion.css
ui.all.css
ui.base.css
ui.core.css
ui.datepicker.css
ui.dialog.css
ui.progressbar.css
ui.resizable.css
ui.slider.css
ui.tabs.css
ui.theme.css
and ui.all.css is:
while ui.base.css is:
The jquery-ui*.css files looks like a concatenation of all the other CSS files.
I don't really have any recommendations at this point because the @imports would bypass the CSS aggregation but I'm not clear on the logic of the theme layouts in other versions.
Comment #85
jbomb CreditAttribution: jbomb commentedWorks nicely. I added a couple CSS rules to my theme to get the theme roller to play nice with the date_popup module. That's more of an issue for the date module assuming the patch from #77 or something similar is committed here, but I've included them for others that might be testing.
Comment #86
vlad.k CreditAttribution: vlad.k commentedSubscribing...
Comment #87
aaron CreditAttribution: aaron commentedsubscribe
Comment #88
kendouglass CreditAttribution: kendouglass commented+
Comment #89
drewish CreditAttribution: drewish commentedMarking #834726: Allow to select jQuery UI theme as a duplicate, since it's functionality should be merged in here.
Comment #90
drewish CreditAttribution: drewish commentedI feel like trying to move all these individual patches in the issue queue isn't very productive so I've applied this and several others to a github repo:
http://github.com/drewish/jquery_ui
The theme roller support is kind of broken right now but themes should work. I'd love feedback if anyone has time to test it. It's compatible with jQuery 1.8.
Comment #91
drewish CreditAttribution: drewish commentedAlso marked #814232: jquery_ui_add() does not add required CSS files as a duplicate.
Comment #92
crea CreditAttribution: crea commentedsubscribing
Comment #93
giorgio79 CreditAttribution: giorgio79 commentedAfter installing the latest version of this module, my old themeroller theme css has been overridden with the base theme included in jquery ui module's jquery folder.
Could this patch restrain the scope of the default themeroller base theme?
http://www.filamentgroup.com/lab/using_multiple_jquery_ui_themes_on_a_si...
Here is what was added to my pages:
I am using Skinr which requires this module, perhaps that is why this line was added.
[Edited by kiamlaluno to make the hidden HTML tag visible.]
Comment #94
pflame CreditAttribution: pflame commentedgiorgio79,
This patch does not allow to add multiple themes. It allow to add same theme for all the widgets used in the site. There is a admin interface to select the theme you want to use.
Comment #95
emptyvoid CreditAttribution: emptyvoid commentedThis may or may not help, I already have a working version of this that I use on projects
Comment #96
kendouglass CreditAttribution: kendouglass commented+
Comment #97
j4 CreditAttribution: j4 commentedHi,
I used both the current jquery_ui 6.1.4 version and the dev version also. Followed all the steps. The accordian blocks works, Theming all fine. But the alwaysOpen=false will not work and once clicked open the blocks will close only on reload of the page. Can you tell me where I am going wrong? Even tested the jquery test.zip module and the accordian in that works in the same manner and that in my site. Also tried changing themes to check if it was a css related problem, but works in the same manner in all themes..once opened blocks will not close. Please help!
Thanks
Jaya
Comment #98
lynnmart881 CreditAttribution: lynnmart881 commentedI installed the Jquery_ui according the steps in #18, using the v5 patch instead of the one listed on #18. Everything went as it should have. My only problem is that I do not have a jquery_ui settings page in my admin/settings area. Any ideas what went wrong?
Comment #99
lynnmart881 CreditAttribution: lynnmart881 commentedNevermind. After I deleted the accordian block module that I was trying to use, the jquery_ui settings page appeared. My problem must be with that module.
Comment #100
davidwhthomas CreditAttribution: davidwhthomas commented@emptyvoid http://drupal.org/node/388384#comment-3667740
Thanks for the code, I'm using that in place of jquery_ui.module and working perfectly with the themeroller download 'theme' dir copied over.
Could do with a tweak to allow jquery_ui themes in one's theme dir, but otherwise good.
DT
Comment #101
marcoka CreditAttribution: marcoka commenteddid not work here using jquery ui 1.6 and the newest jquery_update module version 1.3.2
edit: Updated jquery_ui to 1.7.3, now it works :)
Comment #102
mmilo CreditAttribution: mmilo commentedRerolled #77 to include changes in #95, against 6.x-1.x-dev.
Using -dev verision of jquery_update, and jQuery UI 1.7.3.
So, a summary for anyone else new to the issue:
- There is a new menu option to select a global jquery theme
- there is a new parameter for jquery_ui_add that lets you specify a specific theme to add .css files for
I only did light testing, but this seems to work for both core/custom themes.
Comment #103
BWPanda CreditAttribution: BWPanda commentedI've followed the instructions from #18, using the patch from #102 and the latest jQuery v1.7.3. I downloaded the pepper-grinder theme and installed it into the themes directory of jquery.ui in sites/all/libraries. I then selected that theme on the jQuery UI admin page.
I have some select lists and a datepicker calendar popup in a form on my frontpage, and after clearing all caches, I'm not seeing any difference to the look of the form...
What am I missing?
Comment #104
Manuel Garcia CreditAttribution: Manuel Garcia commentedWe should so work for something like this for drupal 7, now that we have jquery UI in core.
Comment #105
jensimmons CreditAttribution: jensimmons commentedSo we've had a working patch to allow people to use custom jQuery UI themes for over a year without it ever getting committed? WHY??? This should really be part of this module. Otherwise, we all have to fork this module to use a custom jQuery UI theme. ;(
sun, what's up?
Comment #106
tim.plunkettPatch still applies, will test later today.
Comment #107
jensimmons CreditAttribution: jensimmons commentedMeanwhile, I did what I've done in the past:
You don't have to patch this module (maintaining a patched module for over 2 years isn't going to work in most situations), and you can still use a custom jQuery UI theme by putting it in your theme.
Why won't patching the module work in most situations? Because it's likely someone new will be taking care of maintenance of the site, and they won't know how to apply a patch. Or they wouldn't know that they should. Or they'll forget to. And the consequences are big — the look and feel of the site will break if the module is upgraded with a non-patched copy.
Comment #108
jstollerI've been using this patch for several weeks without any problems. I'll second the call to get it committed as soon as possible.
Comment #109
thehong CreditAttribution: thehong commentedTwo years old issue :(
Comment #110
jstollerCould one of the module maintainers provide a little insite here? Is this patch, or one like it, on the roadmap? Is there something preventing it from being committed?
Comment #111
pflame CreditAttribution: pflame commentedI am also waiting for one of the module maintainers to comment about the issue. I am using this patch in the projects where I use jquery ui. Lets start voting for this issue. Please comment here the people who are using this patch.
Comment #112
mansspams CreditAttribution: mansspams commentedthis sounds good http://drupal.org/sandbox/chabby/1105070
Comment #113
mmilo CreditAttribution: mmilo commentedAny update on this?? Here's a re-roll of my patch in #103 with 2011-Feb-25 6.x-1.x-dev. Didn't try out the module in #112.
Comment #114
ohnobinki CreditAttribution: ohnobinki commented+
Comment #115
keichee CreditAttribution: keichee commentedlets make a simpletests
Comment #117
Manuel Garcia CreditAttribution: Manuel Garcia commented0_o
Comment #118
Manuel Garcia CreditAttribution: Manuel Garcia commentedIs there a reason why we'd have to force people to name their theme's css files with the version number?
We could just use a generic one like jquery-ui-THEMENAME.css like this:
$css_path = $theme_path . '/jquery-ui-'. $theme .'.css';
What do you think? Easier to maintain in my opinion.
Comment #119
SchwebDesign CreditAttribution: SchwebDesign commentedsubscribing - I would really LOVE to see this working as well
Comment #120
kongoji CreditAttribution: kongoji commented#113 works for me. This patch should be committed in my opinion
Comment #121
pflame CreditAttribution: pflame commentedComment #122
achtonI couldn't get the patch in #113 to apply cleanly against 6.x-1.5. Here is a git-compatible port of the patch.
Should work for current 6.x-dev too.
Comment #123
j0rd CreditAttribution: j0rd commentedBeen having this problem which is marked dupe of this issue.
#814232: jquery_ui_add() does not add required CSS files
I've applied patch #122. Applied clean against 6.x-1.5 for me. My problem is though, with jquery.ui 1.6 is that the default theme folder is called "default" and not "base" in my install (pre-existing site I'm taking over).
With the admin interface, this was a quick fix, but it didn't work out of the box, which most users (including me, would expect)
Also I've downloaded a version of 1.6 from jquery ui website to see if perhaps someone changed it, before I started working on this site. The fresh download also does not have the "base" folder in themes, instead of folder called "smoothness" and no "default" folder, go figure :D
Here's that link:
http://jqueryui.com/download/jquery-ui-1.6.custom.zip
jQuery 1.7 & 1.8 appears to have both 'base' and 'smoothness' folders:
http://jqueryui.com/download/jquery-ui-1.7.3.custom.zip
http://jqueryui.com/download/jquery-ui-1.8.18.custom.zip
Since my site is not using jquery update, and Drupal 6 my version of jquery is 1.2.6 and I believe jquery ui 1.6 is the last version of jquery ui to support this version of jquery. Because of this, 1.6I think we should make it work "out of the box" with jquery ui 1.6.
http://jqueryui.com/docs/Changelog/1.6
UPDATE:
Just confirmed, as per install instructions, jquery_ui module tells you to download jquery ui 1.6 from here http://code.google.com/p/jquery-ui/downloads/list?q=1.6 . and this version only has a "default" theme folder, not base.
Comment #124
cangeceiro CreditAttribution: cangeceiro commentedapplied the patch in #122 and this seems to have fixed my issue with ui-slider.
Comment #125
apadernoI am closing this issue, which is for a not supported Drupal version.