← 输入框|github.com|刷新
Skip to content

web | scripts: border letter team icons, block dev scripts in worktrees - #16

Merged
yekta merged 1 commit into
masterfrom
fix/logo-text-gap
Oct 17, 2026
Merged

yekta merged 1 commit into
masterfrom
fix/logo-text-gap

Conversation

@yekta

@yekta yekta commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

Web

  • Generated letter icons now use the same bordered circle as uploaded team icons. The background is bg-card, the border is border-border and the letter uses the foreground color, so both kinds of icon look the same in light and dark mode.
  • If an uploaded icon fails to load, the letter now shows inside the same container, so the size and border stay the same.
  • The breadcrumb label sits a little closer to its icon (-mr-0.5), which tightens the gap between the logo and the text.

Scripts

  • dev.sh and the start/reset commands in dev-env.sh now refuse to run from a git worktree. The dev containers are shared and use a relative data path, so running them from a worktree points them at the wrong data. The error message gives the path of the main checkout.
  • dev-cluster.sh now stops k3d cluster start after 180s. After a hard reboot, a corrupt Docker log on the server can leave k3d waiting forever. On timeout the script prints the command that clears that log, then exits.

Testing

Checked the icon changes visually in both light and dark mode.

- Generated letter icons now use the same bordered circle as uploaded icons, with a bg-card background and foreground text, so the two look alike in light and dark mode.
- Pull the breadcrumb label slightly closer to its icon.
- dev.sh and dev-env.sh start/reset now refuse to run from a git worktree. The dev containers are shared and use a relative data path, so running from a worktree points them at the wrong data.
- Cap k3d cluster start at 180s. If it times out, print how to clear the corrupt server Docker log that can leave it hanging after a hard reboot.
@yekta
yekta merged commit 64bfc72 into master Oct 17, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant