Make WordPress Core

Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#28703 closed defect (bug) (fixed)

Table class in Twenty Fourteen editor-style.css need to be updated

Reported by: slobodanmanic's profile slobodanmanic Owned by: lancewillett's profile lancewillett
Milestone: 4.1 Priority: normal
Severity: normal Version: 3.9
Component: Bundled Theme Keywords: has-patch
Focuses: administration Cc:

Description

I guess this happened in 3.9, when TinyMCE was updated, but can't find it in TinyMCE changelog.

.mceItemTable is not being generated any more and is replaced by .mce-item-table

If they changed the format it would be good to also update .mceContentBody class to .mce-content-body, currently they're both used.

Might be best not to use this class at all, but when table is used instead, table borders get messed up, so I'm guessing it was added to override that.

Attachments (2)

28703.diff (826 bytes) - added by slobodanmanic 9 years ago.
28703-1.diff (870 bytes) - added by slobodanmanic 8 years ago.

Download all attachments as: .zip

Change History (9)

@slobodanmanic
9 years ago

#1 @slobodanmanic
9 years ago

Patch replaces the class used for tables.

#2 @chriseverson
9 years ago

  • Keywords has-patch added

#3 @DrewAPicture
8 years ago

  • Component changed from General to Bundled Theme

#4 follow-up: @SergeyBiryukov
8 years ago

Should we keep the old class for backwards compatibility with 3.8.x?

#5 in reply to: ↑ 4 @slobodanmanic
8 years ago

Replying to SergeyBiryukov:

Should we keep the old class for backwards compatibility with 3.8.x?

Good point. I'll update new patch in about an hour.

#6 @lancewillett
8 years ago

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

In 30147:

Twenty Fourteen: update table styles in editor CSS file for new TinyMCE class names.

Props slobodanmanic, fixes #28703.

#7 @lancewillett
8 years ago

  • Milestone changed from Awaiting Review to 4.1
Note: See TracTickets for help on using tickets.