You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lenya.apache.org by bu...@apache.org on 2007/10/15 23:25:43 UTC

DO NOT REPLY [Bug 43628] New: - tinyMCE is lacking a unload handler (as already described in the tinymce todo list)

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43628>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43628

           Summary: tinyMCE is lacking a unload handler (as already
                    described in the tinymce todo list)
           Product: Lenya
           Version: 2.0
          Platform: Macintosh
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P3
         Component: TinyMCE Integration
        AssignedTo: dev@lenya.apache.org
        ReportedBy: ragaller@apache.org


Currently tinyMCE can be exited by clicking any link outside the editable area. This can
lead to data loss and always leads to a locked page (force checkin has to be used).

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lenya.apache.org
For additional commands, e-mail: dev-help@lenya.apache.org


DO NOT REPLY [Bug 43628] - tinyMCE is lacking a unload handler (as already described in the tinymce todo list)

Posted by bu...@apache.org.
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43628>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43628





------- Additional Comments From nettings@apache.org  2007-10-25 06:41 -------
i've read the patch, and it looks very good and in keeping with tinymce coding
style. however, i'm wondering:
tinymce has this issue because it tries to provide true wysiwyg in-place editing.
this capability might be highly desirable for other editors, too.

maybe we could implement a generic onunload() handler that will run an
editor-specific callback if provided (similar to the way richard implemented the
backspace catcher in an editor-agnostic way).

what does an onunload() handler need to do? 
* catch the onunload event and prevent leaving the current page.
* provide a message that informs the user of unsaved data and allows to "leave",
"save and leave", "return to editor".

"leave" must remove the lock on the document and then execute the previously
caught navigation event.
"save and leave" will have to signal the editor (invoke its save callback) and
then navigate away.
"return" just eats the unload event and return to the editor.

wdyt?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lenya.apache.org
For additional commands, e-mail: dev-help@lenya.apache.org


DO NOT REPLY [Bug 43628] - tinyMCE is lacking a unload handler (as already described in the tinymce todo list)

Posted by bu...@apache.org.
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43628>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43628





------- Additional Comments From ragaller@apache.org  2007-10-25 07:22 -------
(In reply to comment #2)
> this capability might be highly desirable for other editors, too.

A big +

> 
> maybe we could implement a generic onunload() handler that will run an
> editor-specific callback if provided (similar to the way richard implemented the
> backspace catcher in an editor-agnostic way).

another big +

> 
> what does an onunload() handler need to do? 
> * catch the onunload event and prevent leaving the current page.
> * provide a message that informs the user of unsaved data and allows to "leave",
> "save and leave", "return to editor".
> 
> "leave" must remove the lock on the document and then execute the previously
> caught navigation event.
> "save and leave" will have to signal the editor (invoke its save callback) and
> then navigate away.
> "return" just eats the unload event and return to the editor.

Another big + for the idea. The problem with document.onbeforeunload is, that it
triggers a predefined screen that can only partially be modified. It would be
very nice to have the three options you described - but I did not find a way to
display and process a custom screen on an onbeforeunload event. Maybe there is a
better way to do the event handling than using window.onbeforeunload.




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lenya.apache.org
For additional commands, e-mail: dev-help@lenya.apache.org


DO NOT REPLY [Bug 43628] - tinyMCE is lacking a unload handler (as already described in the tinymce todo list)

Posted by bu...@apache.org.
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=43628>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=43628





------- Additional Comments From ragaller@apache.org  2007-10-15 14:35 -------
Created an attachment (id=20986)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=20986&action=view)
the attached patch show a possible approach

It is still in a raw form and is meant as an illustration of concept - please
do not apply as is.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lenya.apache.org
For additional commands, e-mail: dev-help@lenya.apache.org