Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#46488 closed defect (bug) (fixed)

CSS line-height values should be unitless - about.css

Reported by: ianbelanger's profile ianbelanger Owned by: ianbelanger's profile ianbelanger
Milestone: 5.2 Priority: normal
Severity: normal Version:
Component: General Keywords: needs-testing has-patch
Focuses: ui, administration, coding-standards Cc:

Description

As outlined in #44643, CSS line-height values should be unitless unless necessary to be defined as a specific pixel value. It was suggested that we break up 44643 by stylesheet in order to better track them.

This ticket covers wp-admin/css/about.css

Note: The patch was tested only on a Windows machine in Chrome, Firefox, IE11 and Edge. As per https://core.trac.wordpress.org/ticket/44643#comment:23 it should probably be tested on other OS's and supported browsers

Attachments (2)

44643.about.css.diff (650 bytes) - added by ianbelanger 6 years ago.
46488.diff (650 bytes) - added by ianbelanger 6 years ago.
Adjusting patch name to new ticket number

Download all attachments as: .zip

Change History (5)

@ianbelanger
6 years ago

Adjusting patch name to new ticket number

#1 @netweb
6 years ago

  • Keywords good-first-bug removed
  • Milestone changed from Awaiting Review to 5.2

#2 @mukesh27
6 years ago

  • Keywords changed from needs-testing, has-patch to needs-testing has-patch

I tested this successfully without any error in Ubuntu 18.04.2 LTS with Google Chrome Version 73.0.3683.75 and Firefox Version 65.0.1

#3 @SergeyBiryukov
6 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed in [45130].

Note: See TracTickets for help on using tickets.