Closed (fixed)
Project:
Rules
Version:
7.x-2.x-dev
Component:
User interface
Priority:
Critical
Category:
Task
Assigned:
Reporter:
Created:
8 Sep 2010 at 07:37 UTC
Updated:
8 Jan 2011 at 16:29 UTC
Jump to comment: Most recent file
>Maybe tweak the JS to popup with autocompletion suggestions as soon as the data selector is focussed?
Not possible without lots of hacking of core-autocomplete :/
Maybe integrating the jqueryUI autocomplete combobox widget (js is in core) would be the best thing to do.
http://jqueryui.com/demos/autocomplete/#combobox
Unfortunately my JS skills are rather limited, so any help is welcome.
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | rules-905158-5.patch | 3.32 KB | sepgil |
| #13 | rules-905158-4.patch | 2.74 KB | sepgil |
| #8 | rules-905158-3.patch | 10.28 KB | sepgil |
| #7 | rules-905158-3.patch | 10.28 KB | sepgil |
| #5 | rules-905158-2.patch | 8.69 KB | sepgil |
Comments
Comment #1
sepgil commentedimplemented it.
If you use the seven theme as you're administration theme you'll need the patch of the patch of this issue #928572: Improve interaction states and add corner classes to jQuery UI Seven Theme in order to see the combo box like button.
Comment #2
sepgil commentedforgot to set the status...
Comment #4
fagoRules is part of Drupal, so I guess we should be somewhere under Drupal -> Drupal.Rules.
I do not think this extra spaces are valid JS code style?
We should try to come up with better variable names here. Perhaps 'instance' makes clear it is an instance of the current "class"? Or better something like Autocomplete ?
Is this wrong spacing part of the hack? Probably.
Let's start to adopt lower case hex codes too.
The comment should be above the actual PHP line.
This is specific to the data selection widget, but the naming doesn't reflect his. Better use 'rules_data_selection' instead of 'rules_autocomplete' as element type. I think we could leave the theme and JS generic as "rules_autocomplete" though.
Comment #5
sepgil commentedFixed the issues
Comment #7
sepgil commentedThis new Patch should apply...
Comment #8
sepgil commentedforgot to set the status
Comment #10
fago#8: rules-905158-3.patch queued for re-testing.
Comment #12
fagoI've done some small improvements, mostly on the PHP side and committed it. thanks :)
+ I've done some css improvements.
Further possible improvements/problems:
* Click in the textfields lets the autocomplete popup. But click on the button to hide it now, doesn't work.
* When selecting a group ending with the colon, the autocomplete suggestions should refresh and the popup should stay open.
Comment #13
sepgil commentedcorrected the 2 problems...
Comment #14
sepgil commentedNoticed that by fixing the first issue, the fix for the second problem was broken. With this patch, both problems should be gone...
Comment #15
fagothanks, committed.
Comment #17
carvalhar commentedhi,
i need to implement jquery ui autocomplete combobox at my drupal 6 installation.
Can you give me an advice about how you did it?
thanks.