Summary: | Breadcrumb style | ||
---|---|---|---|
Product: | Koha | Reporter: | David Nind <david> |
Component: | Staff interface | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | fridolin.somers, gmcharlt, lucas, martin.renvoize, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32031 | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
22.11.00
|
|
Circulation function: | |||
Bug Depends on: | 30952 | ||
Bug Blocks: | |||
Attachments: |
Breadcrumbs reverted to the old style
Breadcrumbs using a caret icon Bug 31751: Breadcrumb style Bug 31751: Breadcrumb style Bug 31751: Breadcrumb style |
Description
David Nind
2022-10-12 21:45:03 UTC
> I prefer breadcrumbs to use the > rather than the /. I say we should use an icon like caret : https://fontawesome.com/v4/icon/caret-right > normal text rather than italics/emphasis +1 Created attachment 142240 [details]
Breadcrumbs reverted to the old style
Screenshot showing breadcrumbs reverted to the old style
Created attachment 142241 [details]
Breadcrumbs using a caret icon
Screenshot showing breadcrumbs using a caret icon
I think I just about prefer the old style rather than the new caret icon... but it's a close call for me.. I like them both better than the current `/` Anyone else got an opinion here? The older style for me (>)(ideally without the italics). Older style (>), normal text. Any news? Created attachment 143926 [details] [review] Bug 31751: Breadcrumb style This patch updates the breadcrumb style to use the old caret icon instead of a slash. I've left the italic because I think it works well to set the text off from the normal text above, in the search bar, and below in the main body of the page. To test, apply the patch and rebuild the staff interface CSS. Look at various pages in the staff interface and check the style of the breadcrumbs menu. Created attachment 143928 [details] [review] Bug 31751: Breadcrumb style This patch updates the breadcrumb style to use the old caret icon instead of a slash. I've left the italic because I think it works well to set the text off from the normal text above, in the search bar, and below in the main body of the page. To test, apply the patch and rebuild the staff interface CSS. Look at various pages in the staff interface and check the style of the breadcrumbs menu. Signed-off-by: David Nind <david@davidnind.com> Created attachment 143950 [details] [review] Bug 31751: Breadcrumb style This patch updates the breadcrumb style to use the old caret icon instead of a slash. I've left the italic because I think it works well to set the text off from the normal text above, in the search bar, and below in the main body of the page. To test, apply the patch and rebuild the staff interface CSS. Look at various pages in the staff interface and check the style of the breadcrumbs menu. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Pushed to master for 22.11. Nice work everyone, thanks! |