packagecontrol.io icon indicating copy to clipboard operation
packagecontrol.io copied to clipboard
verified publisher icon wbond

adding tabindex to logo and search field

Open jamiewilson opened this issue 9 years ago • 2 comments

First tab to search, second tab to logo/home link, third to next anchor, currently first link in the nav

jamiewilson avatar Mar 29 '16 00:03 jamiewilson

Thanks for taking the time to suggest from improvements!

The opacity transition is currently being used (https://github.com/wbond/packagecontrol.io/blob/master/app/css/package.scss#L7-L13). Try navigating from a package detail page to the homepage. Also try hovering over the logo on a package detail page.

Setting the tab index should be good.

Generally I'm not a huge fan of overriding the outline, but I haven't looked to see how this specific example functions.

wbond avatar Mar 29 '16 03:03 wbond

Cool. Thanks for pointing that out. Somehow I missed that opacity usage!

I just removed the styling, so feel free to merge this for the tabindex additions, but if it's easier to add them yourself, no worries. Just one of those things I always find myself doing when I come to search.

jamiewilson avatar Mar 29 '16 18:03 jamiewilson