Make WordPress Core

Opened 9 months ago

Last modified 3 weeks ago

#58836 new enhancement

Remove IE specific checks from more default themes

Reported by: sabernhardt's profile sabernhardt Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords:
Focuses: Cc:

Description

Follow-up to #56699

Remove IE conditional statements and register external resources without enqueuing them for four more themes:

  • Twenty Eleven
  • Twenty Twelve
  • Twenty Fourteen
  • Twenty Sixteen

Change History (4)

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


8 months ago

#2 @oglekler
8 months ago

  • Keywords reporter-feedback added; has-patch removed

This ticket was discussed during a bug scrub.

@sabernhardt does there is still work to be done? If so, please clarify the scope, because #56699 is closed and had the same themes mentioned.

Add props to @mukesh27

#3 @sabernhardt
7 months ago

  • Milestone changed from 6.4 to Future Release

#4 @sabernhardt
3 weeks ago

  • Keywords reporter-feedback removed
  • Summary changed from Remove IE specific checks from more default themes (6.4) to Remove IE specific checks from more default themes

#56699 only committed changes to Twenty Thirteen, Twenty Fifteen and Twenty Seventeen.

The remaining pull requests on that ticket seem close to ready, but I still have not created child themes to test them.

  • PR 3474 is probably closer to how Twenty Eleven should be changed than the other PR, though I would prefer to leave the readme.txt and html5.js intact. That can be refreshed now that [57886] is committed. Removing the IE7 styles would also fix #46771.
  • PR 4642 edits Twenty Twelve.
  • PR 4644 edits Twenty Fourteen.
  • PR 4658 edits Twenty Sixteen.
Note: See TracTickets for help on using tickets.