Problem/Motivation
#2398463: Clean up the "content" component in Bartik removed core/themes/bartik/css/components/content.css
. But bartik.info.yml
's ckeditor_stylesheets
section is still referring to it. Which causes CKEditor to try to load this file, resulting in a 404.
Steps to reproduce
- Create a node.
- Go to that node's full page
- Ensure the comment form is loaded
- Look in the browser's developer console, you'll see a 404.
Proposed resolution
Fix it, by stopping to tell CKEditor to load this file. AFAICT the only selectors left in content.css
were already irrelevant to any HTML in CKEditor, so its contents must've changed significantly since #2064379: Remove ckeditor-iframe.css and load relevant Bartik CSS files for CKEditor's iframe mode. added that file to be loaded.
Remaining tasks
Review.
User interface changes
None.
API changes
None.
Data model changes
None.
Comment | File | Size | Author |
---|---|---|---|
#5 | follow_up_for_2398463-2552175-5.patch | 1.31 KB | nlisgo |
#5 | interdiff-2552175-2-5.txt | 939 bytes | nlisgo |
#2 | 2552175-2.patch | 473 bytes | Wim Leers |
Comments
Comment #2
Wim LeersComment #3
Wim LeersComment #5
nlisgo CreditAttribution: nlisgo commentedComment #6
Wim LeersThanks, @nlisgo!
Comment #7
stefan.r CreditAttribution: stefan.r commentedManually confirmed this gets rid of the 404. All references to this file seem to have been removed, so RTBC!
Comment #9
willzyx CreditAttribution: willzyx commentedAnother case of #2552687: Test failures in ConfigFormOverrideTest and ContainerRebuildWebTest on newly spun up testbot instances back to RTBC
Comment #11
alexpottIs this the right fix? Shouldn't we be loading some of the files added by #2398463: Clean up the "content" component in Bartik
Comment #12
LewisNymanIt doesn't look like it. All the styling in the old content.css file was themed output. None of it was base element styling.
Comment #13
alexpottCommitted a3542ee and pushed to 8.0.x. Thanks!
Perhaps the intention was to have things like this apply. It would not have worked though. I guess another issue should explore what the ckeditor styles for bartik should be.
Comment #16
willzyx CreditAttribution: willzyx commented