Bug 27475

Summary: Improve link text to define a patron card printer profile if none are defined
Product: Koha Reporter: Aleisha Amohia <aleisha>
Component: TemplatesAssignee: James O'Keeffe <jamespfk>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: caroline.cyr-la-rose, fridolin.somers, jamespfk, martin.renvoize
Version: unspecifiedKeywords: Academy
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.05.00
Circulation function:
Bug Depends on:    
Bug Blocks: 26722    
Attachments: Bug 27475 - Improve link text to define a patron card printer profile if none are defined
Bug 27475: Improve link text to define a patron card printer profile if none are defined
Bug 27475: Improve link text to define a patron card printer profile if none are defined

Description Aleisha Amohia 2021-01-20 01:38:52 UTC
koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/edit-template.tt:                                        <a href="/cgi-bin/koha/patroncards/edit-profile.pl?op=new">Click here to define a printer profile.</a>

"Here" should not be used as a link text as it doesn't contain any helpful information for people scanning the page for information and also is a problem for blind users using screen readers jumping from link to link on a page.

to reproduce:
go to Tools -> patron card creator
ensure you have no printer profiles
go to add a new card template
notice that next to Profile there is a 'click here' link

we want to reword this.
Comment 1 James O'Keeffe 2021-01-20 02:19:50 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2021-01-20 16:21:02 UTC
Created attachment 115447 [details] [review]
Bug 27475: Improve link text to define a patron card printer profile if none are defined

This patch changes the text on the text link for defining a printer
profile to make it more clear. It does this by removing the "Click
here".

Test plan:
1. Navigate from home -> tools -> patron card creator
2. Ensure you have no printer profiles
3. Select "new", and create a new card template
4. Note the text link "Click here to define a new printer profile"
5. Apply the patch, and repeat steps 1-3
6. Note that the "Click here" has been removed.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Martin Renvoize (ashimema) 2021-01-21 09:31:30 UTC
Created attachment 115513 [details] [review]
Bug 27475: Improve link text to define a patron card printer profile if none are defined

This patch changes the text on the text link for defining a printer
profile to make it more clear. It does this by removing the "Click
here".

Test plan:
1. Navigate from home -> tools -> patron card creator
2. Ensure you have no printer profiles
3. Select "new", and create a new card template
4. Note the text link "Click here to define a new printer profile"
5. Apply the patch, and repeat steps 1-3
6. Note that the "Click here" has been removed.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize (ashimema) 2021-01-21 09:31:55 UTC
Welcome to the team James :)

Passing QA
Comment 5 Jonathan Druart 2021-01-21 15:25:46 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 6 Fridolin Somers 2021-01-25 15:34:43 UTC
I propose not to backport since it is still in work (children of Bug 26722)

Enhancement not pushed to 20.11.x