Skip to content

Commit fdf6af4

Browse files
fix: [UIE-9853], [UIE-9855] - IAM: fix Roles Table layout shifts, disable email input for users without account_admin (#13208)
* fix: [UIE-9853], [9855] - IAM: fix Roles Table layout shifts, disable email input for users without account_admin * Added changeset: IAM: Roles table layout shifts when switching tabs, and the email input is enabled for users without the account_admin role * review fix
1 parent a4b4e69 commit fdf6af4

File tree

4 files changed

+8
-1
lines changed

4 files changed

+8
-1
lines changed
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@linode/manager": Fixed
3+
---
4+
5+
IAM: Roles table layout shifts when switching tabs, and the email input is enabled for users without the account_admin role ([#13208](https://github.com/linode/manager/pull/13208))

packages/manager/src/features/IAM/Shared/AssignedEntitiesTable/AssignedEntitiesTable.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -286,6 +286,7 @@ export const AssignedEntitiesTable = ({ username }: Props) => {
286286
alignItems: 'center',
287287
justifyContent: 'flex-start',
288288
marginBottom: theme.tokens.spacing.S12,
289+
minHeight: theme.spacingFunction(40),
289290
}}
290291
>
291292
<DebouncedSearchTextField

packages/manager/src/features/IAM/Shared/AssignedRolesTable/AssignedRolesTable.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -376,6 +376,7 @@ export const AssignedRolesTable = () => {
376376
alignItems: 'center',
377377
justifyContent: 'space-between',
378378
marginBottom: theme.tokens.spacing.S12,
379+
minHeight: theme.spacingFunction(40),
379380
}}
380381
>
381382
<Grid container direction="row" rowSpacing={1}>

packages/manager/src/features/IAM/Users/UserDetails/UserEmailPanel.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ export const UserEmailPanel = ({ canUpdateUser, activeUser }: Props) => {
5454

5555
// This should be disabled if this is NOT the current user or if the proxy user is viewing their own profile.
5656
const disableEmailField =
57-
profileUserName !== activeUser.username || isProxyUser;
57+
profileUserName !== activeUser.username || isProxyUser || !canUpdateUser;
5858

5959
return (
6060
<Paper>

0 commit comments

Comments
 (0)