#411 ✓resolved

Bug: CKeditor contentCSS makes too many assumptions

Reported by dleffler | November 23rd, 2011 @ 06:25 PM | in 2.0.4 (closed)

Instead of just loading 'base-styles.css' from the theme folder, we should probably load all the styles from the current theme?

Comments and changes to this ticket

  • dleffler

    dleffler January 2nd, 2012 @ 02:11 AM

    • Assigned user set to “dleffler”

    Problem #1 is we were using 'contentCSS' instead of 'contentCss' which is the biggest hurdle since only the 2nd one is valid.
    Problem #2 is we use different class's/id's across different themes which causes most themes to make the editor appear with black text on a dark background. Even when you can set the class/id for the content it looks kinda odd in the editor since it looks like a page entry, not an item in a column.
    Problem #3 is that most styling causes the ckeditor drop-downs to pick up a dark background unless we place a .cke_panel_frame {background:white;} entry in a theme css file.

  • dleffler

    dleffler January 2nd, 2012 @ 03:53 PM

    Perhaps the best overall solution is to create a ckeditor.css file in a new folder in the theme which will 1) give the editor the view the theme designer intended and 2) automatically create the drop-down style menu based on ckeditor.css parsing.

  • dleffler

    dleffler January 3rd, 2012 @ 08:11 PM

    • Milestone set to 2.0.4
    • Milestone order changed from “124” to “0”

    Based on irc conversations:

    • we'll (re)add the 'editor' folder into the theme folder which will hold the 'ckeditor.css' style sheet.
    • therefore, an additional task will be to create a 'ckeditor.css' style sheet (example?) for each of our shipped themes
  • expNinja

    expNinja January 5th, 2012 @ 09:33 PM

    (from [8ebaa0204d8c0e26358887ef1d347838d5e0f3e7]) Adds CKEditor custom style, paragraph format, & font name dropdown lists. Also provides a better implementation of displaying the theme style inside the editor itself, inlcuding the auto-additon of theme based styles parsed fro the editor stylesheet. simpletheme and simpletheme Fall editor stylesheets included. still need editor stylesheets for other included themes. [#411 #412] https://github.com/exponentcms/exponent-cms/commit/8ebaa0204d8c0e26...

  • dleffler

    dleffler January 6th, 2012 @ 02:42 AM

    • State changed from “new” to “resolved”

    As such, this one is closed, though we could probably tweak each of the theme editor css files?

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Bug Tracker for Exponent CMS

Shared Ticket Bins

People watching this ticket

Referenced by