Summary: | <span> in the title of z39.50 servers page | ||
---|---|---|---|
Product: | Koha | Reporter: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Component: | Templates | Assignee: | Katrin Fischer <katrin.fischer> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | martin.renvoize |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33551 | ||
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.01
|
|
Circulation function: | |||
Attachments: |
Bug 32061: Remove spans from page titles for adding/editing Z39.50/SRU servers
Bug 32061: Remove spans from page titles for adding/editing Z39.50/SRU servers Bug 32061: Remove spans from page titles for adding/editing Z39.50/SRU servers Bug 32061: Remove spans from page titles for adding/editing Z39.50/SRU servers |
Description
Caroline Cyr La Rose
2022-11-01 19:19:46 UTC
Created attachment 143978 [details] [review] Bug 32061: Remove spans from page titles for adding/editing Z39.50/SRU servers In another bug we added spans to ease translations in the PROCESS block for server types. This PROCESS is used in several places, but only creates a problem when used in the title. The patch removes the spans using the remove TT filter avoiding having to change the PROCESS and causing a string change. To test: * Go to administration > Z39.50/SRU servers * Add a new Z39.50 and a SRU server or look at the form * The page title in the browser tab will show the span * Same for editing an existing Z39.50 or SRU server * Apply the patch * Verify the page titles are now correct Hint: you can see the full page title when hovering on the tab. Created attachment 144024 [details] [review] Bug 32061: Remove spans from page titles for adding/editing Z39.50/SRU servers In another bug we added spans to ease translations in the PROCESS block for server types. This PROCESS is used in several places, but only creates a problem when used in the title. The patch removes the spans using the remove TT filter avoiding having to change the PROCESS and causing a string change. To test: * Go to administration > Z39.50/SRU servers * Add a new Z39.50 and a SRU server or look at the form * The page title in the browser tab will show the span * Same for editing an existing Z39.50 or SRU server * Apply the patch * Verify the page titles are now correct Hint: you can see the full page title when hovering on the tab. Signed-off-by: David <david@davidnind.com> Created attachment 144041 [details] [review] Bug 32061: Remove spans from page titles for adding/editing Z39.50/SRU servers In another bug we added spans to ease translations in the PROCESS block for server types. This PROCESS is used in several places, but only creates a problem when used in the title. The patch removes the spans using the remove TT filter avoiding having to change the PROCESS and causing a string change. To test: * Go to administration > Z39.50/SRU servers * Add a new Z39.50 and a SRU server or look at the form * The page title in the browser tab will show the span * Same for editing an existing Z39.50 or SRU server * Apply the patch * Verify the page titles are now correct Hint: you can see the full page title when hovering on the tab. Signed-off-by: David <david@davidnind.com> Signed-off-by: hammat <hammat@inlibro.com> Created attachment 144612 [details] [review] Bug 32061: Remove spans from page titles for adding/editing Z39.50/SRU servers In another bug we added spans to ease translations in the PROCESS block for server types. This PROCESS is used in several places, but only creates a problem when used in the title. The patch removes the spans using the remove TT filter avoiding having to change the PROCESS and causing a string change. To test: * Go to administration > Z39.50/SRU servers * Add a new Z39.50 and a SRU server or look at the form * The page title in the browser tab will show the span * Same for editing an existing Z39.50 or SRU server * Apply the patch * Verify the page titles are now correct Hint: you can see the full page title when hovering on the tab. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Hammat Wele <hammat@inlibro.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Pushed to master for 23.05. Nice work everyone, thanks! Nice work everyone! Pushed to 22.11.x for the next release |