no visual indication of current keyboard focus on links in #toolbar in IE8

I think the :active pseudo-class also needs to be added to the CSS block immediately above the one that the patch currently modifies, i.e., also change

#toolbar div.toolbar-menu ul li a:focus, 
#toolbar div.toolbar-menu ul li a:hover, 
#toolbar div.toolbar-menu ul li a.active:focus {
background:none repeat scroll 0 0 #444444;
}

to

#toolbar div.toolbar-menu ul li a:focus, 
#toolbar div.toolbar-menu ul li a:hover, 
#toolbar div.toolbar-menu ul li a:active, 
#toolbar div.toolbar-menu ul li a.active:focus {
background:none repeat scroll 0 0 #444444;
}

Additionally, I believe that #toolbar div.toolbar-menu ul li a.active:focus can also be removed since it is applied and overwritten by the CSS block that follows and which this patch currently modifies.

My pleasure :) Seems to be working fine now as links in the toolbar show the hover effect when receiving keyboard focus in IE.

Setting to Critical as it prevents certain users from being able to use the toolbar. Luckily it's an easy fix. Thanks Mike and Jason.

Looks good.

Awesome, thanks! Committed to HEAD!

I'm a bit confused here. While :active might have been good to add, too, the actual cause for the accessibility issue should be the override/reset/removal of 'outline', no?

I think removing any outline style shouldn't be any problem as long as you provide an alternative. Like in case of the toolbar different background-colors.