Dubunin/breadcrumb issues (#12)

* #191731: updated hover state color for overflow button

* #191719: added override styles for breadcrumb focus rect

* #191719: code formatting - missing space

* #191723: per design - removed negative margin-right from overflow breadcrumb

* #191723: code cleanup

* #191723: updated to use localized margins

* 191731: removed overrides that are no longer necessary. Updated to use focus-border.

src/components/Breadcrumb/Breadcrumb.scss

@@ -47,15 +47,13 @@
 }
 
 .ms-Breadcrumb-overflow {
-  display: none;
+  display: inline-block;
   position: relative;
-  margin-right: -4px;
 
   .ms-Breadcrumb-overflowButton {
     font-size: $ms-font-size-s;
     display: inline-block;
     color:$ms-color-themePrimary;
-    margin-right: -4px;
     padding: 12px 8px 3px 8px;
     cursor: pointer;
   }
@@ -106,6 +104,7 @@
 
   &:hover {
     background-color: $ms-color-neutralLighter;
+    color: $ms-color-themeDark;
     cursor: pointer;
   }
 
@@ -121,6 +120,7 @@
 }
 
 .ms-Breadcrumb-itemLink {
+  @include focus-border();
   @include ms-font-xl();
   display: inline-block;
   padding: 0 4px;
@@ -130,7 +130,6 @@
   overflow: hidden;
 }
 
-
 @media screen and (max-width: $ms-screen-md-max) {
   .ms-Breadcrumb {
     margin: 10px 0;
@@ -170,14 +169,3 @@
     }
   }
 }
-
-// TODO: Overrides that need to be removed.
-
-.ms-Breadcrumb-overflow {
-  display: inline-block;
-}
-
-.ms-Breadcrumb-overflowButton {
-  line-height: 12px;
-  margin-right: 0px;
-}