Cleanup

* removed commented out cruft
* added tabIndex prop to delete button (with default=0)
* the last change also required a change to the sprite selector test snapshot

Author: chrisgarrity

src/components/delete-button/delete-button.css

@@ -17,11 +17,6 @@
 
 }
 
-/* .delete-button.large:hover {
-    transform: scale(1.1, 1.1);
-    box-shadow: 0 0 0 4px $ui-black-transparent;
-} */
-
 .delete-button-visible {
   display: flex;
   align-items: center;

src/components/delete-button/delete-button.jsx

@@ -13,7 +13,7 @@ const DeleteButton = props => (
             props.className
         )}
         role="button"
-        tabIndex="0"
+        tabIndex={props.tabIndex}
         onClick={props.onClick}
     >
         <div className={styles.deleteButtonVisible}>
@@ -28,7 +28,12 @@ const DeleteButton = props => (
 
 DeleteButton.propTypes = {
     className: PropTypes.string,
-    onClick: PropTypes.func.isRequired
+    onClick: PropTypes.func.isRequired,
+    tabIndex: PropTypes.number
+};
+
+DeleteButton.defaultProps = {
+    tabIndex: 0
 };
 
 export default DeleteButton;

test/unit/components/__snapshots__/sprite-selector-item.test.jsx.snap

@@ -50,7 +50,7 @@ exports[`SpriteSelectorItemComponent matches snapshot when given a number and de
     className=""
     onClick={[Function]}
     role="button"
-    tabIndex="0"
+    tabIndex={0}
   >
     <div
       className={undefined}
@@ -134,7 +134,7 @@ exports[`SpriteSelectorItemComponent matches snapshot when selected 1`] = `
     className=""
     onClick={[Function]}
     role="button"
-    tabIndex="0"
+    tabIndex={0}
   >
     <div
       className={undefined}