As per #308865: Drop IE6 support in Drupal core, specifically http://drupal.org/node/308865#comment-4636296.

We can remove ie6.css. Thereafter we can start refactoring the CSS without the need to support IE6 - which will be a series of separate issues.

CommentFileSizeAuthor
#5 1196170-full.patch1.23 KBcosmicdreams
#1 1196170-remove-ie6.patch493 bytescosmicdreams
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cosmicdreams’s picture

cosmicdreams CreditAttribution: cosmicdreams commented
FileSize
1196170-remove-ie6.patch493 bytes

This patch only removes the ie6.css file from Bartik. I guess other issues will handle the refactoring of the rest of the css.

cosmicdreams’s picture

cosmicdreams CreditAttribution: cosmicdreams commented
Status: Active » Needs review

informing the testbot

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Needs review » Needs work

As in the other issue, this code gets loaded somewhere

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: Jeff Burnz commented

in template.php in bartik_preprocess_html()

cosmicdreams’s picture

cosmicdreams CreditAttribution: cosmicdreams commented
FileSize
1196170-full.patch1.23 KB

Thank you Jeff, tracked down the line aspilicious refers to in the file you referred to.

cosmicdreams’s picture

cosmicdreams CreditAttribution: cosmicdreams commented
Status: Needs work » Needs review

triggering the testbot again.

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Needs review » Needs work 
+++ b/themes/bartik/template.phpundefined
@@ -22,8 +22,7 @@ function bartik_preprocess_html(&$variables) {
+  drupal_add_css(path_to_theme() . '/css/ie.css', array('group' => CSS_THEME, 'browsers' => array('IE' => 'lte IE 7', '!IE' => FALSE), 'preprocess' => FALSE));  ¶

Traling whitespace

Check also the other bartik files, maybe there is something left somewhere.

Powered by Dreditor.

aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Needs work » Closed (duplicate)

Srry going to close this because we are fixing the same thing in several places.

#308865: Drop IE6 support in Drupal core