WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 12 days ago

#46771 new defect (bug)

Twenty Eleven: Negative values for padding

Reported by: Malae Owned by:
Milestone: Awaiting Review Priority: normal
Severity: minor Version: 4.9.8
Component: Bundled Theme Keywords: has-patch 2nd-opinion
Focuses: css Cc:

Description

Line 2799
#ie7 article.intro Value Error : padding-left -7.6% negative values are not allowed : -7.6%

Line 2800
#ie7 article.intro Value Error : padding-right -7.6% negative values are not allowed : -7.6%

Attachments (3)

46771.diff (475 bytes) - added by mukesh27 2 years ago.
Patch that remove left and right padding in IE
46771.1.diff (484 bytes) - added by sabernhardt 9 months ago.
positive padding of 7.6%
46771.2.diff (439 bytes) - added by sabernhardt 12 days ago.
remove the two faulty lines

Download all attachments as: .zip

Change History (7)

@mukesh27
2 years ago

Patch that remove left and right padding in IE

#1 @mukesh27
2 years ago

  • Keywords has-patch added; needs-patch removed

#2 @SergeyBiryukov
2 years ago

  • Component changed from Themes to Bundled Theme
  • Summary changed from Twentyeleven theme style.css to Twenty Eleven: Negative values for padding

#3 @sabernhardt
9 months ago

  • Focuses css added
  • Keywords 2nd-opinion added

Thanks for the report and patch!

Because older Internet Explorer versions are hardly used and no longer officially supported, this would be a candidate for closing as won't fix. However, I think the theme's authors intended to have positive padding of 7.6% on the left and right sides.

Other possibilities:

  1. Remove the lines that set padding for IE7 (so it falls back to 8.9%)
  2. Remove all IE7 styles

@sabernhardt
9 months ago

positive padding of 7.6%

@sabernhardt
12 days ago

remove the two faulty lines

#4 @sabernhardt
12 days ago

Correcting the values this late seems quite absurd. So I uploaded another patch for removing those lines.

If we want to remove more, that would belong in another ticket (to address older IE conditional CSS in Twenty Ten to Twenty Seventeen).

Note: See TracTickets for help on using tickets.