Skip to content

Unify tag displays and collapse related tags - #102

Merged
mathix420 merged 1 commit into
masterfrom
fix-tags-page
Jul 27, 2026
Merged

Unify tag displays and collapse related tags#102
mathix420 merged 1 commit into
masterfrom
fix-tags-page

Conversation

@mathix420

Copy link
Copy Markdown
Owner

Summary

  • add a shared UiTagList based on the homepage tag pills and reuse it on the homepage, dashboard, issue details, and tag detail pages
  • collapse related tags to two responsive rows with an inline +N more / show less control
  • preserve the dedicated /tags directory design while removing the redundant result-count summary from tag detail pages

Why

Large related-tag clouds dominated tag pages, and tag presentation had drifted across different parts of the app. The first collapse control also sat outside the badge flow, making it feel visually detached.

Impact

Tag pages remain compact by default, users can reveal all related tags on demand, and navigable tag pills now share one consistent component. The collapse control measures wrapping responsively and reserves room inside the second row.

Validation

  • bunx vue-tsc --noEmit
  • bunx vize lint app/components/ui/TagList.vue
  • bun run build
  • visually checked the tag page at desktop and mobile widths

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown

🧹 Code quality

Check Summary
vize lint 0 errors · 315 warnings
vize format all files formatted
fallow audit fail · 0 dead-code · 15 complexity · 1 dupes (9 files)
fallow audit output
           8 cyclomatic    4 cognitive   23 lines
          72.0 CRAP
  app/pages/dashboard.vue
    :71 activity HIGH
           8 cyclomatic    7 cognitive   32 lines
          72.0 CRAP
  app/components/ui/TagList.vue
    :77 updateOverflow HIGH
           7 cyclomatic    6 cognitive   34 lines
          56.0 CRAP
  app/pages/index.vue
    :50 resultGroups HIGH
           7 cyclomatic    3 cognitive    5 lines
          56.0 CRAP
  app/pages/issue/[issueId]/index.vue
    :69 submitInfoResponse
           6 cyclomatic    2 cognitive   24 lines
          42.0 CRAP
    :97 submitAppeal
           6 cyclomatic    2 cognitive   24 lines
          42.0 CRAP
  app/pages/dashboard.vue
    :121 activityVerb
           6 cyclomatic    1 cognitive   16 lines
          42.0 CRAP
  app/components/ui/TagList.vue
    :146 <arrow>
           5 cyclomatic    4 cognitive   23 lines
          30.0 CRAP
  app/pages/dashboard.vue
    :29 firstName
           5 cyclomatic    1 cognitive    1 lines
          30.0 CRAP
  Functions and synthetic template or component entries exceeding cyclomatic, cognitive, or CRAP thresholds (https://docs.fallow.tools/explanations/health#complexity-metrics)
  To suppress inline templates: // fallow-ignore-next-line complexity (above @Component)
  To suppress: // fallow-ignore-next-line complexity

✗ 15 above threshold · 103 analyzed (0.00s)
✗ complexity: 15 findings · duplication: 1 clone group · 8 changed files (0.39s)
  audit gate excluded 11 inherited findings (run with --gate all to enforce)

updated for da42a4d · run #288

@mathix420
mathix420 marked this pull request as ready for review July 27, 2026 10:25
@mathix420
mathix420 merged commit c2acf7f into master Jul 27, 2026
10 of 12 checks passed
@mathix420
mathix420 deleted the fix-tags-page branch July 27, 2026 10:31
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