Bugzilla – Attachment 5459 Details for
Bug 6884
Improve TinyMCE configuration on Koha News page
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Proposed fix
0001-Fix-for-Bug-6884-Improve-TinyMCE-configuration-on-Ko.patch (text/plain), 4.55 KB, created by
Owen Leonard
on 2011-09-19 13:11:11 UTC
(
hide
)
Description:
Proposed fix
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2011-09-19 13:11:11 UTC
Size:
4.55 KB
patch
obsolete
>From 9df33422144a5cbad5da45b1a6a7ad00830376db Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Fri, 16 Sep 2011 10:27:24 -0400 >Subject: [PATCH] Fix for Bug 6884, Improve TinyMCE configuration on Koha News page >Content-Type: text/plain; charset="utf-8" > >- Increases the width of the editor to fill the space provided >- Adjusts the layout of the toolbars to better fit the space >- Adds a custom editor stylesheet to make the contents of the > editor better match the expected output > >In changing the toolbar configuration I chose to make the display >of each button explicit rather than assuming a default set of >buttons and adding or subtracting from that. I think this is >clearer for someone approaching the configuration trying to under- >stand how buttons are displayed. I also chose to use the >shorthand "|" instead of "separator" for brevity. This last change >is the only one made to the edithelp configuration. >--- > koha-tmpl/intranet-tmpl/prog/en/css/tinymce.css | 4 ++++ > .../intranet-tmpl/prog/en/modules/help/edithelp.tt | 9 +++++---- > .../prog/en/modules/tools/koha-news.tt | 16 +++++++++------- > 3 files changed, 18 insertions(+), 11 deletions(-) > create mode 100644 koha-tmpl/intranet-tmpl/prog/en/css/tinymce.css > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/css/tinymce.css b/koha-tmpl/intranet-tmpl/prog/en/css/tinymce.css >new file mode 100644 >index 0000000..57fec75 >--- /dev/null >+++ b/koha-tmpl/intranet-tmpl/prog/en/css/tinymce.css >@@ -0,0 +1,4 @@ >+body, td, pre { >+ font-family: Arial,Helvetica,sans-serif; >+ font-size: 13px; >+} >\ No newline at end of file >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/help/edithelp.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/help/edithelp.tt >index 3a40f2d..ad4b4a1 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/help/edithelp.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/help/edithelp.tt >@@ -5,12 +5,13 @@ > tinyMCE.init({ > mode : "textareas", > theme : "advanced", >+ content_css : "[% themelang %]/css/tinymce.css", > plugins : "table,save,advhr,advlink,iespell,searchreplace,print,contextmenu", > theme_advanced_disable : "underline,strikethrough,styleselect,image", >- theme_advanced_buttons1_add_before : "save,separator", >- theme_advanced_buttons2_add_before: "cut,copy,paste,separator,search,replace,separator", >- theme_advanced_buttons3_add_before : "tablecontrols,separator", >- theme_advanced_buttons3_add : "iespell,advhr,separator,print", >+ theme_advanced_buttons1_add_before : "save,|", >+ theme_advanced_buttons2_add_before: "cut,copy,paste,|,search,replace,|", >+ theme_advanced_buttons3_add_before : "tablecontrols,|", >+ theme_advanced_buttons3_add : "iespell,advhr,|,print", > theme_advanced_toolbar_location : "top", > theme_advanced_toolbar_align : "left", > theme_advanced_path_location : "bottom", >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt >index 313701b..dfee834 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/koha-news.tt >@@ -17,19 +17,21 @@ $(document).ready(function() { > <script language="javascript" type="text/javascript"> > tinyMCE.init({ > mode : "textareas", >- theme : "advanced", >+ theme : "advanced", >+ content_css : "[% themelang %]/css/tinymce.css", > plugins : "table,save,advhr,advlink,iespell,searchreplace,print,contextmenu", >- theme_advanced_disable : "underline,strikethrough,styleselect,image", >- theme_advanced_buttons1_add_before : "save,separator", >- theme_advanced_buttons2_add_before: "cut,copy,paste,separator,search,replace,separator", >- theme_advanced_buttons3_add_before : "tablecontrols,separator", >- theme_advanced_buttons3_add : "iespell,advhr,separator,print", >+ theme_advanced_buttons1 : "save,|,bold,italic,|,cut,copy,paste,|,search,replace,|,justifyleft,justifycenter,justifyright,justifyfull,|,formatselect,|,link,unlink,anchor,cleanup,help,code,advhr,|,print", >+ theme_advanced_buttons2 : "tablecontrols,|,bullist,numlist,|,outdent,indent,|,undo,redo,|,removeformat,|,visualaid,|,sub,sup,|,charmap", >+ theme_advanced_buttons3 : "", > theme_advanced_toolbar_location : "top", > theme_advanced_toolbar_align : "left", > theme_advanced_path_location : "bottom", >+ theme_advanced_resizing : true, > plugin_insertdate_dateFormat : "%Y-%m-%d", > plugin_insertdate_timeFormat : "%H:%M:%S", >- apply_source_formatting : true >+ apply_source_formatting : true, >+ height : "300", >+ width : "700" > }); > </script> > </head> >-- >1.7.3 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 6884
:
5458
|
5459
|
5470