April 21, 2015 at 5:08 am
Hi,
just a suggestion, I noticed that there is no tooltip for <div class="toggle_grid toggle_active" data-toggle="product_grid"> and <div class="toggle_grid toggle_active" data-toggle="product_grid"> for toggling the list/grid view of products. These icons might not be totally clear what they do to all users. Although for visually impaired users a title attribute would be could for accessibility reasons.
I think for usuability and accessibility reasons this should become:
<div title="__('Show as grid', virtue)" class="toggle_grid toggle_active" data-toggle="product_grid">
<div title="__('Show as list', virtue)" class="toggle_grid toggle_active" data-toggle="product_grid">
I do not know if “Show as grid” is the appropriate wording. But using tooltips by means of title attribute would be good.
I think there are other places in virtue, too, where a title attribute would make sense. E.g. the mini cart (cart icon) is a link to “cart” but has no tooltip (not good), whereas the cart menu icon (basket icon) has the tooltip “View your shopping cart”. Maybe also “My account” button (tooltip “Login here” if not logged in, “View your account” if already logged in). Maybe also for the category filters on the shop to have ‘Only show “<category-name”‘ for the filtering on categaries…
This is just a suggestion 🙂