Skip to content

Commit

Permalink
Fix Integration Button styles
Browse files Browse the repository at this point in the history
If a user could list integrations, and they could NOT create
integrations, AND integrations existed, this would cause the disabled
'enroll new integration' button to be smashed to the left of the header
(this is due to how its wrapped in an optional Hover Tooltip).

This PR just adds some space-between flex property to the header to
ensure the button stays on the right side. It also moves the
HoverTooltip position to the bottom because it looks better
  • Loading branch information
avatus authored and github-actions committed Dec 16, 2024
1 parent 059cfd6 commit 2df493a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion web/packages/teleport/src/Integrations/Integrations.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ export function Integrations() {
return (
<>
<FeatureBox>
<FeatureHeader>
<FeatureHeader justifyContent="space-between">
<FeatureHeaderTitle>Integrations</FeatureHeaderTitle>
<IntegrationsAddButton
requiredPermissions={[
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@ export function IntegrationsAddButton({

return (
<HoverTooltip
position="bottom"
tipContent={
canCreateIntegrations ? null : (
<MissingPermissionsTooltip
Expand Down

0 comments on commit 2df493a

Please sign in to comment.