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

[Feature Request]: Shortcut to refresh subscriptions (R) #2631

Closed
2 tasks done
Aiz0 opened this issue Sep 27, 2022 · 6 comments · Fixed by #2689
Closed
2 tasks done

[Feature Request]: Shortcut to refresh subscriptions (R) #2631

Aiz0 opened this issue Sep 27, 2022 · 6 comments · Fixed by #2689

Comments

@Aiz0
Copy link
Contributor

Aiz0 commented Sep 27, 2022

Guidelines

  • I have searched the issue tracker for a feature request that matches the one I want to file, without success.
  • I have searched the documentation for information that matches the description of the feature request I want to file, without success.

Problem Description

I often refresh my subscriptions with CTRL + R but this reloads the entire view and is therefore slower than just refreshing the subscriptions.

Proposed Solution

I would like a shortcut key for the refresh subscriptions button.
R would be my recommendation, Since it's already used with modifiers to reload the app.

Alternatives Considered

Refreshing just the subscriptions with CTRL + R might be better, i'm not sure.
Since the plan is to replace all other refresh buttons with #1578 a shortcut for that would be preferable. There hasn't been any updates to that for about a year now since the rework was planned so I don't know how it would reload the subscription page.

Issue Labels

ease of use improvement, new keyboard shortcut

Additional Information

Other relevant issues.
#1966

@ChunkyProgrammer
Copy link
Member

I can assign this issue to you if this is something that you'd like to work on

@Aiz0
Copy link
Contributor Author

Aiz0 commented Oct 7, 2022

I could work on this.
Do I need to open a PR to add the new shortcut to documentation as well?

@efb4f5ff-1298-471a-8973-3d47447115dc

yes please

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

efb4f5ff-1298-471a-8973-3d47447115dc commented Oct 7, 2022

@Aiz0 would u maybe like to give the reload button PR a shot?

Edit: The PR was opened by a team member of ours but he is inactive for a while now because of real life priorities

@Aiz0
Copy link
Contributor Author

Aiz0 commented Oct 7, 2022

@Aiz0 would u maybe like to give the reload button PR a shot?

At the moment, no, later maybe.
I don't really know what the reload button should do and I might have to read some electron and vue documentation before knowing how to do it 😅

If you can post a feature spec or something that would be a good starting point for me and others to try to implement it.

@efb4f5ff-1298-471a-8973-3d47447115dc
Copy link
Member

efb4f5ff-1298-471a-8973-3d47447115dc commented Oct 9, 2022

@Aiz0 maybe u can look into #2138 (comment) because it is shortcuts related

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants