-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
WebUI: Implement double-click behavior controls #21000
Conversation
489347b
to
b79b5e3
Compare
Personally I dislike idea of storing WebUI client settings as regular qBittorrent settings. I would consider any WebUI client (including official one) as separate client application. |
I followed existing logic to make this. If you think this settings should be stored in browser local storage instead, I'm not against it. In fact it's even easier that way. |
b79b5e3
to
4191d71
Compare
Ok, I made it using local storage. The implementation is very simple now. |
ec70553
to
3aa9cee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strange that CI didn't run..
No idea why. #21007 had the same state back then. |
3aa9cee
to
2a66021
Compare
@HanabishiRecca |
This PR implements double-click behavior controls for the WebUI, similar to the desktop GUI.
Closes #20436.
But as I discussed in #20436 (comment), despite the same names, this is not the same settings. Because existing options of the GUI are not applicable for the WebUI, they are implemented as independent standalone settings.
Also the settings implemented as selectors, despite having only 2 options. They actually could be implemented as check boxes instead, but:
Screenshot