Jump to:

17452 Posts in 4473 Topics by 1971 members

Archive

SilverStripe Forums » Archive » tinymce autosave confirmation

Our old forums are still available as a read-only archive.

Moderators: martimiz, Sean, biapar, Willr, Ingo, simon_w

Page: 1 2
Go to End
Author Topic: 5178 Views
  • elijahlofgren
    Avatar
    Google Summer of Code Hacker
    222 Posts

    Re: tinymce autosave confirmation Link to this post

    > "To be honest, this combined with Ingo's gripes about load time makes me wonder if we shouldn't just to it the same way as Google Docs - an ok/cancel box that says "do you really want to leave without saving your changes?" and if they don't, forces people to press cancel and then save manually. "

    I think this is a really good idea. I plan on emailing Sean and asking him about changing the save reminder dialog to be very similar to the Google Docs one.

    >I agree with Ingo that opt-out is the only acceptable solution here

    I won't plan on changing the current method of "registering" fields for auto-save detection. I'll just work on improving the usability of the save reminder dialog by making it appear in all the places is should and making it faster (if Sean gives the go ahead).

  • elijahlofgren
    Avatar
    Google Summer of Code Hacker
    222 Posts

    Re: tinymce autosave confirmation Link to this post

    Sam, Ingo, and Sean all agree that using a dialog similar to the Google Docs one is good, so I'll plan on changing the Save reminder to be like that one.

  • elijahlofgren
    Avatar
    Google Summer of Code Hacker
    222 Posts

    Re: tinymce autosave confirmation Link to this post

    I've changed the alert to a javascript confirm dialog in the latest SVN of the cms gsoc branch. It seems to work in all sections.

    However, in the security section only the "Group name" field has change detection.

    If a new member "Firstname", "Surname", or "Email" is typed into the blanks without the "Add" button being pushed, no alert about unsaved changes is shown if another link is clicked.

    Two questions:
    1. Should there be change detection on the 3 "Firstname", "Surname", or "Email" blanks at the bottom of the member list?
    2. If so, any tips on how to implement it?

    Edit: The add recipients form (shown when "Mailing List" is clicked in Newsletter section), which is very similar to the add member form does have change detection, so I'm guessing we should find a way to implement this to maintain consistency?

    Thanks,

    Elijah

  • elijahlofgren
    Avatar
    Google Summer of Code Hacker
    222 Posts

    Re: tinymce autosave confirmation Link to this post

    I figured out how to enable change detection for "Firstname", "Surname", or "Email" fields. I simply had do remove them from the changeDetection_fieldsToIgnore list which I did with this patch:
    http://www.elijahlofgren.com/silverstripe/patches/Fix-Usability-Issue-91-warning-needs-to-be-added-for-unsaved-changes-in-security-section-cms-gsoc-r36933.patch

    Let me know if I should make any more changes, I'm marking this issue as complete: http://www.elijahlofgren.com/silverstripe/warning-needs-to-be-added-for-unsaved-changes-in-security-section/

    Have a great weekend,

    Elijah Lofgren

    5178 Views
Page: 1 2
Go to Top

Want to know more about the company that brought you SilverStripe? Then check out SilverStripe.com

Comments on this website? Please give feedback.