We are migrating CKEditor issue tracking to GitHub. Please, use GitHub to report any new issues.

The former tracking system (this website) will still be available in the read-only mode. All issues reported in the past will still be available publicly and can be referenced.

Important: we decided not to transfer all the tickets to GitHub, as many of them are not reproducible anymore or simply no longer requested by the community.
If the issue you are interested in, can be still reproduced in the latest version of CKEditor, feel free to report it again on GitHub.
At the same time please note that issues reported on this website are still taken into consideration when picking up candidates for next milestones.

To report a new issue, go to https://github.com/ckeditor/ckeditor-dev/issues and follow the instructions in the issue template.

Context Navigation

← Previous Ticket
Next Ticket →

Opened 15 years ago

Last modified 15 years ago

#6753 closed Bug

Cleanup core/editor class — at Version 3

Reported by:	Frederico Caldeira Knabben	Owned by:	Garry Yao
Priority:	Normal	Milestone:	CKEditor 3.5.1
Component:	General	Version:	3.3
Keywords:		Cc:

Description (last modified by Frederico Caldeira Knabben)

I can't believe it has been accepted, but plugin and theme dependent code has landed into the CKEDITOR.editor class with [5231].

Not even a single line that has been included there is acceptable. That code must be moved to the proper place.

Change History (4)

Changed 15 years ago by Garry Yao

Attachment:	6753.patch added

comment:1 Changed 15 years ago by Garry Yao

Owner:	set to Garry Yao
Status:	new → review

Assuming you mean [5231].

comment:2 Changed 15 years ago by Tobiasz Cudnik

It look like R+ for me, but the code referenced in contextmenu is missing in current trunk. Does this change anything?

comment:3 Changed 15 years ago by Frederico Caldeira Knabben

Description:	modified (diff)

Note: See TracTickets for help on using tickets.

Download in other formats: