Opened 12 years ago

Closed 10 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 12 years ago.
Proposed SVN patch

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by Alfonso Martínez de Lizarrondo

Attachment: 2037.patch added

Proposed SVN patch

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

Keywords: Review? added

comment:2 Changed 12 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 10 years ago by Alfonso Martínez de Lizarrondo

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