#2506581: Remove SafeMarkup::set() from Renderer::doRender broke the ckEditor admin form. Let's fix it.

CommentFileSizeAuthor
#1 2536880.1.patch1.87 KBalexpott
#1 2536880.1.test-only.patch1.18 KBalexpott

Comments

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review
StatusFileSize
new 2536880.1.test-only.patch1.18 KB
new 2536880.1.patch1.87 KB

The last submitted patch, 1: 2536880.1.test-only.patch, failed testing.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Title: ckEditor admin form broken » CKEditor admin form broken
Status: Needs review » Reviewed & tested by the community
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

  • catch committed 3c3692c on 8.0.x 
    Issue #2536880 by alexpott: CKEditor admin form broken

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.