|
In-Portal Issue Tracker - In-Portal CMS
|
|
Viewing Issue Advanced Details |
|
|
ID:
|
Category:
|
Type:
|
Reproducibility:
|
Date Submitted:
|
Last Update:
|
|
1268 |
[In-Portal CMS] Front End |
bug report |
always |
2012-04-26 10:28 |
2012-07-25 05:30 |
|
|
Reporter:
|
erik |
Platform:
|
|
|
|
Assigned To:
|
alex |
OS:
|
|
|
|
Priority:
|
normal |
OS Version:
|
|
|
|
Status:
|
closed |
Product Version:
|
5.2.0-B3 |
|
|
Product Build:
|
|
Resolution:
|
fixed |
|
| |
|
ETA:
|
none |
Fixed in Version:
|
5.2.0-RC1 |
|
|
Reference:
|
https://groups.google.com/d/topic/in-portal-dev/BsLIWUspxso/discussion |
|
Change Log Message:
|
Adds ability to specify CKEditor BODY tag extra css class and id |
|
Estimate Points:
|
2 |
|
|
Summary:
|
0001268: Ease of editing Content Blocks |
|
Description:
|
On the one hand, it is convenient when the HTML-code, is included in the Content Block as simple as possible (do not have a lot of styles, formatting) - then the user is easier to avoid mistakes in the elements styling. On the other hand, it is desirable that when editing a Content Block looks like on the page, that is, that its elements have the same style as on the page.
In order to satisfy both above requirements, I suggest add to ContentBlock tag optional "class" parameter. If this option is specified, then in the CMS-editor window, the value of this parameter is added to the "class" attribute of <body> tag. For example, if class is not specified, <body> tag is the way it is now - <body contenteditable="true" spellcheck="false" class="cke_show_borders">.
If in the template is written <inp2:st_ContentBlock num="1" class="big-font"/>, then the "body"-tag will be the - <body contenteditable="true" spellcheck="false" class="cke_show_borders big-font">
That is, we should not include in the CMS-content unit covering element to apply all related styles in the CMS editor window. And the content of CMS block may contain no "class" attributes, but have adequate and powerful styling applied. |
|
Steps To Reproduce:
|
|
|
Additional Information:
|
As a result tag <inp2:st_ContentBlock will have these additional parameters:
1. body_class - class of BODY tag in CKEditor
2. body_id - id of BODY tag in CKEditor
Additionally there will be ability to specify "extraCss" parameter in CKEditor configuration file (thanks to Drupal modifications of CKEditor at http://drupal.org/project/ckeditor), that allows any custom CKEditor-wide CSS to be applied along with theme-specific external CSS file. |
| Relationships | |
|
Attached Files:
|
 customize_ckeditor_for_every_contentblock.patch (7,935) 2012-04-26 10:29
http://tracker.in-portal.org/file_download.php?file_id=1638&type=bug |
|
|
Issue History |
|
Date Modified |
Username |
Field |
Change |
|
2012-07-25 05:30 |
alex |
Note Added: 0004916 |
|
|
2012-07-25 05:30 |
alex |
Status |
resolved => closed |
|
2012-04-26 10:33 |
alex |
Relationship added |
related to 0001038 |
|
2012-04-26 10:32 |
alex |
Status |
reviewed and tested => resolved |
|
2012-04-26 10:32 |
alex |
Fixed in Version |
=> 5.2.0-RC1 |
|
2012-04-26 10:32 |
alex |
Resolution |
open => fixed |
|
2012-04-26 10:32 |
alex |
Assigned To |
!COMMUNITY => alex |
|
2012-04-26 10:32 |
alex |
Changeset attached |
5.2.x r15325 |
|
2012-04-26 10:32 |
alex |
Note Added: 0004610 |
|
|
2012-04-26 10:31 |
alex |
Note Added: 0004609 |
|
|
2012-04-26 10:31 |
alex |
Status |
needs testing => reviewed and tested |
|
2012-04-26 10:29 |
alex |
Assigned To |
=> !COMMUNITY |
|
2012-04-26 10:29 |
alex |
Developer |
=> alex |
|
2012-04-26 10:29 |
alex |
Status |
active => needs testing |
|
2012-04-26 10:29 |
alex |
File Added: customize_ckeditor_for_every_contentblock.patch |
|
|
2012-04-26 10:29 |
alex |
Estimate Points |
1 => 2 |
|
2012-04-26 10:29 |
alex |
Reporter |
alex => erik |
|
2012-04-26 10:28 |
alex |
New Issue |
|
|
2012-04-26 10:28 |
alex |
Reference |
=> https://groups.google.com/d/topic/in-portal-dev/BsLIWUspxso/discussion |
|
2012-04-26 10:28 |
alex |
Change Log Message |
=> Adds ability to specify CKEditor BODY tag extra css class and id |
|
2012-04-26 10:28 |
alex |
Estimate Points |
=> 1 |