The views-admin.rubik.css should only be loaded when on the views ui.

This should not be loaded on every page.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ericduran’s picture

Status: Active » Needs review
FileSize
7.68 KB

Here's a patch. This patch does a couple of things.

- It remove views-admin-rubik.css from rubik.info file
- it renames the views-admin-rubik.css to views-admin.rubik.css since thats the appropriate syntax.
- Add a css alter to rubik_css_alter because views will add the css if it's assuming it's on the views module folder. Instead we change the data attribute to replace it with our rubik theme folder.

ericduran’s picture

FYI: I added a patch to views #1221560: Allow any theme to declare their own views-admin.{theme_key}.css which should make this easier. If that patch gets in, we can remove the css_alter for the views admin css file.

fxarte’s picture

I could not apply the patch, there may have been some changes in the theme since it was created so I created a new one

fxarte’s picture

Assigned: ericduran » Unassigned
ericduran’s picture

Assigned: ericduran » Unassigned

This patch is no longer needed, at least not the whole patch.

I open an issue in the views queue to fix the problem with views-admin theme not being loaded.

CoffeyMachine’s picture

Status: Needs review » Fixed

The patch by fxarte did not include the new renamed views-admin.rubik.css. I went ahead and re-did the changes myself and just committed.

Even though views may allow for this in the future, I think this is a fix worth including now. We can revisit this and remove the lines in the rubik_css_alter once #1221560: Allow any theme to declare their own views-admin.{theme_key}.css gets committed and released.

Thanks folks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

codi’s picture

Assigned: Unassigned » codi
Status: Closed (fixed) » Needs review
FileSize
2.11 KB

Sorry to re-open. The change to views was made so I've removed the css_alter.

ericduran’s picture

Version: 7.x-4.0-beta6 » 7.x-4.x-dev
Status: Needs review » Reviewed & tested by the community

Looks correct to me, and with some clean up :)

CoffeyMachine’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x-4.x
Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.