Skip to content

Commit d1a5f4d

Browse files
committed
chore: update link components
1 parent 26ccfe3 commit d1a5f4d

File tree

2 files changed

+10
-12
lines changed

2 files changed

+10
-12
lines changed

site/src/pages/ManagementSettingsPage/IdpSyncPage/IdpSyncPage.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ export const IdpSyncPage: FC = () => {
8585
<p className="flex flex-row gap-1 text-sm text-content-secondary font-medium m-0">
8686
Automatically assign groups or roles to a user based on their IdP
8787
claims.
88-
<Link text="View docs" href={docs("/admin/users/idp-sync")} />
88+
<Link href={docs("/admin/users/idp-sync")}>View docs</Link>
8989
</p>
9090
</div>
9191
</header>

site/src/pages/ManagementSettingsPage/IdpSyncPage/IdpSyncPageView.tsx

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -166,13 +166,12 @@ const IdpMappingTable: FC<IdpMappingTableProps> = ({
166166
<TableRow>
167167
<TableCell colSpan={999}>
168168
<EmptyState
169-
message={`No ${type} Mappings`}
169+
message={`No ${type.toLocaleLowerCase()} mappings`}
170170
isCompact
171171
cta={
172-
<Link
173-
text={`How to setup IdP ${type} sync`}
174-
href={docs("/admin/users/idp-sync")}
175-
/>
172+
<Link href={docs("/admin/users/idp-sync")}>
173+
How to setup IdP {type.toLocaleLowerCase()} sync
174+
</Link>
176175
}
177176
/>
178177
</TableCell>
@@ -202,7 +201,7 @@ const GroupRow: FC<GroupRowProps> = ({ idpGroup, coderGroup, onDelete }) => {
202201
<TableCell>
203202
<Button
204203
variant="outline"
205-
className="w-8 h-8 px-1.5 py-1.5 text-content-secondary"
204+
className="w-8 h-8 min-w-10 text-content-primary"
206205
aria-label="delete"
207206
onClick={() => onDelete(idpGroup)}
208207
>
@@ -655,7 +654,7 @@ const RoleRow: FC<RoleRowProps> = ({ idpRole, coderRoles, onDelete }) => {
655654
<TableCell>
656655
<Button
657656
variant="outline"
658-
className="w-8 h-8 px-1.5 py-1.5 text-content-secondary"
657+
className="w-8 h-8 min-w-10 text-content-primary"
659658
aria-label="delete"
660659
onClick={() => onDelete(idpRole)}
661660
>
@@ -700,10 +699,9 @@ const LegacyGroupSyncHeader: FC = () => {
700699
configure IdP sync via the CLI or the UI, which enables sync to be
701700
configured for any organization, and for those settings to be
702701
persisted without manually setting environment variables.{" "}
703-
<Link
704-
text="Learn more&hellip;"
705-
href={docs("/admin/users/idp-sync")}
706-
/>
702+
<Link href={docs("/admin/users/idp-sync")}>
703+
Learn more&hellip;
704+
</Link>
707705
</HelpTooltipText>
708706
</HelpTooltipContent>
709707
</HelpTooltip>

0 commit comments

Comments
 (0)