Merged
Conversation
There was a problem hiding this comment.
Pull request overview
该 PR 为 TUI 目录导航新增“按下 I/i 切换隐藏目录显示”的能力,并在底部新增状态/提示栏展示当前隐藏目录显示状态;同时扩展部分按键支持大写输入,并调整 README 的下载链接指向 latest release。
Changes:
- 在
App中引入show_hidden状态并通过toggle_hidden()触发重新加载目录,按需过滤以.开头的隐藏目录 - 在事件循环中新增
I/i快捷键触发隐藏目录切换,并让Q/H/J/K/L等大写按键与小写等效 - UI 底部增加提示栏;README 下载示例改用
releases/latest
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/app.rs | 增加 show_hidden 状态、在读取目录时按开关过滤隐藏目录,并提供 toggle_hidden() |
| src/main.rs | 增加 I/i 切换隐藏目录;扩展退出/移动按键支持大写 |
| src/ui.rs | 增加底部状态提示栏,展示隐藏目录开关状态与快捷键提示 |
| README.md | 下载链接改为 latest;(目前未同步新增快捷键说明) |
| README_EN.md | 下载链接改为 latest;(目前未同步新增快捷键说明) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
27
to
35
| # Linux | ||
| wget https://github.com/harkerhand/jumping/releases/download/v0.1.0/jumping-linux | ||
| wget https://github.com/harkerhand/jumping/releases/latest/download/jumping-linux | ||
| chmod +x jumping-linux | ||
| sudo mv jumping-linux /usr/local/bin/jumping | ||
|
|
||
| # macOS | ||
| wget https://github.com/harkerhand/jumping/releases/download/v0.1.0/jumping-macos | ||
| wget https://github.com/harkerhand/jumping/releases/latest/download/jumping-macos | ||
| chmod +x jumping-macos | ||
| sudo mv jumping-macos /usr/local/bin/jumping |
Comment on lines
29
to
37
| # Linux | ||
| wget https://github.com/harkerhand/jumping/releases/download/v0.1.0/jumping-linux | ||
| wget https://github.com/harkerhand/jumping/releases/latest/download/jumping-linux | ||
| chmod +x jumping-linux | ||
| sudo mv jumping-linux /usr/local/bin/jumping | ||
|
|
||
| # macOS | ||
| wget https://github.com/harkerhand/jumping/releases/download/v0.1.0/jumping-macos | ||
| wget https://github.com/harkerhand/jumping/releases/latest/download/jumping-macos | ||
| chmod +x jumping-macos | ||
| sudo mv jumping-macos /usr/local/bin/jumping |
| f.render_widget(right_list, chunks[2]); | ||
|
|
||
| let status = if app.show_hidden { "SHOW" } else { "HIDDEN" }; | ||
| let hint_text = format!("I/i: TOGGLE HIDDEN FOLDER (CURRENT: {}) | Q/q: QUIT", status); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.