Opened 13 years ago
Last modified 12 years ago
#9227 confirmed Bug
OnTextChanged event required for CKEditor ASP.NET control doesn't fire
| Reported by: | Koen Van Exem | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | Server : ASP.Net | Version: | 3.6.4 |
| Keywords: | HasPatch | Cc: |
Description
The event is fired in OnPreRender(). This is not the proper place to fire this event.
It should be fired in IPostBackDataHandler.RaisePostDataChangedEvent().
In attachment you can find a pach
Attachments (3)
Change History (7)
Changed 13 years ago by
| Attachment: | CKEditorControl.cs.patch added |
|---|
comment:1 follow-up: 2 Changed 13 years ago by
| Keywords: | OnTextChanged removed |
|---|---|
| Status: | new → pending |
comment:2 Changed 13 years ago by
Replying to j.swiderski:
Hi KoenVanExem,
Could you explain why is it important to move this event?
What sort of the problem has current code been causing for you?
When I used the editor control in a dynamic control the event didn't fire. (using new Control() instead of just dropping it on the design surface)
Using the IPostBackDataHandler is the standard way of doing it, and will make your control more robust (e.g. dynamic, partial postback, ...)
comment:3 Changed 13 years ago by
| Keywords: | HasPatch added |
|---|---|
| Status: | pending → confirmed |
Changed 12 years ago by
| Attachment: | Ticket9227.zip added |
|---|
Example of problem with current implementation
Changed 12 years ago by
| Attachment: | CKEditorControl.cs.2.patch added |
|---|
Patch w/ removed unnecessary isChanged variable

Hi KoenVanExem,
Could you explain why is it important to move this event?
What sort of the problem has current code been causing for you?