| Summary: | Breadcrumb wrong on reports dictionary | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jared Camins-Esakov <jcamins> |
| Component: | Staff interface | Assignee: | Jared Camins-Esakov <jcamins> |
| Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
| Severity: | trivial | ||
| Priority: | P5 - low | CC: | bgkriegel, chris, gmcharlt, jonathan.druart |
| Version: | Main | ||
| 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: | ||
| Circulation function: | |||
| Attachments: |
Bug 9658: Incorrect breadcrumb on report dictionary
[SIGNED-OFF] Bug 9658: Incorrect breadcrumb on report dictionary Bug 9658: Incorrect breadcrumb on report dictionary |
||
|
Description
Jared Camins-Esakov
2013-02-18 13:11:01 UTC
Created attachment 15499 [details] [review] Bug 9658: Incorrect breadcrumb on report dictionary Prior to this patch, the breadcrumb on the reports dictionary looks like this: Home > Reports > Guided reports wizard Dictionary (notice the lack of punctuation between "wizard" and "Dictionary"). This patch makes it consistent with the rest of the system so it looks like this: Home > Reports > Guided reports wizard > Dictionary I left dictionary bolded, since that seems to have been intentional, even if I can't find a good reason for it. To test: 1) View reports dictionary page before and after patch is applied, observing difference. Created attachment 15501 [details] [review] [SIGNED-OFF] Bug 9658: Incorrect breadcrumb on report dictionary Prior to this patch, the breadcrumb on the reports dictionary looks like this: Home > Reports > Guided reports wizard Dictionary (notice the lack of punctuation between "wizard" and "Dictionary"). This patch makes it consistent with the rest of the system so it looks like this: Home > Reports > Guided reports wizard > Dictionary I left dictionary bolded, since that seems to have been intentional, even if I can't find a good reason for it. To test: 1) View reports dictionary page before and after patch is applied, observing difference. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Comment: Trivial patch. Work as described. No errors QA comment: Works as expected. Simple patch. Marked as Passed QA. Created attachment 15526 [details] [review] Bug 9658: Incorrect breadcrumb on report dictionary Prior to this patch, the breadcrumb on the reports dictionary looks like this: Home > Reports > Guided reports wizard Dictionary (notice the lack of punctuation between "wizard" and "Dictionary"). This patch makes it consistent with the rest of the system so it looks like this: Home > Reports > Guided reports wizard > Dictionary I left dictionary bolded, since that seems to have been intentional, even if I can't find a good reason for it. To test: 1) View reports dictionary page before and after patch is applied, observing difference. Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Comment: Trivial patch. Work as described. No errors Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> This patch has been pushed to master. Pushed to 3.10.x and 3.8.x will be in 3.10.4 and 3.8.11 |