Skip to content
Snippets Groups Projects
Commit 4e47cc87 authored by Chris Garrity's avatar Chris Garrity
Browse files

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
parent 415ecee8
No related branches found
No related tags found
Loading
......@@ -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;
......
......@@ -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;
......@@ -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}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment