WordPress.org
Get WordPress

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#40893 closed defect (bug) (fixed)

TinyMCE: prevent warnings about missing stylesheets

Reported by: azaozz's profile azaozz Owned by: azaozz's profile azaozz
Milestone: 4.8 Priority: normal
Severity: normal Version: 4.8
Component: TinyMCE Keywords: has-patch commit
Focuses: Cc:

Description

It appears that some themes still try to load fonts from Google where the URLs have unencoded commas. That breaks this functionality in TinyMCE as it splits multiple URLs on commas and tries to load the fragments.

The difference is that the latest TinyMCE shows warnings in the UI when a stylesheet fails to load.

Attachments (2)

40893.patch​ (621 bytes) - added by azaozz 8 years ago.
warning.png​ (3.7 KB) - added by ocean90 8 years ago.
Screenshot of the warning, props @azaozz

Download all attachments as: .zip

Change History (6)

@azaozz
8 years ago

#1 @azaozz
8 years ago

In 40893.patch: force the urlencoding of commas in the URLs that are passed to the content_css TinyMCE setting.

This ticket was mentioned in ​Slack in #core by jeffpaul. ​View the logs.
8 years ago

@ocean90
8 years ago

Screenshot of the warning, props @azaozz

#3 @ocean90
8 years ago

  • Keywords has-patch commit added

+1 for 40893.patch​, after changing $editor_styles[$key] to $editor_styles[ $key ] and adding a full stop to the comment.

#4 @azaozz
8 years ago

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

In 40862:

TinyMCE: Force urlencoding of commas in URLs added by plugins to prevent warnings about missing stylesheets.

Fixes #40893.

Note: See TracTickets for help on using tickets.