It seems that a recent change to CKEditor 4.x has caused some buttons to be disabled even when specified in the list of toolbar buttons: http://ckeditor.com/forums/CKEditor/CKEditor4-Underline-button-not-appea...

I've noticed this behavior in our module as well. If you enabled the underline button in the admin, it never shows up in the toolbar when creating content.

CommentFileSizeAuthor
#7 ckeditor_remove_buttons-1887174.patch725 bytesquicksketch

Comments

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

I'm 90% certain this is because of the default config.js file, which contains this line:

	// Remove some buttons, provided by the standard plugins, which we don't
	// need to have in the Standard(s) toolbar.
	config.removeButtons = 'Underline,Subscript,Superscript';

:)

quicksketch’s picture

quicksketch
he/him
commented

Oh, yes I meant to include a patch. I'll need to dig it up. We just need to add 'removeButtons' = '' to our function that adds the JS settings.

wwalc’s picture

wwalc commented

Just to clarify this thing to anyone that will accidentally read this issue.

Each build of CKEditor (basic/standard/full) has its own configuration file, where we enable/disable certain features. In the "full preset", the configuration file does not make any changes: full-ckeditor-config.js
on the other side the configuration file in the basic package: basic-ckeditor-config.js does for example the following things:
- defines a single row toolbar, by adding no row separators between toolbarGroups (via config.toolbarGroups)
- removes some buttons (as you noticed)
- sets pasting as plain text as the default method
- removes the "Advanced" tab in the Link dialog to keep the whole editor simple.

izmeez’s picture

izmeez commented

is there a patch to test? Thanks.

mermemladeK’s picture

mermemladeK commented

This is also happening for 7.x

quicksketch’s picture

quicksketch
he/him
commented
Version: 8.x-1.x-dev » 7.x-1.x-dev
Status: Active » Fixed

Fixed in the 7.x version. I'm unpublishing the 8.x version since it's already part of core.

quicksketch’s picture

quicksketch
he/him
commented
StatusFileSize
new ckeditor_remove_buttons-1887174.patch725 bytes

Here's the patch that was committed.

Status: Fixed » Closed (fixed)

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