Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
Original file line number Diff line number Diff line change
Expand Up @@ -420,7 +420,12 @@ function InstanceSessionManagement(
props.onProjectChange(value as ProjectId);
}}
>
<SelectTrigger aria-label="Project for ACP sessions" className="w-40" size="sm">
<SelectTrigger
variant="well"
aria-label="Project for ACP sessions"
className="w-40"
size="compact"
>
<SelectValue>
{props.projects.find((project) => project.id === projectId)?.title}
</SelectValue>
Expand Down
1 change: 1 addition & 0 deletions apps/web/src/components/settings/AddCodexAccountDialog.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,7 @@ export function AddCodexAccountDialog({
description="Shown in the provider list and model picker."
control={
<Input
size="settings"
aria-label="Account name"
value={name}
disabled={pending}
Expand Down
43 changes: 22 additions & 21 deletions apps/web/src/components/settings/AddEnvironmentDialog.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ import {
import {
Dialog,
DialogDescription,
DialogFooter,
DialogHeader,
DialogPanel,
DialogPopup,
Expand Down Expand Up @@ -471,15 +472,6 @@ export function AddEnvironmentDialog({
) : null}
{renderRemoteFields()}
{error ? <p className="text-meta text-destructive">{error}</p> : null}
<Button
variant="secondary"
className="w-full"
disabled={isAddingSavedBackend}
onClick={() => void handleAddSavedBackend()}
>
<PlusIcon className="size-3.5" />
{isAddingSavedBackend ? "Adding…" : routeTarget ? "Add route" : "Add environment"}
</Button>
</div>
);
const renderSshFields = () => (
Expand Down Expand Up @@ -590,15 +582,6 @@ export function AddEnvironmentDialog({
<AlertDescription>{error ?? discoveredSshHostsError}</AlertDescription>
</Alert>
) : null}
<Button
variant="secondary"
className="w-full"
disabled={isAddingSavedBackend}
onClick={() => void handleAddSavedBackend()}
>
<PlusIcon className="size-3.5" />
{isAddingSavedBackend ? "Adding…" : routeTarget ? "Add route" : "Add environment"}
</Button>
</div>
</div>
);
Expand All @@ -610,7 +593,7 @@ export function AddEnvironmentDialog({
render={
<DialogTrigger
render={
<Button size="xs" variant="secondary" aria-label="Add environment">
<Button size="standard" variant="secondary" aria-label="Add environment">
<PlusIcon className="size-3" />
<span>Add environment</span>
</Button>
Expand All @@ -623,7 +606,7 @@ export function AddEnvironmentDialog({
<DialogPopup className="max-h-[80dvh] sm:max-w-3xl">
<DialogHeader>
<DialogTitle>
{routeTarget ? `Add a route to ${routeTarget.label}` : "Add Environment"}
{routeTarget ? `Add a route to ${routeTarget.label}` : "Add environment"}
</DialogTitle>
<DialogDescription>
{routeTarget
Expand All @@ -633,7 +616,7 @@ export function AddEnvironmentDialog({
</DialogHeader>
<DialogPanel>
<div className="space-y-4">
<div className="grid gap-3 sm:grid-cols-2">
<div className={cn("grid gap-3", desktopBridge && "sm:grid-cols-2")}>
{renderConnectionModeCard({
mode: "remote",
title: "Remote link",
Expand All @@ -653,6 +636,24 @@ export function AddEnvironmentDialog({
{savedBackendMode === "ssh" ? renderSshFields() : renderRemoteModeBody()}
</AnimatedHeight>
</div>
<DialogFooter variant="bare" className="-mx-5 -mb-4">
<Button
size="standard"
variant="secondary"
disabled={isAddingSavedBackend}
onClick={() => handleOpenChange(false)}
>
Cancel
</Button>
<Button
size="standard"
disabled={isAddingSavedBackend}
onClick={() => void handleAddSavedBackend()}
>
<PlusIcon className="size-3.5" />
{isAddingSavedBackend ? "Adding…" : routeTarget ? "Add route" : "Add environment"}
</Button>
</DialogFooter>
</DialogPanel>
</DialogPopup>
</Dialog>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -639,7 +639,7 @@ export function AddProviderInstanceDialog({
<Input
id="add-provider-label"
aria-describedby="add-provider-label-description"
size="sm"
size="settings"
className="w-full @min-[26rem]/settings-row:w-56"
placeholder="e.g. Work"
value={label}
Expand Down Expand Up @@ -673,7 +673,7 @@ export function AddProviderInstanceDialog({
? "add-provider-instance-id-description add-provider-instance-id-error"
: "add-provider-instance-id-description"
}
size="sm"
size="settings"
className="w-full @min-[26rem]/settings-row:w-56"
placeholder={`${driver}_work`}
value={instanceId}
Expand Down
14 changes: 12 additions & 2 deletions apps/web/src/components/settings/AppearanceSettingsPanel.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -263,7 +263,12 @@ export function AppearanceSettingsPanel() {
updateSettings({ chatWidth: value });
}}
>
<SelectTrigger size="sm" className="w-full min-w-0" aria-label="Chat width">
<SelectTrigger
variant="well"
size="compact"
className="w-full min-w-0"
aria-label="Chat width"
>
<SelectValue>{CHAT_WIDTH_LABELS[settings.chatWidth]}</SelectValue>
</SelectTrigger>
<SelectPopup align="end" alignItemWithTrigger={false}>
Expand Down Expand Up @@ -298,7 +303,12 @@ export function AppearanceSettingsPanel() {
updateSettings({ effortDisplay: value });
}}
>
<SelectTrigger size="sm" className="w-full min-w-0" aria-label="Effort display">
<SelectTrigger
variant="well"
size="compact"
className="w-full min-w-0"
aria-label="Effort display"
>
<SelectValue>{EFFORT_DISPLAY_LABELS[settings.effortDisplay]}</SelectValue>
</SelectTrigger>
<SelectPopup align="end" alignItemWithTrigger={false}>
Expand Down
16 changes: 11 additions & 5 deletions apps/web/src/components/settings/AuthorizedClientsSettings.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -386,11 +386,15 @@ const PairingLinkListRow = memo(function PairingLinkListRow({
) : null}
</DialogPanel>
<DialogFooter variant="bare">
<Button variant="secondary" onClick={() => setIsRevealDialogOpen(false)}>
<Button
size="standard"
variant="secondary"
onClick={() => setIsRevealDialogOpen(false)}
>
Done
</Button>
{canCopyToClipboard ? (
<Button variant="secondary" onClick={handleCopyCode}>
<Button size="standard" variant="secondary" onClick={handleCopyCode}>
Copy code
</Button>
) : null}
Expand Down Expand Up @@ -682,7 +686,7 @@ export const AuthorizedClientsHeaderAction = memo(function AuthorizedClientsHead
if (part === "revoke") {
return (
<Button
size="xs"
size="standard"
variant="destructive-outline"
disabled={isRevokingOtherClients || !canRevokeOtherClients(clientSessions)}
onClick={() => void onRevokeOtherClients()}
Expand All @@ -696,7 +700,7 @@ export const AuthorizedClientsHeaderAction = memo(function AuthorizedClientsHead
<div className="flex items-center gap-2">
{pairingDenial !== undefined ? (
<ManagedControl reason={pairingDenial}>
<Button size="xs" variant="default" disabled>
<Button size="standard" variant="default" disabled>
<QrCodeIcon />
Create pairing link
</Button>
Expand All @@ -717,7 +721,7 @@ export const AuthorizedClientsHeaderAction = memo(function AuthorizedClientsHead
{pairingDenial === undefined ? (
<DialogTrigger
render={
<Button size="xs" variant="default">
<Button size="standard" variant="default">
<QrCodeIcon />
Create pairing link
</Button>
Expand Down Expand Up @@ -821,13 +825,15 @@ export const AuthorizedClientsHeaderAction = memo(function AuthorizedClientsHead
</DialogPanel>
<DialogFooter variant="bare">
<Button
size="standard"
variant="secondary"
disabled={isCreatingPairingLink}
onClick={() => setDialogOpen(false)}
>
Cancel
</Button>
<Button
size="standard"
disabled={isCreatingPairingLink || selectedScopes.length === 0}
onClick={() => void handleCreatePairingLink()}
>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,8 @@ function BackgroundActivityAdvancedDialog({
}}
>
<SelectTrigger
size="sm"
variant="well"
size="compact"
className="w-full sm:w-40"
aria-label="Shared background policy"
>
Expand Down Expand Up @@ -385,7 +386,8 @@ export function BackgroundActivitySettingsRow() {
}}
>
<SelectTrigger
size="sm"
variant="well"
size="compact"
className="w-full sm:w-40"
aria-label="Background activity profile"
>
Expand Down
3 changes: 2 additions & 1 deletion apps/web/src/components/settings/BranchNamingSettings.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ export function BranchNamingSettings() {
}
}}
>
<SelectTrigger size="sm" aria-label="Worktree branch naming">
<SelectTrigger variant="well" size="compact" aria-label="Worktree branch naming">
<SelectValue>
{(value: BranchNamingMode | null) => (value === null ? "Mixed" : MODES[value])}
</SelectValue>
Expand Down Expand Up @@ -95,6 +95,7 @@ export function BranchNamingSettings() {
}
control={
<Input
size="settings"
key={`${scopeKey}:${prefixMixed}:${settings.branchNamePrefix}`}
aria-label="Branch prefix"
autoCapitalize="none"
Expand Down
3 changes: 3 additions & 0 deletions apps/web/src/components/settings/CodexSetupSection.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -896,6 +896,7 @@ export function CodexManagedRuntimeFields({
control={
<div className="w-full sm:w-80">
<Input
size="settings"
aria-label="Codex binary path"
value={executablePath}
title={executablePath}
Expand All @@ -911,6 +912,7 @@ export function CodexManagedRuntimeFields({
control={
<div className="w-full sm:w-80">
<Input
size="settings"
aria-label="Codex home path"
value={provider?.runtimePaths?.homePath ?? ""}
title={provider?.runtimePaths?.homePath}
Expand All @@ -930,6 +932,7 @@ export function CodexManagedRuntimeFields({
control={
<div className="w-full sm:w-80">
<Input
size="settings"
aria-label="Codex shadow home path"
value={provider?.runtimePaths?.shadowHomePath ?? ""}
title={provider?.runtimePaths?.shadowHomePath ?? undefined}
Expand Down
3 changes: 2 additions & 1 deletion apps/web/src/components/settings/ConnectionsSettings.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -954,7 +954,8 @@ function ConnectionsSettingsContent({
}}
>
<SelectTrigger
size="sm"
variant="well"
size="compact"
className="w-full sm:w-56"
aria-label="WSL backend"
disabled={isUpdatingWslBackend}
Expand Down
4 changes: 3 additions & 1 deletion apps/web/src/components/settings/DeviceSourcesSettings.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -527,7 +527,9 @@ export function DeviceSourcesSettings(props: {
const addMenu = (
<Menu>
<MenuTrigger
render={<Button size="sm" variant="secondary" disabled={disabled || editing !== null} />}
render={
<Button size="standard" variant="secondary" disabled={disabled || editing !== null} />
}
>
<PlusIcon className="size-3.5" /> Add source
<ChevronDownIcon className="size-3.5" />
Expand Down
5 changes: 4 additions & 1 deletion apps/web/src/components/settings/FontFamilyPicker.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,10 @@ export function FontFamilyPicker({
void listRef.current?.scrollIndexIntoView?.({ index: eventDetails.index, animated: false });
}}
>
<ComboboxTrigger aria-label={ariaLabel} render={<SelectButton size="sm" />}>
<ComboboxTrigger
aria-label={ariaLabel}
render={<SelectButton variant="well" size="compact" />}
>
{selectedFamily.length === 0 ? defaultFamily : selectedFamily}
</ComboboxTrigger>
<ComboboxPopup align="end" className="flex w-72 flex-col">
Expand Down
22 changes: 14 additions & 8 deletions apps/web/src/components/settings/IntegrationsSettings.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,7 @@ function BrowserViewportSetting({ disabled }: { readonly disabled: boolean }) {
control={
<div className="flex w-full flex-wrap items-center justify-end gap-2 sm:w-auto">
<Select value={selectedViewport} onValueChange={selectViewport} disabled={disabled}>
<SelectTrigger size="sm" aria-label="Default browser viewport">
<SelectTrigger variant="well" size="compact" aria-label="Default browser viewport">
<SelectValue>{viewportSelectLabel(viewport)}</SelectValue>
</SelectTrigger>
<SelectPopup align="end" alignItemWithTrigger={false}>
Expand All @@ -402,7 +402,7 @@ function BrowserViewportSetting({ disabled }: { readonly disabled: boolean }) {
max={PREVIEW_VIEWPORT_MAX_DIMENSION}
disabled={disabled}
format={NO_GROUPING}
size="sm"
size="compact"
className="w-20"
onValueCommitted={(value) => commitDimension("width", value)}
>
Expand All @@ -417,7 +417,7 @@ function BrowserViewportSetting({ disabled }: { readonly disabled: boolean }) {
max={PREVIEW_VIEWPORT_MAX_DIMENSION}
disabled={disabled}
format={NO_GROUPING}
size="sm"
size="compact"
className="w-20"
onValueCommitted={(value) => commitDimension("height", value)}
>
Expand Down Expand Up @@ -481,7 +481,8 @@ function BrowserZoomSetting({ disabled }: { readonly disabled: boolean }) {
}}
>
<SelectTrigger
size="sm"
variant="well"
size="compact"
className="w-full sm:w-40"
aria-label="Default browser zoom"
leadingIcon={<ZoomInIcon />}
Expand Down Expand Up @@ -526,7 +527,7 @@ function BrowserAppearanceSetting({ disabled }: { readonly disabled: boolean })
updateSettings({ browserDefaultAppearance: value });
}}
>
<SelectTrigger size="sm" aria-label="Default browser appearance">
<SelectTrigger variant="well" size="compact" aria-label="Default browser appearance">
<SelectValue>{APPEARANCE_LABELS[appearance]}</SelectValue>
</SelectTrigger>
<SelectPopup align="end" alignItemWithTrigger={false}>
Expand Down Expand Up @@ -609,7 +610,7 @@ function BrowserRecordingFrameRateSetting({ disabled }: { readonly disabled: boo
if (next !== undefined) updateSettings({ browserRecordingFrameRate: next });
}}
>
<SelectTrigger size="sm" aria-label="Browser recording frame rate">
<SelectTrigger variant="well" size="compact" aria-label="Browser recording frame rate">
<SelectValue>{frameRate} fps</SelectValue>
</SelectTrigger>
<SelectPopup align="end" alignItemWithTrigger={false}>
Expand Down Expand Up @@ -720,7 +721,12 @@ function BrowserSearchEngineSetting() {
}
}}
>
<SelectTrigger size="sm" className="w-full sm:w-40" aria-label="Search engine">
<SelectTrigger
variant="well"
size="compact"
className="w-full sm:w-40"
aria-label="Search engine"
>
<SelectValue>{SEARCH_ENGINE_LABELS[engine]}</SelectValue>
</SelectTrigger>
<SelectPopup align="end" alignItemWithTrigger={false}>
Expand All @@ -745,7 +751,7 @@ function BrowserSearchEngineSetting() {
control={
<DraftInput
nativeInput
size="sm"
size="settings"
className="w-full sm:w-64"
aria-label="Custom search URL"
placeholder="https://example.com/search?q=%s"
Expand Down
Loading
Loading