Opened 8 years ago

Last modified 5 years ago

#6875 confirmed Bug

CKReleaser should properly compress filters in CSS files

Reported by: Wiktor Walc Owned by:
Priority: Normal Milestone:
Component: Project : CKBuilder Version: 4.0
Keywords: Cc:

Description

IE needs a comma after each filter. The following does not work:

filter:progid:DXImageTransform.Microsoft.AlphaImageLoader(src='icons.png',sizingMethod='scale'),alpha(opacity=30);}

This one is ok (note: space after comma before alpha):

filter:progid:DXImageTransform.Microsoft.AlphaImageLoader(src='icons.png',sizingMethod='scale'), alpha(opacity=30);

Dirt hack comitted with [6231]. Ideally it should be fixed in yuicompressor.

Related issues:

  • if we move two filters into two separate CSS rules, #4821 will be back.
  • if there is no space before alpha(), #6857 appears in release version of CKEditor 3.5

Attachments (1)

kama_ie8_zoom.png (17.6 KB) - added by Wiktor Walc 5 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 Changed 8 years ago by Wiktor Walc

Status: newconfirmed

Changed 5 years ago by Wiktor Walc

Attachment: kama_ie8_zoom.png added

comment:2 Changed 5 years ago by Wiktor Walc

Component: Project : CKReleaserProject : CKBuilder
Version: 4.0

Ops, it looks like this hack should be ported to CKBuilder. Kama skin in IE8 with zoom enabled looks bad again:

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy