WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#26898 closed defect (bug) (fixed)

Typo in wp-admin.css ".widgets-chooser li" transition

Reported by: ciantic Owned by: SergeyBiryukov
Milestone: 3.9 Priority: normal
Severity: normal Version: 3.8
Component: Widgets Keywords: has-patch
Focuses: ui Cc:
PR Number:

Description

In wp-admin/css/wp-admin.css (line 11209) reads followingly:

	.widgets-chooser li {
			...
			transition: background: 0.2s ease-in-out;
	}

I suspect this extra colon is typo, it should be:

	.widgets-chooser li {
			...
			transition: background 0.2s ease-in-out;
	}

However it's missing completely the browser specific syntaxes for transition, so perhaps those should be added also.

Attachments (1)

26898.diff (407 bytes) - added by MikeHansenMe 6 years ago.
remove extra colon

Download all attachments as: .zip

Change History (4)

#1 @MikeHansenMe
6 years ago

  • Keywords has-patch added

It looks like all current browsers support this without prefix. Looking at http://caniuse.com/#feat=css-transitions it shows that the prefix would only work on mobile browsers. Most of the desktop browsers went from not supporting to support without the prefix between.

@MikeHansenMe
6 years ago

remove extra colon

#2 @SergeyBiryukov
6 years ago

  • Component changed from Administration to Widgets
  • Keywords ui-focus added
  • Milestone changed from Awaiting Review to 3.9
  • Version changed from trunk to 3.8

Introduced in [26271].

#3 @SergeyBiryukov
6 years ago

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

In 27020:

Fix typo in CSS. props ciantic, MikeHansenMe. fixes #26898.

Note: See TracTickets for help on using tickets.