Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#43195 closed enhancement (fixed)

dashicons fail w3c css validation

Reported by: manuel_84's profile manuel_84 Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.1 Priority: normal
Severity: normal Version: 4.9.2
Component: Administration Keywords: has-patch
Focuses: ui, coding-standards Cc:

Description

/wp-includes/css/dashicons.min.css?ver=4.9.2

line 2: .dashicons, .dashicons-before:before

0 is not a value for transition : color 0.1s ease-in 0

Attachments (1)

43195.diff (427 bytes) - added by chetan200891 7 years ago.
Created patch to drop the transition-delay completely from dashicons.css

Download all attachments as: .zip

Change History (14)

#1 @manuel_84
7 years ago

fix by adding a "s" after the final 0:

transition : color 0.1s ease-in 0s

#2 @GaryJ
7 years ago

Thank you @manuel_84 for opening your first ticket :-)

Good catch with the bug.

An alternative solution would be to drop the transition-delay completely since the default is 0s anyway:

transition: color 0.1s ease-in;

@chetan200891
7 years ago

Created patch to drop the transition-delay completely from dashicons.css

#3 @SergeyBiryukov
7 years ago

  • Component changed from Bundled Theme to Administration
  • Focuses template removed
  • Milestone changed from Awaiting Review to 5.0

#4 @chetan200891
7 years ago

  • Keywords has-patch added

#5 @SergeyBiryukov
7 years ago

Per https://developer.mozilla.org/en-US/docs/Web/CSS/time:

Although unitless zero is allowed for <length>s, it's invalid for <time>s.

#6 @SergeyBiryukov
7 years ago

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

In 42689:

Administration: Remove erroneous transition-delay values from CSS files.

Although unitless zero is allowed for <length> CSS data type, it's invalid for <time> data type.

Props chetan200891, manuel_84, GaryJ.
Fixes #43195.

#7 @SergeyBiryukov
7 years ago

Created a GitHub issue for genericons.css: https://github.com/Automattic/Genericons/issues/109, since it's bundled with Twenty Fifteen and Twenty Sixteen.

#8 @SergeyBiryukov
7 years ago

#43769 was marked as a duplicate.

#9 @pento
6 years ago

  • Milestone changed from 5.0 to 5.1

#10 @desrosj
6 years ago

#45559 was marked as a duplicate.

#11 @swissspidy
6 years ago

#46182 was marked as a duplicate.

This ticket was mentioned in Slack in #core by malae. View the logs.


6 years ago

#13 @desrosj
6 years ago

In 45097:

Administration: Remove unnecessary transition-delay values from Dashicons.

A unitless 0 value for a <time> CSS data type was reintroduced in [45040]. This is allowed for <length> data types, but is invalid for <time>.

See #43195.
Fixes #46774.

Note: See TracTickets for help on using tickets.