Skip to content

✨ feat: Add sorting functionality and improve responsive layout for book entries#48

Merged
james20140802 merged 3 commits into
mainfrom
feature/sort-entries
Mar 8, 2026
Merged

✨ feat: Add sorting functionality and improve responsive layout for book entries#48
james20140802 merged 3 commits into
mainfrom
feature/sort-entries

Conversation

@james20140802

Copy link
Copy Markdown
Owner

📝 Summary

This PR introduces sorting and filtering capabilities for the book entries on the Book Detail page, while significantly improving the layout and UI across both desktop and mobile environments. It also addresses a hydration error caused by invalid HTML nesting.

✨ Key Changes

  • Sorting & Filtering Functionality:
    • Implemented real-time dropdown and tab filtering functionality for entries (All, Public, Private).
    • Added sorting logic to display entries by asc (oldest) or desc (newest).
  • Responsive UI Improvements:
    • Revamped BookDetailContent layout: On desktop, the title, filters, and action buttons align smoothly on a single line. On mobile, components stack dynamically for a more compact and mobile-friendly view.
    • Refined the style of filter tabs and the sorting dropdown button so heights and paddings are fully uniform.
    • The "기록 추가하기" (Add Entry) button's text correctly adjusts its length on smaller screens to prevent visual clutter.
  • Bug Fix: Resolves a React hydration error caused by a nested anchor (<a>) tag inside components/books/BookList.tsx

✅ Review Notes

  • The visibility filters (Public/Private) are now correctly hidden when viewing a friend's profile, as only public entries are loaded for social viewing anyway.

@vercel

vercel Bot commented Mar 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
readiary Ready Ready Preview, Comment Mar 8, 2026 5:01am

@james20140802 james20140802 merged commit 6494cf7 into main Mar 8, 2026
2 of 3 checks passed
@james20140802 james20140802 deleted the feature/sort-entries branch March 8, 2026 05:04
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