Bug 34385 - Inconsistencies in Transport cost matrix page header
Summary: Inconsistencies in Transport cost matrix page header
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 34396
  Show dependency treegraph
 
Reported: 2023-07-25 17:00 UTC by Caroline Cyr La Rose
Modified: 2023-12-28 20:47 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: String patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.11.00,23.05.04,22.11.10
Circulation function:


Attachments
Bug 34385: Correct page heading of transport cost matrix page (1.51 KB, patch)
2023-08-18 11:36 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 34385: Correct page heading of transport cost matrix page (1.57 KB, patch)
2023-08-18 14:13 UTC, Lucas Gass
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-07-25 17:00:20 UTC
While testing bug 33909, I had to look at all the pages in the administration module. In doing so, I found some inconsistencies between the various pages. 

For the Transport cost matrix page, here is what should be changed to make it consistent with the other pages in the module:

Page header
- Change 'Defining transport costs between libraries' to 'Transport cost matrix'
Comment 1 Owen Leonard 2023-08-18 11:36:37 UTC
Created attachment 154644 [details] [review]
Bug 34385: Correct page heading of transport cost matrix page

This patch corrects the main heading on the transport cost matrix page
so that it matches the page title and breadcrumb navigation.

To test, apply the patch and go to Administration -> Transport cost
matrix. Confirm that the page heading reads "Transport cost matrix."
Comment 2 Lucas Gass 2023-08-18 14:13:27 UTC
Created attachment 154658 [details] [review]
Bug 34385: Correct page heading of transport cost matrix page

This patch corrects the main heading on the transport cost matrix page
so that it matches the page title and breadcrumb navigation.

To test, apply the patch and go to Administration -> Transport cost
matrix. Confirm that the page heading reads "Transport cost matrix."

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Comment 3 Lucas Gass 2023-08-18 14:14:09 UTC
Trivial fix can go right to PQA.
Comment 4 Tomás Cohen Arazi (tcohen) 2023-08-29 17:58:36 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 5 Fridolin Somers 2023-08-30 06:58:30 UTC
Pushed to 23.05.x for 23.05.04
Comment 6 Matt Blenkinsop 2023-08-31 10:24:25 UTC
Nice work everyone!

Pushed to oldstable for 22.11.x