Personally, I prefer breadcrumbs to use the > rather than the /. In addition, I think they should be normal text rather than italics/emphasis. I think the size for this are OK - visible, but not intrusive. Some research on breadcrumbs from the Nielsen Norman Group: Breadcrumbs: 11 Design Guidelines for Desktop and Mobile. 23 December 2018. https://www.nngroup.com/articles/breadcrumbs/
> 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!