The following settings inside Configuration/Development/Performance are killing the Color module

  1. Compress cached pages
  2. Aggregate and compress CSS files
  3. Aggregate JavaScript files
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch’s picture

Version: 7.0 » 8.x-dev
Issue tags: +Needs backport to D7

Could you describe 'killing' in a bit more detail?

globexplorer’s picture

FileSize
145.84 KB
158.75 KB
154.01 KB
153.76 KB

When you enable JavaScript compression than, the JavaScript on the Preview won't work anymore. The colors are gone. Also a white space on the page top will appear. The colors will change, but the Preview is corrupt.

Look at the screenshots. I tested it with Bartik and some other themes. They are all the same.

catch’s picture

Priority: Critical » Major

Since aggregation is breaking only the color module interface, not the front facing site, I think this qualifies as major rather than critical. I have a feeling there may already be an issue in the queue dealing with this.

sun’s picture

Priority: Major » Normal
Issue tags: -color module

Since there is a simple workaround (disable caching/optimization), and because you usually configure Color module before configuring caching/optimization, demoting to normal.

Devin Carlson’s picture

Do you have any more information on how to reproduce this problem and what environment you're using? I can't reproduce this with any of my colorable themes or combination of caching/aggregation settings.

markhalliwell’s picture

Priority: Normal » Major

JS not working when aggregation/minification is enabled usually means there's a syntax error that gets introduced when the source gets compressed or combined. Bumping this back up to major because (despite what @sun said), disabling caching/optimization on a live site just to configure color changes is absolutely not a viable work around.

markhalliwell’s picture

tagging

nod_’s picture

Can't reproduce. I guess our work on the JS paied off :)

markhalliwell’s picture

Version: 8.x-dev » 7.x-dev
Priority: Major » Normal
Status: Active » Postponed (maintainer needs more info)
Issue tags: -Needs backport to D7
FileSize
150.68 KB
49.11 KB

I also cannot reproduce in 8.x. Moving back to 7.x.
Screen Shot 2013-08-03 at 4.40.34 AM.png

Screen Shot 2013-08-03 at 4.41.51 AM.png