Skip to content

Hide Uncategorized when a book has no parts#20

Merged
catherineluse merged 1 commit into
mainfrom
codex/hide-empty-uncategorized
Jul 16, 2026
Merged

Hide Uncategorized when a book has no parts#20
catherineluse merged 1 commit into
mainfrom
codex/hide-empty-uncategorized

Conversation

@catherineluse

Copy link
Copy Markdown
Collaborator

Summary

  • show unassigned chapters directly in the desktop book sidebar when the book has no parts
  • retain the “Uncategorized” group once at least one part exists
  • prevent the mobile book detail view from showing both an Uncategorized section and a second plain chapter list when there are no parts

Why

“Uncategorized” only describes a meaningful distinction when a book contains at least one part. For books without parts, the chapters are simply the book’s chapter list.

User impact

Books with no parts now have a cleaner chapter list with no unnecessary category label. Books that use parts keep the existing Uncategorized group for chapters that have not been assigned.

Validation

  • 143 runnable tests passed
  • TypeScript check passed
  • ESLint passed
  • production Vite build passed
  • git diff --check passed

Notes

  • no data or schema changes
  • no environment variable changes

@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ai-beta-reader-frontend Ready Ready Preview, Comment Jul 16, 2026 4:50am

Request Review

@catherineluse
catherineluse marked this pull request as ready for review July 16, 2026 04:53
@catherineluse
catherineluse merged commit e926c0d into main Jul 16, 2026
3 checks passed
@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/components/book/BookDesktopChapterSidebar.vue 0.00% 2 Missing ⚠️
src/components/book/BookMobileSection.vue 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

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.

2 participants