Opened 16 years ago

Closed 14 years ago

#2037 closed New Feature (wontfix)

Additional debug function

Reported by: Alfonso Martínez de Lizarrondo Owned by: Alfonso Martínez de Lizarrondo
Priority: Normal Milestone:
Component: General Version: FCKeditor 2.4.3
Keywords: Review- Cc:

Description

This is a proposal to add a new method in the FCKDebug object to send debug messages to the error console (without an additional window, it's specially useful while debugging in chrome)

Attachments (1)

2037.patch (1.0 KB) - added by Alfonso Martínez de Lizarrondo 16 years ago.
Proposed SVN patch

Download all attachments as: .zip

Change History (4)

Changed 16 years ago by Alfonso Martínez de Lizarrondo

Attachment: 2037.patch added

Proposed SVN patch

comment:1 Changed 16 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review? added

comment:2 Changed 16 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

While the proposed idea is good, I don't feel that including further debugging stuff in the code is a good thing. Not because of the debugging benefits, of course, but because we don't need it normally.

Actually, we would do better if we would completely avoid the debug code reaching the compressed files. We should instead load the fckdebug.js separately, if debugging is enabled. I've opened #2043, so this ticket can be considered dependent on that.

comment:3 Changed 14 years ago by Alfonso Martínez de Lizarrondo

Resolution: wontfix
Status: newclosed
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