WordPress.org

Make WordPress Core

Opened 14 months ago

Closed 13 months ago

Last modified 13 months ago

#20470 closed defect (bug) (fixed)

Twenty Eleven: Hide footer separator in IE8

Reported by: iandstewart Owned by: nacin
Priority: normal Milestone: 3.4
Component: Bundled Theme Version: 3.4
Severity: normal Keywords: has-patch commit
Cc:

Description

Twenty Eleven is letting it's footer text separator just hang out there in IE8.

https://img.skitch.com/20120417-c6dceben3wcw779x5iat3ctyyk.jpg

Attachments (3)

bye-sep-20495.diff (400 bytes) - added by iandstewart 14 months ago.
20470.patch (541 bytes) - added by SergeyBiryukov 14 months ago.
20470.sep.png (2.6 KB) - added by SergeyBiryukov 14 months ago.

Download all attachments as: .zip

Change History (9)

iandstewart14 months ago

SergeyBiryukov14 months ago

comment:1 SergeyBiryukov14 months ago

This only affects WordPress.com version of Twenty Eleven.

#site-generator .sep was removed from the bundled version in [17954]. Perhaps the styling should be removed as well (20470.patch).

SergeyBiryukov14 months ago

comment:2 SergeyBiryukov14 months ago

FWIW, bye-sep-20495.diff doesn't hide the separator in IE 8 (20470.sep.png is the screenshot of Twenty Eleven demo page).

comment:3 nacin13 months ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.4

comment:4 nacin13 months ago

  • Milestone changed from 3.4 to Awaiting Review

This leaves a leftover and unused images/wordpress.png. In Twenty Ten, we did include the icon. Don't really care what we do here, but moving it out of 3.4 until there's some kind of consensus.

comment:5 nacin13 months ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In [20907]:

Twenty Eleven: Remove unused CSS. props SergeyBiryukov. fixes #20470.

comment:6 SergeyBiryukov13 months ago

  • Milestone changed from Awaiting Review to 3.4
Note: See TracTickets for help on using tickets.