mirror of
https://github.com/coder/coder.git
synced 2026-09-24 15:04:27 +08:00
chore: fix idle state icon when disabled (#18554)
When the workspace is off, we set a disabled text/stroke color, but for the idle icon that also needs a fill, this only changed the outline making it look weird. Instead, move the disabled logic into the component so we can apply a matching fill. I felt it looked too thick with both the outline and fill, so I also removed the outline. Really I think maybe the workspace status should be a separate column rather than disabling these icons, but this maintains the status quo. Before with mismatching stroke and fill color:  After with disabled fill and stroke removal:  Enabled fill and stroke removal: 
This commit is contained in:
@@ -69,9 +69,43 @@ export const LongMessage: Story = {
|
||||
},
|
||||
};
|
||||
|
||||
export const Disabled: Story = {
|
||||
export const DisabledComplete: Story = {
|
||||
args: {
|
||||
status: MockWorkspaceAppStatus,
|
||||
disabled: true,
|
||||
},
|
||||
};
|
||||
|
||||
export const DisabledFailure: Story = {
|
||||
args: {
|
||||
status: {
|
||||
...MockWorkspaceAppStatus,
|
||||
state: "failure",
|
||||
message: "Couldn't figure out how to start the dev server",
|
||||
},
|
||||
disabled: true,
|
||||
},
|
||||
};
|
||||
|
||||
export const DisabledWorking: Story = {
|
||||
args: {
|
||||
status: {
|
||||
...MockWorkspaceAppStatus,
|
||||
state: "working",
|
||||
message: "Starting dev server...",
|
||||
uri: "",
|
||||
},
|
||||
disabled: true,
|
||||
},
|
||||
};
|
||||
|
||||
export const DisabledIdle: Story = {
|
||||
args: {
|
||||
status: {
|
||||
...MockWorkspaceAppStatus,
|
||||
state: "idle",
|
||||
message: "Done for now",
|
||||
},
|
||||
disabled: true,
|
||||
},
|
||||
};
|
||||
|
||||
@@ -7,7 +7,6 @@ import {
|
||||
} from "components/Tooltip/Tooltip";
|
||||
import capitalize from "lodash/capitalize";
|
||||
import { AppStatusStateIcon } from "modules/apps/AppStatusStateIcon";
|
||||
import { cn } from "utils/cn";
|
||||
|
||||
type WorkspaceAppStatusProps = {
|
||||
status: APIWorkspaceAppStatus | null;
|
||||
@@ -37,9 +36,6 @@ export const WorkspaceAppStatus = ({
|
||||
latest
|
||||
disabled={disabled}
|
||||
state={status.state}
|
||||
className={cn({
|
||||
"text-content-disabled": disabled,
|
||||
})}
|
||||
/>
|
||||
<span className="whitespace-nowrap max-w-72 overflow-hidden text-ellipsis text-sm text-content-primary font-medium">
|
||||
{message}
|
||||
|
||||
Reference in New Issue
Block a user