Fix how disabled app rows in Apps work and look (#3263)

This commit is contained in:
Lukasz Ostrowski 2023-03-02 14:56:39 +01:00 committed by GitHub
parent 3654d82600
commit 5b7ad12ef0
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -34,7 +34,7 @@ export const InstalledAppListRow: React.FC<InstalledApp> = props => {
state={{ from: location.pathname }} state={{ from: location.pathname }}
className={sprinkles({ display: "contents" })} className={sprinkles({ display: "contents" })}
inline={false} inline={false}
disabled={!!app.isActive} disabled={!app.isActive}
> >
<List.Item <List.Item
padding={7} padding={7}
@ -45,6 +45,7 @@ export const InstalledAppListRow: React.FC<InstalledApp> = props => {
flexDirection="row" flexDirection="row"
flexWrap="wrap" flexWrap="wrap"
backgroundColor={!app.isActive ? "surfaceNeutralSubdued" : undefined} backgroundColor={!app.isActive ? "surfaceNeutralSubdued" : undefined}
cursor={app.isActive ? "pointer" : "not-allowed"}
> >
<Box <Box
gap={5} gap={5}
@ -93,7 +94,9 @@ export const InstalledAppListRow: React.FC<InstalledApp> = props => {
justifyContent={{ mobile: "flex-end", desktop: "flex-start" }} justifyContent={{ mobile: "flex-end", desktop: "flex-start" }}
gap={6} gap={6}
> >
<AppPermissions permissions={app.permissions} /> <Box marginLeft="auto">
<AppPermissions permissions={app.permissions} />
</Box>
<TableButtonWrapper> <TableButtonWrapper>
<Button <Button
variant="secondary" variant="secondary"