Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The Seven theme includes an override for jQuery UI's jquery.ui.theme.css
. Hence in the backend, the autocomplete continues to look fine. However, in the front-end (Bartik), it looks like utter crap (i.e. jQuery UI's own jquery.ui.theme.css
is used). This means that when you use in-place editing or an exposed Views filter that filters on tags… it's going to look like crap:
- Seven
- Bartik
Comment | File | Size | Author |
---|---|---|---|
#29 | Screen Shot 2023-01-12 at 11.42.17 am.png | 28.71 KB | pameeela |
#12 | autocomplete-bartik.png | 12.39 KB | tompagabor |
#12 | autocomplete-seven.png | 14.05 KB | tompagabor |
#6 | interdiff.txt | 600 bytes | Karmen |
#6 | drupal_core-autocomplete_clean_up_jquery_ui_css_and_styles-2186649-6.patch | 32.21 KB | Karmen |
Comments
Comment #1
InternetDevels CreditAttribution: InternetDevels commentedComment #2
barraponto CreditAttribution: barraponto commentedI don't think we need
filter:alpha
since Internet Explorer 6, 7 and 8 are no longer supported. See https://drupal.org/node/1569578.Comment #3
Karmen CreditAttribution: Karmen commentedI just remove the filter for ie. It's ok?
Comment #5
Karmen CreditAttribution: Karmen commentedComment #6
Karmen CreditAttribution: Karmen commentedAdd the new patch, with the interdiff. Sorry for the confusion.
Comment #8
penyaskito6: drupal_core-autocomplete_clean_up_jquery_ui_css_and_styles-2186649-6.patch queued for re-testing.
Comment #9
Karmen CreditAttribution: Karmen commentedComment #10
penyaskitoTagging.
Comment #12
tompagabor CreditAttribution: tompagabor commentedi tested it again.
Why we need, but why we need the ui icons? I didn't see any icons when tested the patch.
And there was some style changes, see attached screenshot, we need border on seven theme, and the dashed separator i don't think good choice for bartik.
Seven:
Bartik:
Comment #13
feonor CreditAttribution: feonor commentedComment #14
kostyashupenkoI can't reproduce it on my side with the last patch. Actually there was no jquery.ui.theme.css in sources of browser. All files from /core/themes/bartik/css was enabled except of jquery.ui.theme.css . I've tried to add this file and saw nothing. So https://www.drupal.org/node/2186649#comment-9122471 looks weird for me
Comment #15
emma.mariaI'm moving this issue into the Bartik queue as the patch contains 100% Bartik files and I need to keep track of these things :-)
Comment #29
pameeela CreditAttribution: pameeela commentedThe problems identified here are no longer reproducible, and the history of this and the related issue make this really confusing.
Current state of autocomplete in Bartik:
If folks think this needs improvement, we should create a new issue with the proposed changes.