WordPress.org
Get WordPress

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#61955 closed defect (bug) (wontfix)

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

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

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 2 years ago.
Patch for this issue

Download all attachments as: .zip

Change History (3)

@pitamdey
2 years ago

Patch for this issue

#1 @sabernhardt
2 years ago

  • Keywords close 2nd-opinion added
  • Severity normal → trivial
  • Summary Twenty Seventeen : Also define the standard property 'filter' for compatibility → 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
2 years ago

  • Milestone Awaiting Review
  • Resolution → wontfix
  • Status new → closed

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

Note: See TracTickets for help on using tickets.