Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#6166 closed Bug (fixed)

Error on maximise command when toolbar button not shown

Reported by: William Owned by:
Priority: Normal Milestone: CKEditor 3.5.1
Component: UI : Toolbar Version: 3.3.2
Keywords: HasPatch Cc:

Description

Hi,

The maximise plugin has an error, on line 291, when the editor is set to be automaximised on start and the user has opted to not show the maximise button on the toolbar.

_source\plugins\maximize\plugin.js

a possible fix is (adding the lines with +):

+if(button!=null)
  var buttonNode = editor.element.getDocument().getById( button._.id );
  buttonNode.getChild( 1 ).setHtml( label );
  buttonNode.setAttribute( 'title', label );
  buttonNode.setAttribute( 'href', 'javascript:void("' + label + '");' );
+}

I checked nightly build 5826 and problem was evident there.

A thread about the bug. http://cksource.com/forums/viewtopic.php?f=11&t=19820

Attachments (1)

6166.patch (1.2 KB) - added by Charlie 14 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 14 years ago by Charlie

Keywords: HasPatch added; maximise removed
Summary: maximise of start (no maximise button casuses error)Error on maximise command when toolbar button not shown
Version: 3.3.23.4.1 (SVN - trunk)

I changed the title to be a little more descriptive and generic. Hope that's alright! Also, here's his patch in true patch file form.

Changed 14 years ago by Charlie

Attachment: 6166.patch added

comment:2 Changed 14 years ago by William

Thankyou comp615

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

Status: newconfirmed
Version: 3.4.1 (SVN - trunk)3.3.2

comment:4 Changed 13 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.5.1

comment:5 Changed 13 years ago by Sa'ar Zac Elias

Resolution: fixed
Status: confirmedclosed

Thanks @comp615; Committed as a microchange with [6253].

comment:6 Changed 13 years ago by Wiktor Walc

#5329 marked as duplicate

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy