17452 Posts in 4473 Topics by 1971 members
Page: 1 2
|Go to End|
14 June 2007 at 5:46am
> "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).
14 June 2007 at 1:51pm
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.
15 June 2007 at 8:33am Last edited: 15 June 2007 9:08am
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.
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?
16 June 2007 at 3:34pm
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:
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,
Page: 1 2
|Go to Top|