Problem/Motivation

#2091855: CKEditor inserting 'Loading' indicator's HTML into HTML while using Quick Edit uncovered a bug where when using CKEditor for in-place editing, the "Loading" indicator's HTML could end up being stored. Oops. That's being fixed by

Proposed resolution

Move the "Loading" indicator into the toolbar, so that it can never end up being stored.

Remaining tasks

  1. Do it.
  2. Remove the work-arounds introduced in #2091855: CKEditor inserting 'Loading' indicator's HTML into HTML while using Quick Edit — basically just revert that patch.

User interface changes

Design TBD.

API changes

None.

Data model changes

None.

Comments

Wim Leers created an issue. See original summary.

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented
Title: [PP-1] Move the "loading" indicator into the CKEditor toolbar » Move the "loading" indicator into the CKEditor toolbar
Category: Feature request » Task
Issue summary: View changes
Status: Postponed » Active
Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers at Acquia commented
Status: Active » Closed (cannot reproduce)

Ehm, looks like #2091855: CKEditor inserting 'Loading' indicator's HTML into HTML while using Quick Edit was marked as Closed (cannot reproduce), so then this should probably follow suit.