mirror of
https://github.com/aome510/hackernews-TUI.git
synced 2026-04-26 09:25:56 +03:00
[PR #33] [MERGED] [no-issue] implement article view #57
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/hackernews-TUI#57
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/aome510/hackernews-TUI/pull/33
Author: @aome510
Created: 4/28/2021
Status: ✅ Merged
Merged: 4/29/2021
Merged by: @aome510
Base:
main← Head:implement-article-view📝 Commits (10+)
1626d31add article_viewc9eecf0add serde_json8f26169add error handling in get_article_view_async45aae3eadd Article struct231b9daadd keybindings to open artcile/url in article_view3ef4f48add add_comment_view_layerbaebf92wrap article_view with status_bar, footer2c9597fadd j,k,d,u for article_view583e14dadd regexs to parse links/images from markdown raw textacf9555update url regex to greedy matching📊 Changes
16 files changed (+693 additions, -50 deletions)
View changed files
📝
Cargo.lock(+2 -0)📝
Cargo.toml(+2 -0)📝
README.md(+28 -4)📝
src/config.rs(+4 -0)📝
src/hn-tui-default.toml(+16 -0)📝
src/keybindings.rs(+45 -0)📝
src/prelude.rs(+1 -0)➕
src/view/article_view.rs(+361 -0)📝
src/view/async_view.rs(+45 -4)📝
src/view/comment_view.rs(+61 -22)📝
src/view/error_view.rs(+2 -2)📝
src/view/help_view.rs(+98 -7)📝
src/view/list_view.rs(+4 -2)📝
src/view/mod.rs(+1 -0)📝
src/view/search_view.rs(+7 -4)📝
src/view/story_view.rs(+16 -5)📄 Description
Add
ArticleViewto read articles in reader mode directly in the application with the help of mercury-parserBrief description of changes
ArticleViewand its shortcutsArticleViewREADMEserde_jsonpackage to parse json bytes toArticlestructurlpackage🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.