Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: help popup display viewing progress #2388

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

wugeer
Copy link
Contributor

@wugeer wugeer commented Oct 15, 2024

It changes the following:

  • help popiup page show viewing progress

I followed the checklist:

  • I added unittests
  • I ran make check without errors
  • I tested the overall application
  • I added an appropriate item to the changelog

before
gitui_help_popup_before.webm

after
gitui_help_popup_after.webm

@extrawurst
Copy link
Collaborator

Thanks for looking into this. The gitui idiomatic way to show progress on a vertical scroll though is using a scrollbar. so I would suggest we integrate that here too. feel free to join our discord to discuss there things upfront in the future to not waste any effort.

@wugeer
Copy link
Contributor Author

wugeer commented Oct 15, 2024

In my personal view, having a percentage-based progress might make it a bit easier to understand. :)
It seems that when scrolling up and down on this page, I didn’t notice a scrollbar on the right side.
image
Also, I’ll create a Discord account a bit later.

@wugeer
Copy link
Contributor Author

wugeer commented Oct 16, 2024

@extrawurst I'll try to add a scrollbar on the right, similar to the one in a browser.:)

@extrawurst
Copy link
Collaborator

Browser?

Just use the same code we use everywhere else do scroll bars in for example the log tab

@wugeer
Copy link
Contributor Author

wugeer commented Oct 16, 2024

Yes, I will try it. :)
image

@wugeer
Copy link
Contributor Author

wugeer commented Oct 16, 2024

@extrawurst Have Done!

@extrawurst
Copy link
Collaborator

extrawurst commented Oct 16, 2024

looking good! i wonder why you have different formatting - lets see what the ci says about that

edit: it fails on the formatting. please make sure to use the latest rust stable

@wugeer
Copy link
Contributor Author

wugeer commented Oct 16, 2024

My bad, my local environment was on an unstable version. I switched to the stable version of Rust, and cargo clippy works fine.

@extrawurst extrawurst merged commit fb1ba7c into gitui-org:master Oct 18, 2024
20 of 21 checks passed
@extrawurst
Copy link
Collaborator

Thanks for your contribution

tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Jan 28, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [extrawurst/gitui](https://github.com/extrawurst/gitui) | minor | `v0.26.3` -> `v0.27.0` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>extrawurst/gitui (extrawurst/gitui)</summary>

### [`v0.27.0`](https://github.com/extrawurst/gitui/releases/tag/v0.27.0)

[Compare Source](gitui-org/gitui@v0.26.3...v0.27.0)

**new: manage remotes**

![add-remote](assets/add-remote.png)

##### Breaking Changes

-   use default shell instead of bash on Unix-like OS \[[@&#8203;yerke](https://github.com/yerke)] ([#&#8203;2343](gitui-org/gitui#2343))

##### Added

-   add popups for viewing, adding, updating and removing remotes \[[@&#8203;robin-thoene](https://github.com/robin-thoene)] ([#&#8203;2172](gitui-org/gitui#2172))
-   support for `Copy Path` action in WSL \[[@&#8203;johnDeSilencio](https://github.com/johnDeSilencio)] ([#&#8203;2413](gitui-org/gitui#2413))
-   help popup scrollbar \[[@&#8203;wugeer](https://github.com/wugeer)] ([#&#8203;2388](gitui-org/gitui#2388))

##### Fixes

-   respect env vars like `GIT_CONFIG_GLOBAL` ([#&#8203;2298](gitui-org/gitui#2298))
-   Set `CREATE_NO_WINDOW` flag when executing Git hooks on Windows ([#&#8203;2371](gitui-org/gitui#2371))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4xMzcuMSIsInVwZGF0ZWRJblZlciI6IjM5LjEzNy4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiXX0=-->
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.

2 participants