Make WordPress Core

Opened 2 months ago

Last modified 3 weeks ago

#53795 reopened defect (bug)

WP5.8 CSS Validation errors

Reported by: jim5471 Owned by:
Milestone: 5.8.2 Priority: normal
Severity: normal Version:
Component: Editor Keywords: has-patch
Focuses: Cc:

Description (last modified by SergeyBiryukov)




I created a ticket in the Gutenberg repository on 3 June and the issue hasn't been actioned.

So many people work very hard on WordPress development and to release software that hasn't been fully validated is difficult to understand.

Some part of WordPress will fail to function correctly and more people will waste effort trying to discover why.

Am I wrong to report this ?

Change History (4)

#1 @SergeyBiryukov
2 months ago

  • Component changed from General to Editor
  • Description modified (diff)
  • Milestone Awaiting Review deleted
  • Resolution set to reported-upstream
  • Status changed from new to closed

Hi there, welcome back to WordPress Trac! Thanks for the report.

It looks like you have already reported this at https://github.com/WordPress/gutenberg/issues/32434 in the Gutenberg repository, which is the correct place for this issue.

I'm closing the ticket as reported-upstream, as the fix needs to made upstream in the Gutenberg plugin first. When the changes are made, they will be backported to WordPress core as part of package updates.

#2 @SergeyBiryukov
8 weeks ago

  • Milestone set to 5.8.1
  • Resolution reported-upstream deleted
  • Status changed from closed to reopened

On second thought, reopening for more visibility, and to see if this can be prioritized for 5.8.1.

#3 @peterwilsoncc
3 weeks ago

There is a pull request in progress for this on the Gutenberg repository.

The reported error on line 267 (selector list in :not()) is a false positive as the validator is for CSS 3 and :not() began supporting selector lists in CSS Level 4. All browsers supported by WordPress support the feature. https://caniuse.com/css-not-sel-list

The reported error on line 2283 is a genuine positive: width: none is not valid.

#4 @desrosj
3 weeks ago

  • Keywords has-patch added
  • Milestone changed from 5.8.1 to 5.8.2

With 5.8.1 RC in less than 24 hours, I'm going to punt this one to 5.8.2. I've commented on the PR, and if it can be refined before tomorrow it can be reconsidered for 5.8.1.

Note: See TracTickets for help on using tickets.