Skip to content

Show module code in sidebar instead of name when set - #10

Merged
veniplex merged 1 commit into
mainfrom
claude/module-abbreviation-sidebar
Jul 13, 2026
Merged

veniplex merged 1 commit into
mainfrom
claude/module-abbreviation-sidebar

Conversation

@veniplex

Copy link
Copy Markdown
Owner

Summary

  • Modules already have an optional code field (nullable text column, labeled "Kürzel" in the German locale) — no new field or migration needed.
  • The sidebar's module list now shows code when set, falling back to name otherwise.
  • The full module name stays available on hover (title attribute) and is still used for the context-menu label and aria-labels, so accessibility and the right-click menu are unaffected.

Test plan

  • npx tsc --noEmit
  • npx eslint src/components/layout/app-sidebar.tsx
  • npx prettier --check src/components/layout/app-sidebar.tsx
  • Manual QA: set a module's "Kürzel"/code in the module dialog, confirm the sidebar shows the code instead of the name, and that a module without a code still shows its full name

Generated by Claude Code

Modules already have an optional "code" field (e.g. "M12", labeled
"Kürzel" in German). The sidebar module list showed the full name
regardless; it now prefers the code when present, falling back to the
name otherwise. The full name stays available via a tooltip and is
still used for the context menu label and aria-labels.
@veniplex veniplex added the release-candidate Creates a new release and deployment of docker image, once merged. label Jul 13, 2026
@veniplex
veniplex merged commit 702a89f into main Jul 13, 2026
1 of 2 checks passed
@veniplex
veniplex deleted the claude/module-abbreviation-sidebar branch July 13, 2026 17:36
github-actions Bot added a commit that referenced this pull request Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-candidate Creates a new release and deployment of docker image, once merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants