WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#26563 closed defect (bug) (fixed)

min-width property without value in ie.css

Reported by: tobiasbg's profile TobiasBg Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 3.9 Priority: lowest
Severity: trivial Version: 3.8
Component: Administration Keywords: has-patch commit
Focuses: Cc:

Description

[26819] (for #26369) added a min-width: property without a property value to ie.css:
​http://core.trac.wordpress.org/browser/trunk/src/wp-admin/css/ie.css?rev=26819#L215

As per ​https://irclogs.wordpress.org/chanlog.php?channel=wordpress-dev&day=2013-12-11&sort=asc#m752053 it's not critical and can be fixed in 3.8.1.

Attachments (1)

26563.diff​ (327 bytes) - added by jorbin 11 years ago.

Download all attachments as: .zip

Change History (8)

#1 @SergeyBiryukov
11 years ago

  • Milestone changed from Awaiting Review to 3.8.1
  • Priority changed from normal to lowest
  • Severity changed from normal to trivial

@jorbin
11 years ago

#2 @jorbin
11 years ago

  • Keywords has-patch added

#3 @SergeyBiryukov
11 years ago

I've checked the Themes screen in IE7 to see if it actually needs some min-width value, and looks like it doesn't.

#4 @SergeyBiryukov
11 years ago

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

In 26886:

Remove invalid min-width property.

props jorbin.
fixes #26563 for trunk.

#5 @SergeyBiryukov
11 years ago

  • Keywords commit fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

#6 @nacin
11 years ago

Seems like we can fix this in just 3.9, and save the change to ie.css in 3.8.1. It still works fine, right?

#7 @SergeyBiryukov
11 years ago

  • Keywords fixed-major removed
  • Milestone changed from 3.8.1 to 3.9
  • Resolution set to fixed
  • Status changed from reopened to closed

Yes, still works fine.

Note: See TracTickets for help on using tickets.