Clean up hover/focus - In Seven

@anandps and I tested this with and without VoiceOver on Safari, Chrome and Firefox in Mac OS. Additionally, we checked it with and without JAWS on Chrome and Firefox in Windows; and without JAWS on IE9, IE10 and IE11 on Windows. In all cases, the focus states are obvious.

Note when I say "without [screenreader software]" I mean "just using the tab key".

Looks like a testbot hiccup because this is just CSS.

Missing fixes in seven's..

.quickedit-button.action-save:hover,
.quickedit-button.action-saving:hover {
  box-shadow: 0 1px 2px hsla(203, 10%, 10%, 0.25);
}

quickedit.css

.form-actions .dropbutton-wrapper input:hover {
  background: none;
  border: none;
}

style.css

.joyride-close-tip:hover {
  color: rgba(255,255,255, 0.9);
  text-decoration: none;
}

tour.theme.css

Can I get some screens?

I dont fully understand why we also have the dotted line around it, isnt the underline enough of a visual change?

I'm not sure why the latest patch isn't being tested but this patch needs a reroll

Ok, only two small things.

+++ b/core/themes/seven/seven.base.css
@@ -11,8 +11,10 @@ a,
-a:hover,
-.link:hover {
+ a:hover,
+.link:hover,
+a:focus,
+.link:focus {
   text-decoration: underline;
 }

If we're going to add underline can we please remove the outline? It's too busy

I noticed this one bug when tabbing over the secondary tabs in Views.

The breadcrumbs seems to use outline & not underline which is inconsistent too. It should just use the underline on focus so that it works just like the mouse only experience.

Agreed

Starting to work on this.

There were a lot of changes in core. quickedit.css, seven.base.css and style.css don't exist anymore. Here is what I did:

• Re-rolled the patch.
• Removed the outline from the breadcrumb
• Removed the focus state from the views secondary buttons

@LewisNyman agreed on keeping this issue small and creating new issues. These are my findings:

Tabbing in Firefox is broken in general:

• The skip to main content link doesn't work
• Primary and secondary tabs don't work
• Breadcrumbs don't work

Focus states should be fixed in views. This is why I removed some changes from the patch.

I guess we should open two new issues for that. Agreed?

I can confirm that the views secondary tabs bug is now fixed and the breadcrumbs look better. Great work everyone.

Same here, just 2 lines offset. Try again?

Committed f3cf8b0 and pushed to 8.0.x. Thanks!