As a developer of Drupal modules using ExtJs extensively, it has been a real pain seeing Drupal's css conflict with ExtJs' css. And all efforts to get them to work together have proved futile without hacking into the core directly.

And I think most would agree on the usefulness of ExtJs inside Drupal modules. So, I request the core css to be tweaked to be compatible with ExtJs out-of-the-box.

I have found the system module's system.css to be the main culprit together with other css files.

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Version: 6.14 » 8.x-dev
Category: task » feature
r_honey’s picture

r_honey commented

8.x, well that looks like a lot of delay. Couldn't it be accommodated in 7.x atleast, and possibly 6.x (I might sound greedy here, but ExtJs really makes you so)!!!

damien tournoud’s picture

damien tournoud commented

6.x is the stable version, no change can possibly get in. 7.x is frozen, changes could get in if they are limited in scope and well justified. Given that we don't know the amount of changes required here, or if any change is necessary at all, the correct target version number is indeed 8.x.

r_honey’s picture

r_honey commented

Well, I tried tweaking the classes in system.css, and some changes made the majority of ExtJs components render correctly. However, it was still not perfect, and required further tweaking which I dumped because not many people would like to fiddle around with the core files.

I have made this request at ExtJs forums also, and would now hope, they turn up something much before Druapl 8.x.

casey’s picture

casey commented

You could post an initial patch, so we know what needs to be done.

OliverColeman’s picture

OliverColeman commented

I would be keen on seeing a patch for this. I'm looking at ExtJS 4 which has only recently been released. I'll eventually post a patch against Drupal 6.x (we're tied to 6.x for the time being) if none is forthcoming (I'm guessing this won't be very useful for any changes to Drupal 8, but perhaps it will make someone's life easier).

smovva3’s picture

smovva3 commented

Have you had a chance to create a patch for this. I'm also working with Drupal 6.x. Please share it when you get a chance.
Thanks!

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Issue summary: View changes
Status: Active » Closed (works as designed)

Stable theme has much less markup and designed to be as bare as possible so you don't need to remove as much from core. Likely not going to tie Drupal's CSS ExtJS but feel free to create a theme in Contrib that does this for the extjs community and hopefully our efforts to provide a less-Classy base theme will help with this feature in contrib.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

Oh and for reference: https://www.lullabot.com/articles/a-tale-of-two-base-themes-in-drupal-8-...

If you are following this and curious.