Opened 10 years ago

Closed 10 years ago

#1411 closed Bug (fixed)

Simple form of combo controls show improper (enabled) state when viewing HTML code

Reported by: Mark Bryson Owned by:
Priority: Normal Milestone: FCKeditor 2.6
Component: UI : Toolbar Version: FCKeditor 2.4.3
Keywords: Review+ Cc:

Description

This is true in both 2.4.3 and 2.5.beta when using IE7 (other browsers not tested). When FCK_TOOLBARITEM_ONLYTEXT is used for anything derived from FCKToolbarSpecialCombo; the state of the control will not be shown in the grayed out disabled state when in Source View mode. (Can be duplicated in 6th example for plugins) I think this is really a flaw in whatever skin is being used. If you find the line (412 if default skin) in the 'fck_editor.css' file of the skin that looks like:

.SC_FieldDisabled .SC_FieldButton, .SC_FieldDisabled .SC_FieldCaption

and change it to:

.SC_FieldDisabled .SC_FieldButton, .SC_FieldDisabled .SC_FieldCaption, .SC_FieldDisabled

That fixes it for me.

Attachments (1)

1411.patch (1.8 KB) - added by Frederico Caldeira Knabben 10 years ago.
Proposal patch

Download all attachments as: .zip

Change History (5)

Changed 10 years ago by Frederico Caldeira Knabben

Attachment: 1411.patch added

Proposal patch

comment:1 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: HasPatch added
Milestone: FCKeditor 2.5FCKeditor 2.6
Version: FCKeditor 2.5 BetaFCKeditor 2.4.3

The proposed fix breaks the combos in normal mode instead. I've attached a patch for it.

As we are working to stabilize the 2.5 Beta, I'm targeting it to the 2.6 to avoid undesired surprises.

comment:2 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review? added; HasPatch removed

comment:3 Changed 10 years ago by Martin Kou

Keywords: Review+ added; Review? removed

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: newclosed

Fixed with [1715]. Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy