Fix: Add tooltip for clarity and fix wording

This commit is contained in:
Daniel Hougaard
2024-05-07 17:30:35 +02:00
parent c7d2dfd351
commit 44370d49e3
@@ -344,20 +344,27 @@ export const SpecificPrivilegeSecretForm = ({
render={({ field }) => { render={({ field }) => {
if (policies) { if (policies) {
return ( return (
<FormControl label="Secret Path"> <Tooltip
<Select isDisabled={!!selectablePaths.length}
{...field} content="The selected environment doesn't have any policies."
isDisabled={isMemberEditDisabled || !selectablePaths.length} >
className="w-48" <div>
onValueChange={(e) => field.onChange(e)} <FormControl label="Secret Path">
> <Select
{selectablePaths.map((path) => ( {...field}
<SelectItem value={path} key={path}> isDisabled={isMemberEditDisabled || !selectablePaths.length}
{path} className="w-48"
</SelectItem> onValueChange={(e) => field.onChange(e)}
))} >
</Select> {selectablePaths.map((path) => (
</FormControl> <SelectItem value={path} key={path}>
{path}
</SelectItem>
))}
</Select>
</FormControl>
</div>
</Tooltip>
); );
} }
return ( return (
@@ -515,8 +522,9 @@ export const SpecificPrivilegeSecretForm = ({
); );
}} }}
> >
{temporaryAccessField.isTemporary ? "Restart" : "Grant"} {temporaryAccessField.isTemporary && !policies ? "Restart" : "Grant"}
</Button> </Button>
{temporaryAccessField.isTemporary && ( {temporaryAccessField.isTemporary && (
<Button <Button
size="xs" size="xs"
@@ -528,7 +536,7 @@ export const SpecificPrivilegeSecretForm = ({
}); });
}} }}
> >
Revoke Access Cancel
</Button> </Button>
)} )}
</div> </div>