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

Make the theme qBittorrent v5 compatible #4

Closed
JohanSF opened this issue Nov 12, 2024 · 7 comments
Closed

Make the theme qBittorrent v5 compatible #4

JohanSF opened this issue Nov 12, 2024 · 7 comments

Comments

@JohanSF
Copy link

JohanSF commented Nov 12, 2024

As binhex writes here: repslet#20 (comment)

Quote:

Hi love this theme, any chance you can fix up the theme to support qBittorrent v5 please.

Things I have noticed so far (there could and probably are more):

  • missing paused/stopped icon
  • Inability to pause any torrents

It looks like this fork might have fixed these issues: https://github.com/Jorsher/nightwalker

@joshft91
Copy link

@Jorsher, if your code is ready, would you be willing to put your fork up for review against CallMeBruce's repo? May be able to get this merged in to get it working for hotio images.

@Jorsher
Copy link

Jorsher commented Nov 20, 2024

@Jorsher, if your code is ready, would you be willing to put your fork up for review against CallMeBruce's repo? May be able to get this merged in to get it working for hotio images.

To be honest, I don't know how to do that and I'm not sure my changes are up to snuff.

Essentially I just find/replace-d all variants of pause/paused/resume/resumed with stop/stopped/start/started. May have even changed some that didn't need to be changed. Because of this, it won't work with versions prior to 5.0. Pause icon should probably be changed to a Stop one too :)

Qbit dark theme wasn't quite as sleek as this and it was the main thing holding me back from upgrading to 5.0.

@rakosi2
Copy link

rakosi2 commented Nov 20, 2024

You can create a pull request in the pull request tab. However, you don't want to add the changes to the README so the easiest way is to make a new branch and checkout the main commit or cherry-pick it and pull request that branch.
Don't worry about incompatibility for older Qbit as @CallMeBruce could make a new branch for pre-5.0 or people can checkout the version that they want.

@IIIdefconIII
Copy link

Unacceptable file type, only regular file is allowed.

@joshft91
Copy link

joshft91 commented Nov 21, 2024

To be honest, I don't know how to do that and I'm not sure my changes are up to snuff.

Essentially I just find/replace-d all variants of pause/paused/resume/resumed with stop/stopped/start/started. May have even changed some that didn't need to be changed. Because of this, it won't work with versions prior to 5.0. Pause icon should probably be changed to a Stop one too :)

Qbit dark theme wasn't quite as sleek as this and it was the main thing holding me back from upgrading to 5.0.

That's fair, I am still on 4.6.7. I wasn't sure the scope of your changes or what has changed in qBittorrent 5 that was breaking with Nightwalker.

Don't worry about incompatibility for older Qbit as @CallMeBruce could make a new branch for pre-5.0 or people can checkout the version that they want.

I'd be a bit cautious about this approach since this repo currently gets bundled into the hotio legacy release. Merging in breaking changes for users using the legacy release would cause problems for them.

@CallMeBruce
Copy link
Owner

Just did a re-combine of latest 5.0.2 version and the nightwalker theme. Let me know if you find something that isn't working.

@IIIdefconIII
Copy link

IIIdefconIII commented Nov 21, 2024 via email

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

No branches or pull requests

6 participants