WordPress.org
Get WordPress

Make WordPress Core

Opened 9 days ago

Closed 8 days ago

#61955 closed defect (bug) (wontfix)

Twenty Seventeen: Also define the standard property 'filter' for compatibility

Reported by: pitamdey's profile pitamdey Owned by:
Milestone: Priority: normal
Severity: trivial Version:
Component: Bundled Theme Keywords: has-patch close 2nd-opinion
Focuses: css Cc:

Description

In Twenty Seventeen, There is a lint error in css file Also define the standard property 'filter' for compatibility

I have added the patch

Attachments (1)

61955.patch​ (975 bytes) - added by pitamdey 9 days ago.
Patch for this issue

Download all attachments as: .zip

Change History (3)

@pitamdey
9 days ago

Patch for this issue

#1 @sabernhardt
9 days ago

  • Keywords close 2nd-opinion added
  • Severity changed from normal to trivial
  • Summary changed from Twenty Seventeen : Also define the standard property 'filter' for compatibility to Twenty Seventeen: Also define the standard property 'filter' for compatibility

I do not think that the ie8.css file should be updated or checked anymore. It was only used for old versions of Internet Explorer, and #56699 stopped enqueuing it. The CSS simply remained available in case a child theme (or plugin) enqueues it.

That said, the patch adds only two lines to a file that is mostly unused.

The -ms-filter rules were added in ​PR 405.

#2 @karmatosed
8 days ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

I agree with closing this so going to follow through as recommended.

Note: See TracTickets for help on using tickets.