Skip to content

refactor(site): make HelpTooltip easier to reuse and compose #11242

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Dec 19, 2023
Merged

refactor(site): make HelpTooltip easier to reuse and compose #11242

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
ce7394c
Refactor help tooltip
BrunoQuaresma Dec 15, 2023
94390a6
Fix size type
BrunoQuaresma Dec 18, 2023
77fec0e
Remove duplicated math round
BrunoQuaresma Dec 18, 2023
0d4e728
Fix agent outdated tooltip role
BrunoQuaresma Dec 18, 2023
11d56b1
Fix diff use cases
BrunoQuaresma Dec 18, 2023
06fc2ed
Fix a few use cases
BrunoQuaresma Dec 18, 2023
59fd96a
Merge branch 'bq/refactor-help-tooltip' of https://github.com/coder/c…
BrunoQuaresma Dec 18, 2023
682dd02
Merge branch 'main' of https://github.com/coder/coder into bq/refacto…
BrunoQuaresma Dec 19, 2023
89ec3df
Fix colors
BrunoQuaresma Dec 19, 2023
9ff6ad0
Make gylphi more accessible
BrunoQuaresma Dec 19, 2023
30432e6
Fix tooltip on template settings
BrunoQuaresma Dec 19, 2023
3f20de8
Fix workspace outdated tooltip spacing
BrunoQuaresma Dec 19, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix tooltip on template settings
  • Loading branch information
@BrunoQuaresma
BrunoQuaresma committed Dec 19, 2023
commit 30432e61f1634e59c8b49b010b5bf1bf89379d51
21 changes: 15 additions & 6 deletions site/src/pages/TemplateSettingsPage/TemplateGeneralSettingsPage/TemplateSettingsForm.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,9 @@ import { Stack } from "components/Stack/Stack";
import Checkbox from "@mui/material/Checkbox";
import {
HelpTooltip,
HelpTooltipContent,
HelpTooltipText,
HelpTooltipTrigger,
} from "components/HelpTooltip/HelpTooltip";
import { EnterpriseBadge } from "components/Badges/Badges";

Expand Down Expand Up @@ -161,9 +163,13 @@ export const TemplateSettingsForm: FC<TemplateSettingsForm> = ({
>
Allow users to cancel in-progress workspace jobs.
<HelpTooltip>
<HelpTooltipText>
If checked, users may be able to corrupt their workspace.
</HelpTooltipText>
<HelpTooltipTrigger />
<HelpTooltipContent>
<HelpTooltipText>
If checked, users may be able to corrupt their
workspace.
</HelpTooltipText>
</HelpTooltipContent>
</HelpTooltip>
</Stack>
<span css={styles.optionHelperText}>
Expand Down Expand Up @@ -193,9 +199,12 @@ export const TemplateSettingsForm: FC<TemplateSettingsForm> = ({
>
Require workspaces automatically update when started.
<HelpTooltip>
<HelpTooltipText>
This setting is not enforced for template admins.
</HelpTooltipText>
<HelpTooltipTrigger />
<HelpTooltipContent>
<HelpTooltipText>
This setting is not enforced for template admins.
</HelpTooltipText>
</HelpTooltipContent>
</HelpTooltip>
</Stack>
<span css={styles.optionHelperText}>
Expand Down