[PR #961] [MERGED] UI/comment threads fixes #1040

Closed
opened 2026-03-03 00:24:22 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BoostIO/BoostNote-App/pull/961
Author: @ButteryCrumpet
Created: 5/24/2021
Status: Merged
Merged: 5/25/2021
Merged by: @ButteryCrumpet

Base: masterHead: ui/comment-threads-fixes


📝 Commits (10+)

  • d2487b4 order by last replied
  • e1fb185 always show context summary
  • 3390dad use live user for UserIcon
  • 8d3a0f8 set closed comments to outdated also
  • 790b557 move store state injection up the tree
  • a0d8dca status filter dropdown select
  • 4be1e22 re-inline specific func
  • 01614aa remove unused imports
  • 2d75f83 remove console log
  • 7e0ad7e more specific css selectors to avoid collision

📊 Changes

9 files changed (+299 additions, -61 deletions)

View changed files

src/cloud/components/atoms/ThreadStatusFilterControl.tsx (+114 -0)
📝 src/cloud/components/molecules/Editor/index.tsx (+46 -4)
📝 src/cloud/components/molecules/ThreadActionButton.tsx (+2 -7)
📝 src/cloud/components/molecules/ThreadItem.tsx (+3 -11)
src/cloud/components/molecules/ThreadList.tsx (+57 -0)
📝 src/cloud/components/organisms/CommentManager.tsx (+45 -39)
📝 src/shared/components/atoms/Button.tsx (+4 -0)
📝 src/shared/components/atoms/Icon.tsx (+12 -0)
📝 src/shared/lib/utils/comments.ts (+16 -0)

📄 Description

FIxes

  • Correctly display user icon
  • sort by latest reply
  • Correctly set Closed threads to outdated
  • Add thread context summary to thread list

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/BoostIO/BoostNote-App/pull/961 **Author:** [@ButteryCrumpet](https://github.com/ButteryCrumpet) **Created:** 5/24/2021 **Status:** ✅ Merged **Merged:** 5/25/2021 **Merged by:** [@ButteryCrumpet](https://github.com/ButteryCrumpet) **Base:** `master` ← **Head:** `ui/comment-threads-fixes` --- ### 📝 Commits (10+) - [`d2487b4`](https://github.com/BoostIO/BoostNote-App/commit/d2487b46f7d47bc9f7a7f454e31cb1a68e1eda96) order by last replied - [`e1fb185`](https://github.com/BoostIO/BoostNote-App/commit/e1fb185f86c85917c8343e17900cad6ce5a09e38) always show context summary - [`3390dad`](https://github.com/BoostIO/BoostNote-App/commit/3390dadd8c5cc7d858a4948cce7f121e589d0544) use live user for UserIcon - [`8d3a0f8`](https://github.com/BoostIO/BoostNote-App/commit/8d3a0f8a0bc86acfe7fa23af12726fd4ebe11550) set closed comments to outdated also - [`790b557`](https://github.com/BoostIO/BoostNote-App/commit/790b55705812c6be12728a6674fc94816bb05dd8) move store state injection up the tree - [`a0d8dca`](https://github.com/BoostIO/BoostNote-App/commit/a0d8dca874c4057f02c63fed682805369ea6565d) status filter dropdown select - [`4be1e22`](https://github.com/BoostIO/BoostNote-App/commit/4be1e22e782e59d38688429550120c70bc87c776) re-inline specific func - [`01614aa`](https://github.com/BoostIO/BoostNote-App/commit/01614aa76b144211cf76c90c21b135e17da1f5f7) remove unused imports - [`2d75f83`](https://github.com/BoostIO/BoostNote-App/commit/2d75f836420bfc12c52bb050c8c5a9b0f4a988d6) remove console log - [`7e0ad7e`](https://github.com/BoostIO/BoostNote-App/commit/7e0ad7e86d6336fdc10557970c1cbc20169aab8b) more specific css selectors to avoid collision ### 📊 Changes **9 files changed** (+299 additions, -61 deletions) <details> <summary>View changed files</summary> ➕ `src/cloud/components/atoms/ThreadStatusFilterControl.tsx` (+114 -0) 📝 `src/cloud/components/molecules/Editor/index.tsx` (+46 -4) 📝 `src/cloud/components/molecules/ThreadActionButton.tsx` (+2 -7) 📝 `src/cloud/components/molecules/ThreadItem.tsx` (+3 -11) ➕ `src/cloud/components/molecules/ThreadList.tsx` (+57 -0) 📝 `src/cloud/components/organisms/CommentManager.tsx` (+45 -39) 📝 `src/shared/components/atoms/Button.tsx` (+4 -0) 📝 `src/shared/components/atoms/Icon.tsx` (+12 -0) 📝 `src/shared/lib/utils/comments.ts` (+16 -0) </details> ### 📄 Description ### FIxes - Correctly display user icon - sort by latest reply - Correctly set `Closed` threads to outdated - Add thread context summary to thread list --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-03 00:24:22 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/BoostNote-App#1040
No description provided.