Bug 33882

Summary: member.tt Date of birth column makes it difficult to hide the age hint
Product: Koha Reporter: Lucas Gass (lukeg) <lucas>
Component: PatronsAssignee: Lucas Gass (lukeg) <lucas>
Status: CLOSED FIXED QA Contact: David Cook <dcook>
Severity: minor    
Priority: P5 - low CC: dcook, gmcharlt, kyle.m.hall
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.11.00,23.05.01,22.11.07
Circulation function:
Attachments: Bug 33882: Add spans in dateofbirth column on member.tt
Bug 33882: Add spans in dateofbirth column on member.tt
Bug 33882: Add spans in dateofbirth column on member.tt

Description Lucas Gass (lukeg) 2023-06-01 20:03:11 UTC
When doing a patron search ( member.tt) there is a column for Date of Birth. That column also includes a hint about the age of the patron. Example: (67 years)

Some libraries don't want to show information about the patron's age. Currently that info. is difficult to hide with CSS. 

If we wrapped it in an <span> it would be much easier.
Comment 1 Lucas Gass (lukeg) 2023-06-01 20:07:05 UTC
Created attachment 151953 [details] [review]
Bug 33882: Add spans in dateofbirth column on member.tt

To test:
1 Apply patch
2 Do a patron search that will return multiple patrons.
3 Look at the date of birth column and notice the text is now wrapped in a class called 'dateofbirth'
4. Notice that the age hint is wrapped in a class called 'agehint'.
5. Add '.agehint { display: none }' to intranetusercss.
6. Now notice the date of birth still displays but no age information.
Comment 2 Sam Lau 2023-06-01 20:44:03 UTC
Created attachment 151954 [details] [review]
Bug 33882: Add spans in dateofbirth column on member.tt

To test:
1 Apply patch
2 Do a patron search that will return multiple patrons.
3 Look at the date of birth column and notice the text is now wrapped in a class called 'dateofbirth'
4. Notice that the age hint is wrapped in a class called 'agehint'.
5. Add '.agehint { display: none }' to intranetusercss.
6. Now notice the date of birth still displays but no age information.

Signed-off-by: Sam Lau <samalau@gmail.com>
Comment 3 David Cook 2023-06-06 06:30:28 UTC
Created attachment 152034 [details] [review]
Bug 33882: Add spans in dateofbirth column on member.tt

To test:
1 Apply patch
2 Do a patron search that will return multiple patrons.
3 Look at the date of birth column and notice the text is now wrapped in a class called 'dateofbirth'
4. Notice that the age hint is wrapped in a class called 'agehint'.
5. Add '.agehint { display: none }' to intranetusercss.
6. Now notice the date of birth still displays but no age information.

Signed-off-by: Sam Lau <samalau@gmail.com>
Signed-off-by: David Cook <dcook@prosentient.com.au>
Comment 4 Tomás Cohen Arazi (tcohen) 2023-06-06 13:01:28 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 5 Tomás Cohen Arazi (tcohen) 2023-06-09 14:21:48 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 6 Matt Blenkinsop 2023-06-12 10:58:29 UTC
Nice work everyone!

Pushed to oldstable for 22.11.x