Opened 12 years ago
Last modified 11 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 12 years ago by
Attachment: | CKEditorControl.cs.patch added |
---|
comment:1 follow-up: 2 Changed 12 years ago by
Keywords: | OnTextChanged removed |
---|---|
Status: | new → pending |
comment:2 Changed 12 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 12 years ago by
Keywords: | HasPatch added |
---|---|
Status: | pending → confirmed |
Changed 11 years ago by
Attachment: | Ticket9227.zip added |
---|
Example of problem with current implementation
Changed 11 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?