This theme has a number of rendering differences between Firefox and IE.

Noted instances:

- Differences in paragraph spacing - a blank line appears between paragraphs in FF and no space appears between paragraphs in IE. This issue is SIGNIFICANT.

- The dots to the left of the navigation menu are center vert aligned in FF and top vert aligned in IE. There is also a space between the dot and the associated navigation menu item in FF and no space in IE.

OS: Win XP SP3

IE version 6.0.2900.5512
FF version 3.06

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Closed (won't fix)

Thanks for taking the time to look at this.

No further support is planned for Internet Explorer 6. I will consider community contributed patches for IE6 that are vetted by other community members, but I will not be doing any development for IE 6.

DrDrupal’s picture

DrDrupal commented

I think that decision is a mistake. I have checked numerous sources and most agree that IE6 has a market share as large or larger than FireFox:

http://www.thecounter.com/stats/2009/January/browser.php

That link shows January 2009 stats of

IE7 42%
IE6 34%
FF 17%

Other sites have different numbers, but they all agree that IE6 is still a prominent browser. BTW, my company serves well as another example. We have over 100,000 employees...all with IE6 on their desktops. Sure, we have test machines with a range of browsers, but IE6 is the standard for our corporation. The corporate world is not so quick on the upgrade path...and Sophos prevents anyone in my company from doing it themselves.

It's your theme. You get to make the call...but this is low-hanging fruit to benefit one third of browser users...not some fringe browser...and ignoring it is a mistake IMHO.

BTW, the fix for the paragraph spacing issue appears to be:

- open html.css
- find the class that looks like this:

html, body, div, span, applet, object, iframe, h1, h2, h3, h4, h5, h6,
p, blockquote, pre, a, abbr, acronym, address, big, cite, code, del,
dfn, em, font, img, ins, kbd, q, s, samp, small, strike, strong, sub,
sup, tt, var, b, u, i, center, fieldset, form, label, legend, table,
caption, tbody, tfoot, thead, tr, th, td {
margin: 0;
padding: 0;
}

- Delete the "p" in the list of HTML elements to which it (esp. margin: 0:) applies.

I'm not sure what other effects that change may have, but I have seen nothing else negative on my site so far.

There are a number of other IE6 warts, but this is the biggest one.

Thanks for listening.

Anonymous’s picture

Anonymous (not verified) commented

Thanks for your feedback. I fully understand there is still a sizable percentage of IE 6 users, but it's not going to go away until people stop supporting it. This is the approach that the government of Norway has taken. I develop this theme in my spare time, and dealing with IE 6's many warts is not how I want to spend my spare time.

However, I am willing to accommodate this in the following way:

  • Add fix-ie6.css and fix-ie6-rtl.css files to the theme.
  • Add conditional IE 6 imports for those files in the phptemplate_get_ie_styles() function.
  • Integrate community submitted and verified patches to those files.

But I will not be developing and testing for IE 6 myself.

mokko’s picture

mokko commented
Version: 6.x-2.x-dev » 6.x-3.4
Status: Closed (won't fix) » Needs review

Again: I am really no expert in CSS, so don't trust me blindly.

I introduced the following to minimalize the problem with paragraph spacing in the IE6. So far I see no bad side issues.

/* $Id: fix-ie6.css,v 1.1 2009/02/28 16:20:09 shannonlucas Exp $
* @file fix-ie6.css
* Fixes for Internet Explorer 6
*/

p {
margin-bottom:0.8em;
}

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Closed (fixed)

Report IE issues in new bug report.