[PR #40] [MERGED] Add comment collapsing #64

New issue

Closed

opened 2026-03-14 12:30:08 +03:00 by kerem · 0 comments

kerem commented

2026-03-14 12:30:08 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/aome510/hackernews-TUI/pull/40
Author: @aome510
Created: 6/13/2021
Status: ✅ Merged
Merged: 6/13/2021
Merged by: @aome510

Base: main ← Head: add-comment-collapsing

📝 Commits (9)

9c876c4 add initial support for comment collapsing
038c6d9 collapse comment with minimized info kind works
5ad121d collapse all child comments of the currently focused comment works
3200a1a implement toggle_collapse_child_comments, and find_comment_id_by_max_height
58c3e33 change collapse_comment command name, fix linter warning
c5e92da add parent_comment command
631ea68 update README, help_view manual
02e0255 add more documentation
f59e372 fix TODO roadmap in README

📊 Changes

7 files changed (+207 additions, -49 deletions)

View changed files

📝 README.md (+4 -2)
📝 hackernews_tui/src/hn-tui-default.toml (+2 -0)
📝 hackernews_tui/src/keybindings.rs (+4 -0)
📝 hackernews_tui/src/view/comment_view.rs (+165 -43)
📝 hackernews_tui/src/view/help_view.rs (+14 -4)
📝 hackernews_tui/src/view/list_view.rs (+10 -0)
📝 hackernews_tui/src/view/text_view.rs (+8 -0)

📄 Description

implement comment collpasing
add parent_comment navigation

_{🔄 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/aome510/hackernews-TUI/pull/40 **Author:** [@aome510](https://github.com/aome510) **Created:** 6/13/2021 **Status:** ✅ Merged **Merged:** 6/13/2021 **Merged by:** [@aome510](https://github.com/aome510) **Base:** `main` ← **Head:** `add-comment-collapsing` --- ### 📝 Commits (9) - [`9c876c4`](https://github.com/aome510/hackernews-TUI/commit/9c876c4b2bb30062930ea1b5644ef0f7418b6455) add initial support for comment collapsing - [`038c6d9`](https://github.com/aome510/hackernews-TUI/commit/038c6d9caaa3326a65fbcd6bc4b4eb28421d3d19) collapse comment with minimized info kind works - [`5ad121d`](https://github.com/aome510/hackernews-TUI/commit/5ad121df6f5918acd11947448d4e2e365f89b196) collapse all child comments of the currently focused comment works - [`3200a1a`](https://github.com/aome510/hackernews-TUI/commit/3200a1a64c3eb27e5192566865fb42e2bf9b5dcf) implement toggle_collapse_child_comments, and find_comment_id_by_max_height - [`58c3e33`](https://github.com/aome510/hackernews-TUI/commit/58c3e33535a1b13752144d0c8379acb5d2b98c03) change collapse_comment command name, fix linter warning - [`c5e92da`](https://github.com/aome510/hackernews-TUI/commit/c5e92da849523b4a9cbf9506c4eee345422ad3db) add parent_comment command - [`631ea68`](https://github.com/aome510/hackernews-TUI/commit/631ea68796466e7b3ce7f55062d2d8bb673504b3) update README, help_view manual - [`02e0255`](https://github.com/aome510/hackernews-TUI/commit/02e0255c5291cfdfab21ed04f89a59d0c7bc75a5) add more documentation - [`f59e372`](https://github.com/aome510/hackernews-TUI/commit/f59e372da7d4a68bca40d4aaa829fe0faeabf56e) fix TODO roadmap in README ### 📊 Changes **7 files changed** (+207 additions, -49 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+4 -2) 📝 `hackernews_tui/src/hn-tui-default.toml` (+2 -0) 📝 `hackernews_tui/src/keybindings.rs` (+4 -0) 📝 `hackernews_tui/src/view/comment_view.rs` (+165 -43) 📝 `hackernews_tui/src/view/help_view.rs` (+14 -4) 📝 `hackernews_tui/src/view/list_view.rs` (+10 -0) 📝 `hackernews_tui/src/view/text_view.rs` (+8 -0) </details> ### 📄 Description - implement comment collpasing - add `parent_comment` navigation --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>