The options for the 'Change messaging preferences to default for this category?' popup that appears when editing a patron's category are Cancel and OK, but I think No instead of Cancel and Yes instead of OK would be clearer. A library could reasonably think that Cancel refers to cancelling the change in category, and No would make it clearer that refers to the preferences only.
To make this language more clear I think we should use a modal popup instead of a JS alert. +1
Created attachment 168480 [details] [review] Bug 31354: Use modal popup for messaging preferences when editing a patron's category. To test: 1) In Administration->Patron categories, edit the 'Home Bound' category and select every messaging preferences. 2) Edit the 'Patron' category and select email for the 'Item due' messaging pref. 3) Visit a patrons page from any category except Homebound. Keep note of their messaging preferences (will probably just be none) 4) Edit this patron and change their category to 'Homebound'. You should be greeted with a modal that asks if you want to "Change messaging preferences to default for this category?". Select yes. Save changes on the edit form 5) In the details page, note the messaging preferences were set to the defualt for the Homebound category (all selected) 6) Edit patron again, this time change their category to 'Patron'. 7) When the modal pops up, select no. Save the edit form. 8) In details page, the messaging prefs should not have changed.
Created attachment 168482 [details] [review] Bug 31354: Keep same functionality as before
Created attachment 168484 [details] [review] Bug 31354: Use modal popup for messaging preferences when editing a patron's category. To test: 1) In Administration->Patron categories, edit the 'Home Bound' category and select every messaging preferences. 2) Edit the 'Patron' category and select email for the 'Item due' messaging pref. 3) Visit a patrons page from any category except Homebound. Keep note of their messaging preferences (will probably just be none) 4) Edit this patron and change their category to 'Homebound'. You should be greeted with a modal that asks if you want to "Change messaging preferences to default for this category?". Select yes. Save changes on the edit form 5) In the details page, note the messaging preferences were set to the defualt for the Homebound category (all selected) 6) Edit patron again, this time change their category to 'Patron'. 7) When the modal pops up, select no. Save the edit form. 8) In details page, the messaging prefs should not have changed.
Created attachment 168578 [details] [review] Bug 31354: Use modal popup for messaging preferences when editing a patron's category. To test: 1) In Administration->Patron categories, edit the 'Home Bound' category and select every messaging preferences. 2) Edit the 'Patron' category and select email for the 'Item due' messaging pref. 3) Visit a patrons page from any category except Homebound. Keep note of their messaging preferences (will probably just be none) 4) Edit this patron and change their category to 'Homebound'. You should be greeted with a modal that asks if you want to "Change messaging preferences to default for this category?". Select yes. Save changes on the edit form 5) In the details page, note the messaging preferences were set to the defualt for the Homebound category (all selected) 6) Edit patron again, this time change their category to 'Patron'. 7) When the modal pops up, select no. Save the edit form. 8) In details page, the messaging prefs should not have changed. Signed-off-by: David Nind <david@davidnind.com>
Not related to this bug, but I identified this issue - Bug 37269 - Saving edits to a patron - error message clears or resets some fields.
Created attachment 171474 [details] Something looks a bit weird here
Thanks for the effort to make this a real modal! The close button is however unstyled and I run into the following issues: 1) After running through the test plan I changed the patron category back to Homebound again 2) I then selected 'yes' in the modal. 3) Then I selected Homebound again. 4) Then I tried to select 'no' in the modal, but there was no reaction. 5) There was an alert with a spinner above the messaging preferences, though. Informing me on that message preference details were loaded. Could you try to reproduce this issue and then set it to Failed QA, or supply a patch for it?
Created attachment 171905 [details] [review] Bug 31354: Adjust modal for Bootstrap5 changes
> Could you try to reproduce this issue and then set it to Failed QA, or > supply a patch for it? Paul can you try again now? I adjusted the modal for Bootstrap 5.
Oh, something I noticed. I can only change the patron category once. -Change to HOMEBOUND, say yes to change the prefs. -Change to another patron category, no pop-op and prefs stay the same.
(In reply to Lucas Gass (lukeg) from comment #11) > Oh, something I noticed. I can only change the patron category once. > > -Change to HOMEBOUND, say yes to change the prefs. > -Change to another patron category, no pop-op and prefs stay the same. Please clarify or fix. Changing status to reflect need for feedback