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

Torrent listing in qBittorrent #724

Closed
wants to merge 3 commits into from
Closed

Conversation

dbtdsilva
Copy link

I've added torrent listing in the qBittorrent widget. Let me know your thoughts.
I based myself a little bit from Tautulli.

This is the end result (the torrents names were auto-generated, but I guess you can image how it looks like):

qbittorrent_torrent_list

@shamoon
Copy link
Collaborator

shamoon commented Dec 24, 2022

Thanks for the PR. This is cool as a proof-of-concept but I’m not sure if I’d be in favor of merging this, it doesn’t really feel in line with other widgets, although Tautulli perhaps. It also feels a little unwieldy?

I think of the homepage widgets as aiming to be quick useful glances of info, not a comprehensive alternative to the actual service.

Of course, happy to hear others’ perspective

@Oupsman
Copy link
Contributor

Oupsman commented Dec 25, 2022

Maybe just display the top 3 active torrents (aka downloading or uploading) sorted out by speed ?

@dbtdsilva
Copy link
Author

Maybe just display the top 3 active torrents (aka downloading or uploading) sorted out by speed ?

We could even add those kinds of display options into the widget configuration

@shadofall
Copy link

I like it, i'd also would love to see something similar with SAB, if its something that can be made optional for if users want to see it or not, Combine it with a nice calendar widget with "today, this week, this month in sonarr/radarr" and you'd have to only two features from organizrr i'm missing ;) but just my two bits as a new user :P

@FullMetalEd
Copy link

Recently switched to Homepage, and I think this would be a good feature too. My suggestion is that you make the list start collapsed and click to reveal. loading the full list at page load is unwieldy.

@JazzFisch
Copy link
Collaborator

Thank you for the feedback everyone. We are weighing the usability / visibility aspects of this PR against issues from other users who want more consistency in the sizing of widgets so that they line up and align in a way that is aesthetically pleasing for them. A possible solution we've considered would be for the list to be off by default with an opt in mechanism. We've done this in other areas so that older behavior is the default and new behavior is behind an opt-in configuration.

Copy link
Collaborator

@shamoon shamoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes if you can put behind a flag

@shamoon shamoon marked this pull request as draft January 29, 2023 00:38
@shamoon
Copy link
Collaborator

shamoon commented Feb 19, 2023

Please re-open if / when you're ready to proceed with this, just closing to tidy things up. Thanks

@shamoon shamoon closed this Feb 19, 2023
Copy link
Contributor

github-actions bot commented Feb 6, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion for related concerns.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants