Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit ec161cd

Browse files
aqandrewaqandrew
committed
chore(site): enable jsx-curly-brace-presence oxlint rule
1 parent d726903 commit ec161cd

4 files changed

Lines changed: 49 additions & 51 deletions

File tree

‎site/.oxlintrc.jsonc‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,7 @@
134134
"one-var": ["error", "never"],
135135
"prefer-number-properties": ["error", { "checkInfinity": true }],
136136
"no-else-return": ["error", { "allowElseIf": false }],
137+
"jsx-curly-brace-presence": "error",
137138

138139
// --- suspicious ---
139140
"no-cond-assign": "error",
@@ -207,7 +208,6 @@
207208
"no-invalid-void-type": "off", // suspicious/noConfusingVoidType (large)
208209
"no-redeclare": "off", // suspicious/noRedeclare (medium: TS declaration merging)
209210
"no-inferrable-types": "off", // style/noInferrableTypes (medium, coder error rule)
210-
"jsx-curly-brace-presence": "off", // style/useConsistentCurlyBraces (medium, coder error rule)
211211
"no-fallthrough": "off", // suspicious/noFallthroughSwitchClause (medium)
212212
"click-events-have-key-events": "off", // a11y/useKeyWithClickEvents (small)
213213
"prefer-function-type": "off", // style/useShorthandFunctionType (small)

‎site/src/components/TagInput/TagInput.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ export const TagInput: FC<TagInputProps> = ({
8383
</div>
8484

8585
<p className="text-content-secondary text-xs mt-1">
86-
{'Type "," to separate the values'}
86+
Type "," to separate the values
8787
</p>
8888
</div>
8989
);

‎site/src/modules/resources/AgentExternal.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ export const AgentExternal: FC<AgentExternalProps> = ({ agent, workspace }) => {
3737
code={credentials?.command ?? ""}
3838
secret={false}
3939
redactPattern={/CODER_AGENT_TOKEN="([^"]+)"/g}
40-
redactReplacement={`CODER_AGENT_TOKEN="********"`}
40+
redactReplacement='CODER_AGENT_TOKEN="********"'
4141
showRevealButton
4242
/>
4343
</section>

‎site/src/pages/TemplatePage/TemplateInsightsPage/TemplateInsightsPage.tsx‎

Lines changed: 46 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -442,57 +442,55 @@ const TemplateUsagePanel: FC<TemplateUsagePanelProps> = ({
442442
<PanelTitle>App & IDE Usage</PanelTitle>
443443
</PanelHeader>
444444
<PanelContent error={error} data={validUsage}>
445-
{
446-
<div className="flex flex-col gap-6">
447-
{(validUsage || []).map((usage, i) => {
448-
const percentage = (usage.seconds / totalInSeconds) * 100;
449-
const colorStop =
450-
usageCount <= 1 ? 0 : (i / (usageCount - 1)) * 100;
451-
return (
452-
<div key={usage.slug} className="flex items-center gap-6">
453-
<div className="flex items-center gap-2">
454-
<div className="flex justify-center items-center size-5">
455-
<ExternalImage
456-
src={usage.icon}
457-
alt=""
458-
className="h-full w-full object-contain"
459-
/>
460-
</div>
461-
<div className="text-sm font-medium w-[200px]">
462-
{usage.display_name}
463-
</div>
445+
<div className="flex flex-col gap-6">
446+
{(validUsage || []).map((usage, i) => {
447+
const percentage = (usage.seconds / totalInSeconds) * 100;
448+
const colorStop =
449+
usageCount <= 1 ? 0 : (i / (usageCount - 1)) * 100;
450+
return (
451+
<div key={usage.slug} className="flex items-center gap-6">
452+
<div className="flex items-center gap-2">
453+
<div className="flex justify-center items-center size-5">
454+
<ExternalImage
455+
src={usage.icon}
456+
alt=""
457+
className="h-full w-full object-contain"
458+
/>
464459
</div>
465-
<Tooltip>
466-
<TooltipTrigger asChild>
467-
<div className="relative w-full h-2 rounded-full bg-surface-quaternary">
468-
<div
469-
className="absolute inset-y-0 left-0 rounded-full"
470-
style={{
471-
width: `${percentage}%`,
472-
backgroundColor: `color-mix(in lch, var(--color-content-success), var(--color-content-warning) ${colorStop}%)`,
473-
}}
474-
/>
475-
</div>
476-
</TooltipTrigger>
477-
<TooltipContent>
478-
{Math.floor(percentage)}%
479-
<TooltipArrow className="fill-border" />
480-
</TooltipContent>
481-
</Tooltip>
482-
<div className="flex flex-col text-sm font-normal shrink-0 leading-normal text-content-secondary w-[120px]">
483-
{formatTime(usage.seconds)}
484-
{usage.times_used > 0 && (
485-
<span className="text-[12px] text-content-disabled">
486-
Opened {usage.times_used.toLocaleString()}{" "}
487-
{usage.times_used === 1 ? "time" : "times"}
488-
</span>
489-
)}
460+
<div className="text-sm font-medium w-[200px]">
461+
{usage.display_name}
490462
</div>
491463
</div>
492-
);
493-
})}
494-
</div>
495-
}
464+
<Tooltip>
465+
<TooltipTrigger asChild>
466+
<div className="relative w-full h-2 rounded-full bg-surface-quaternary">
467+
<div
468+
className="absolute inset-y-0 left-0 rounded-full"
469+
style={{
470+
width: `${percentage}%`,
471+
backgroundColor: `color-mix(in lch, var(--color-content-success), var(--color-content-warning) ${colorStop}%)`,
472+
}}
473+
/>
474+
</div>
475+
</TooltipTrigger>
476+
<TooltipContent>
477+
{Math.floor(percentage)}%
478+
<TooltipArrow className="fill-border" />
479+
</TooltipContent>
480+
</Tooltip>
481+
<div className="flex flex-col text-sm font-normal shrink-0 leading-normal text-content-secondary w-[120px]">
482+
{formatTime(usage.seconds)}
483+
{usage.times_used > 0 && (
484+
<span className="text-[12px] text-content-disabled">
485+
Opened {usage.times_used.toLocaleString()}{" "}
486+
{usage.times_used === 1 ? "time" : "times"}
487+
</span>
488+
)}
489+
</div>
490+
</div>
491+
);
492+
})}
493+
</div>
496494
</PanelContent>
497495
</Panel>
498496
);

0 commit comments

Comments
 (0)