Skip to content

Commit

Permalink
Add TODO
Browse files Browse the repository at this point in the history
  • Loading branch information
@MartijnCuppens @XhmikosR
MartijnCuppens authored and XhmikosR committed Nov 4, 2018
1 parent a31468c commit 38e9e2b
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions scss/_reboot.scss
View file
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ html {
}

// Shim for "new" HTML5 structural elements to display correctly (IE10, older browsers)
// TODO: remove in v5
// stylelint-disable-next-line selector-list-comma-newline-after
article, aside, figcaption, figure, footer, header, hgroup, main, nav, section {
display: block;
Expand Down

3 comments on commit 38e9e2b

@leeoniya
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MartijnCuppens @XhmikosR

just FYI, if v5 intends to support IE11 this section cannot be removed entirely. normalize v8 actually breaks IE11 (inadvertently).

see necolas/normalize.css#727

it's baffling that this issue has not been addressed in an immediate patch release of Normalize when it was identified back in February 😨

our traffic still shows 6-7% daily IE11 users (US, home improvement industry).

cc @necolas

@mdo
Copy link
Member

@mdo mdo commented on 38e9e2b Nov 4, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While v5 doesn't have a release date or full set of planned changes, we don't plan to support IE11 or IE10 with that release.

@leeoniya
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay, thanks @mdo

Please sign in to comment.