| Summary: | On circulation home page, URL to offline circulation Firefox addon should be a hyperlink | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Owen Leonard <oleonard> |
| Component: | Templates | Assignee: | Owen Leonard <oleonard> |
| Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
| Severity: | minor | ||
| Priority: | P5 - low | CC: | m.de.rooy, nengard, paul.poulain |
| Version: | 3.8 | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Bug 7828 - On circulation home page, URL to offline circulation Firefox addon should be a hyperlink
[SIGNED-OFF] Bug 7828 - On circulation home page, URL to offline circulation Firefox addon should be a hyperlink |
||
|
Description
Owen Leonard
2012-03-26 14:41:39 UTC
Created attachment 9114 [details] [review] Bug 7828 - On circulation home page, URL to offline circulation Firefox addon should be a hyperlink Along with making the URL a hyperlink I removed the lanugage part of the URL ("/fr/"). The link correctly redirects to 'en' for me. I'm curious whether it will for other languages. Created attachment 9124 [details] [review] [SIGNED-OFF] Bug 7828 - On circulation home page, URL to offline circulation Firefox addon should be a hyperlink Along with making the URL a hyperlink I removed the lanugage part of the URL ("/fr/"). The link correctly redirects to 'en' for me. I'm curious whether it will for other languages. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Link works and redirects to my langauge (english). QA Comment: Looks good to me. Not sure if the link will work for all variations of staff client language. (Do not have e.g. nl-NL on my staff client, but addons.mozilla.org redirected nl-NL to ../nl/.. Probably HTTP_ACCEPT_LANGUAGE or so.) The same will hold for main languages like de-DE and fr-FR. Since there was no link at first, and it will probably work for most languages and broken links can be fixed with a followup, I'm marking it as Passed QA. No translation issues. patch pushed (and yes, if your browser if in french, you're redirected to fr page) |