Opened 13 years ago

Closed 12 years ago

#6108 closed defect (fixed)

[meta] IE8 compat

Reported by: alex Owned by: anonymous
Priority: high Milestone: 1.3
Component: General Version: 1.0
Keywords: Cc:
Blocked By: Blocking:

Description

IE 8 beta 1 is out and we need to ensure that Dojo works in it and can take advantage of things like querySelectorAll and DOM Storage.

Change History (7)

comment:1 Changed 13 years ago by alex

Summary: [meta]IE 8 compat → [meta] IE 8 compat

comment:2 Changed 12 years ago by bill

Milestone: 1.1.1 → 1.2
Summary: [meta] IE 8 compat → [meta] IE8 compat

I'm not sure what this task even entails but here are a few things to think about.

Sniffing:

dojo.isIE and also the CSS flags dijit applies to <body> probably need to differentiate between:

  1. IE6 quirks mode
  2. IE6 standards mode
  3. IE7 quirks mode
  4. IE7 standards mode
  5. IE8 quirks mode (compatible w/IE6 quirks mode) (can this just mimic #3/#1? although the fact the we differentiate between #3 and #1 already suggests no)
  6. IE8: IE7 standards mode compatibility mode (can this just mimic #4?)
  7. IE8: standards mode

The last three are the new ones.

CSS hacks:

Currently the dijit themes have some code like #display: foo to only run on IE but that may need to change to be more specific, so it doesn't trigger on IE8.

Tests:

I'd hate to generate more test files but should be testing IE8 in all three modes (quirks mode, IE7 standards compatibility, and IE8 standards mode)

comment:3 Changed 12 years ago by bill

Milestone: 1.2 → 1.3

As per today's dojo meeting, we are predicting IE8 at end of summer and thus this ticket can wait until the 1.3 release. In any case need to wait for IE8 beta2 before we start working on this, because IE8 beta2 will have a lot of JS changes.

comment:4 Changed 12 years ago by bill

(In [13602]) Add .dijitContentBox/.dijitBorderBox flags to <html> in addition to .dj_ie6 etc. Patch from Sam Foster (CLA on file).

Fixes #6685, refs #6108.

comment:5 Changed 12 years ago by Adam Peller

(in [15522]) fixes BorderContainer? and gets themeTester up and running. Un-jankifies bordercontainer. #7950

comment:6 Changed 12 years ago by Adam Peller

this ticket has probably outlived its usefulness. Shall we close and track new issues individually?

comment:7 Changed 12 years ago by Adam Peller

Resolution: → fixed
Status: new → closed
Note: See TracTickets for help on using tickets.