Skip to content

Simplify nested tag layout - #23

Merged
Clearner1 merged 1 commit into
mainfrom
codex/simplify-tag-tree
Aug 12, 2026
Merged

Simplify nested tag layout#23
Clearner1 merged 1 commit into
mainfrom
codex/simplify-tag-tree

Conversation

@Clearner1

Copy link
Copy Markdown
Owner

What changed

  • remove the decorative oval from tag rows while preserving pin controls and pinned tags
  • remove duplicate computed indentation and use one 12px nesting step
  • keep tag names on one line with ellipsis and expose the full path as a title
  • make browser favicon and in-app logo assets transparent
  • preserve the existing warm-white Apple touch and PWA PNG assets
  • bump icon references to refresh cached branding

Root cause

Nested tag rows accumulated both recursive container margins and depth-based inline padding. A fourth-level child therefore lost enough horizontal space to wrap Chinese text vertically.

Validation

  • npm run check
  • 9 test files / 38 tests passed
  • focused tag and icon assertions passed
  • all eight brand assets are present in the production build
  • transparent 32px favicon and opaque Apple touch icon visually inspected

@Clearner1
Clearner1 merged commit c77ae90 into main Aug 12, 2026
1 check passed
@Clearner1
Clearner1 deleted the codex/simplify-tag-tree branch August 14, 2026 05:09
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