From f3bca9af39d81811a22f6c3f7ba49250a3216feb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Dro=C5=84?= Date: Mon, 21 Mar 2022 12:37:55 +0100 Subject: [PATCH] Fix invisible edit button in media tiles hover state (#1920) * Fix control icons in media tiles * Update snapshots * Change divs to buttons * Update snapshots --- src/components/MediaTile/MediaTile.tsx | 29 +- .../__snapshots__/Stories.test.ts.snap | 256 +++++++----------- 2 files changed, 126 insertions(+), 159 deletions(-) diff --git a/src/components/MediaTile/MediaTile.tsx b/src/components/MediaTile/MediaTile.tsx index 90ca30228..f30a74f10 100644 --- a/src/components/MediaTile/MediaTile.tsx +++ b/src/components/MediaTile/MediaTile.tsx @@ -1,6 +1,5 @@ import { CircularProgress } from "@material-ui/core"; -import EditIcon from "@material-ui/icons/Edit"; -import { DeleteIcon, IconButton, makeStyles } from "@saleor/macaw-ui"; +import { DeleteIcon, EditIcon, makeStyles } from "@saleor/macaw-ui"; import classNames from "classnames"; import React from "react"; @@ -28,7 +27,8 @@ const useStyles = makeStyles( width: 148 }, mediaOverlay: { - background: "rgba(0, 0, 0, 0.6)", + background: theme.palette.background.default, + opacity: 0.8, cursor: "move", display: "none", height: 148, @@ -47,6 +47,21 @@ const useStyles = makeStyles( mediaOverlayToolbar: { display: "flex", justifyContent: "flex-end" + }, + controlButton: { + color: theme.palette.saleor.main[1], + backgroundColor: "transparent", + border: "none", + cursor: "pointer", + margin: theme.spacing(2), + padding: 0, + + "&:hover": { + color: theme.palette.saleor.active[1] + }, + "&:first-child": { + marginRight: 0 + } } }), { name: "MediaTile" } @@ -84,14 +99,14 @@ const MediaTile: React.FC = props => { ) : (
{onEdit && ( - + )} {onDelete && ( - + )}
)} diff --git a/src/storybook/__snapshots__/Stories.test.ts.snap b/src/storybook/__snapshots__/Stories.test.ts.snap index dabfbff7a..46ab32b03 100644 --- a/src/storybook/__snapshots__/Stories.test.ts.snap +++ b/src/storybook/__snapshots__/Stories.test.ts.snap @@ -40337,27 +40337,21 @@ exports[`Storyshots Views / Categories / Update category default 1`] = ` class="MediaTile-mediaOverlayToolbar-id" > @@ -41116,27 +41110,21 @@ exports[`Storyshots Views / Categories / Update category form errors 1`] = ` class="MediaTile-mediaOverlayToolbar-id" > @@ -43232,27 +43220,21 @@ exports[`Storyshots Views / Categories / Update category no products 1`] = ` class="MediaTile-mediaOverlayToolbar-id" > @@ -44006,27 +43988,21 @@ exports[`Storyshots Views / Categories / Update category no subcategories 1`] = class="MediaTile-mediaOverlayToolbar-id" > @@ -44778,27 +44754,21 @@ exports[`Storyshots Views / Categories / Update category products 1`] = ` class="MediaTile-mediaOverlayToolbar-id" > @@ -51179,27 +51149,21 @@ exports[`Storyshots Views / Collections / Collection detailsCollection details d class="MediaTile-mediaOverlayToolbar-id" > @@ -52655,27 +52619,21 @@ exports[`Storyshots Views / Collections / Collection detailsCollection details f class="MediaTile-mediaOverlayToolbar-id" > @@ -55027,27 +54985,21 @@ exports[`Storyshots Views / Collections / Collection detailsCollection details n class="MediaTile-mediaOverlayToolbar-id" >