remove hover effect from item loading placeholders
this has been annoying me for a while 😅 it's confusing and surprising that they act interactable when they very much aren't yet!
This commit is contained in:
parent
64ad77ec49
commit
bfb0979801
1 changed files with 16 additions and 15 deletions
|
@ -102,7 +102,7 @@ function Item({ item, itemNameId, isWorn, isInOutfit, dispatchToOutfit }) {
|
||||||
*/
|
*/
|
||||||
function ItemSkeleton() {
|
function ItemSkeleton() {
|
||||||
return (
|
return (
|
||||||
<ItemContainer>
|
<ItemContainer isFocusable={false}>
|
||||||
<Skeleton width="50px" height="50px" />
|
<Skeleton width="50px" height="50px" />
|
||||||
<Box width="3" />
|
<Box width="3" />
|
||||||
<Skeleton height="1.5rem" width="12rem" />
|
<Skeleton height="1.5rem" width="12rem" />
|
||||||
|
@ -117,7 +117,7 @@ function ItemSkeleton() {
|
||||||
* styles - including for its children, who sometimes reference it as an
|
* styles - including for its children, who sometimes reference it as an
|
||||||
* .item-container parent!
|
* .item-container parent!
|
||||||
*/
|
*/
|
||||||
function ItemContainer({ children }) {
|
function ItemContainer({ children, isFocusable = true }) {
|
||||||
const theme = useTheme();
|
const theme = useTheme();
|
||||||
|
|
||||||
const focusBackgroundColor = useColorModeValue(
|
const focusBackgroundColor = useColorModeValue(
|
||||||
|
@ -142,25 +142,26 @@ function ItemContainer({ children }) {
|
||||||
borderRadius="lg"
|
borderRadius="lg"
|
||||||
d="flex"
|
d="flex"
|
||||||
alignItems="center"
|
alignItems="center"
|
||||||
cursor="pointer"
|
cursor={isFocusable ? "pointer" : undefined}
|
||||||
border="1px"
|
border="1px"
|
||||||
borderColor="transparent"
|
borderColor="transparent"
|
||||||
className={cx([
|
className={cx([
|
||||||
"item-container",
|
"item-container",
|
||||||
css`
|
isFocusable &&
|
||||||
&:hover,
|
css`
|
||||||
input:focus + & {
|
&:hover,
|
||||||
background-color: ${focusBackgroundColor};
|
input:focus + & {
|
||||||
}
|
background-color: ${focusBackgroundColor};
|
||||||
|
}
|
||||||
|
|
||||||
input:active + & {
|
input:active + & {
|
||||||
border-color: ${activeBorderColor};
|
border-color: ${activeBorderColor};
|
||||||
}
|
}
|
||||||
|
|
||||||
input:checked:focus + & {
|
input:checked:focus + & {
|
||||||
border-color: ${focusCheckedBorderColor};
|
border-color: ${focusCheckedBorderColor};
|
||||||
}
|
}
|
||||||
`,
|
`,
|
||||||
])}
|
])}
|
||||||
>
|
>
|
||||||
{children}
|
{children}
|
||||||
|
|
Loading…
Reference in a new issue