Bug 33549 - Patron restriction types - Style missing for dialog messages
Summary: Patron restriction types - Style missing for dialog messages
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: System Administration (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low trivial (vote)
Assignee: Lucas Gass
QA Contact: Katrin Fischer
URL:
Keywords:
Depends on: 23681
Blocks:
  Show dependency treegraph
 
Reported: 2023-04-17 20:01 UTC by Caroline Cyr La Rose
Modified: 2023-12-28 20:43 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.06


Attachments
Bug 33549: Fix dialog box in patron restrictions (1.15 KB, patch)
2023-04-18 17:08 UTC, Lucas Gass
Details | Diff | Splinter Review
Bug 33549: Fix dialog box in patron restrictions (1.22 KB, patch)
2023-04-19 09:05 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 33549: Fix dialog box in patron restrictions (1.28 KB, patch)
2023-04-20 21:08 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 33549: Improve wording of dialog messages (1.76 KB, patch)
2023-04-20 21:08 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Caroline Cyr La Rose 2023-04-17 20:01:43 UTC
When adding or deleting patron restriction types, the dialog messages ("Type added", "Type deleted", etc.) have no styling. Normally, such messages are on a blue background.

To test:
1. Enable PatronRestrictionTypes
2. Go to Administration > Patron restrictions
3. Click 'New restriction'
4. Enter a code and label
5. Click 'Save'
   --> 'Type added' at the top of the page is not styled

To see a regular dialog message:
1. Go to Administration > Debit types
2. Click 'New debit type'
3. Enter a code and description
4. Click 'Save'
   --> 'Debit type saved successfully' on blue background
Comment 1 Lucas Gass 2023-04-18 17:08:06 UTC
Created attachment 149824 [details] [review]
Bug 33549: Fix dialog box in patron restrictions

To test:
1. Enable PatronRestrictionTypes
2. Go to Administration > Patron restrictions
3. Click 'New restriction'
4. Enter a code and label
5. Click 'Save'
   --> 'Type added' at the top of the page is not styled
6. Apply patch
7. Try again, dialog is styled correctly
Comment 2 ByWater Sandboxes 2023-04-19 09:05:40 UTC
Created attachment 149845 [details] [review]
Bug 33549: Fix dialog box in patron restrictions

To test:
1. Enable PatronRestrictionTypes
2. Go to Administration > Patron restrictions
3. Click 'New restriction'
4. Enter a code and label
5. Click 'Save'
   --> 'Type added' at the top of the page is not styled
6. Apply patch
7. Try again, dialog is styled correctly

Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
Comment 3 Sally 2023-04-19 09:06:01 UTC
Works as described, thanks for the patch.
Comment 4 Katrin Fischer 2023-04-20 21:08:41 UTC
Created attachment 149996 [details] [review]
Bug 33549: Fix dialog box in patron restrictions

To test:
1. Enable PatronRestrictionTypes
2. Go to Administration > Patron restrictions
3. Click 'New restriction'
4. Enter a code and label
5. Click 'Save'
   --> 'Type added' at the top of the page is not styled
6. Apply patch
7. Try again, dialog is styled correctly

Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Katrin Fischer 2023-04-20 21:08:44 UTC
Created attachment 149997 [details] [review]
Bug 33549: Improve wording of dialog messages

In terms of translations type can be a lot of things.
Usually translations are easier with more context, so
I've added restriction to the type :)

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Tomás Cohen Arazi 2023-04-24 13:50:06 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 7 Jacob O'Mara 2023-05-10 15:00:43 UTC
Nice work, thanks everyone!

Pushed to 22.11.x for the next release.
Comment 8 Lucas Gass 2023-05-15 21:30:03 UTC
Not needed for 22.05.x, no backport.