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

WebUI: Improve subcategories #21269

Merged
merged 3 commits into from
Sep 8, 2024

Conversation

skomerko
Copy link
Contributor

This PR improves subcategories.


image

Now they should fully match GUI behavior, please let me know if I missed something.
Still plenty of room to improve them further (e.g styling/CSS) but for now I wanted to keep the changes to the minimum.

Also included small tweaks to category context menu actions

@luzpaz luzpaz added the WebUI WebUI-related issues/changes label Aug 27, 2024
src/webui/www/private/views/filters.html Outdated Show resolved Hide resolved
src/webui/www/private/newcategory.html Outdated Show resolved Hide resolved
src/webui/www/private/scripts/client.js Outdated Show resolved Hide resolved
src/webui/www/private/scripts/client.js Outdated Show resolved Hide resolved
src/webui/www/private/scripts/client.js Outdated Show resolved Hide resolved
src/webui/www/private/scripts/client.js Outdated Show resolved Hide resolved
src/webui/www/private/scripts/client.js Outdated Show resolved Hide resolved
src/webui/www/private/scripts/client.js Show resolved Hide resolved
src/webui/www/private/scripts/client.js Outdated Show resolved Hide resolved
src/webui/www/private/scripts/client.js Outdated Show resolved Hide resolved
@skomerko
Copy link
Contributor Author

skomerko commented Sep 1, 2024

@Chocobo1, during my tests I noticed that categories are sorted differently in the WebUI (numeric collation is used):

image

vs GUI:

image

Not sure which one should be used, just letting you know.

@Chocobo1
Copy link
Member

Chocobo1 commented Sep 1, 2024

@Chocobo1, during my tests I noticed that categories are sorted differently in the WebUI (numeric collation is used):

We just stick to the platform native natural sorting if it provides one. The occasional discrepancy between gui and webui is known and if that is what the platform provides then so be it, after all qbt just accustom to the native platform.
In case you didn't notice, in webUI window.qBittorrent.Misc.naturalSortCollator should be used for sorting whenever possible.

@Chocobo1 Chocobo1 added this to the 5.1 milestone Sep 6, 2024
@Chocobo1 Chocobo1 requested a review from a team September 6, 2024 08:03
@Chocobo1 Chocobo1 merged commit 1b53fdf into qbittorrent:master Sep 8, 2024
14 checks passed
@Chocobo1
Copy link
Member

Chocobo1 commented Sep 8, 2024

@skomerko
Thank you!

Chocobo1 pushed a commit that referenced this pull request Oct 12, 2024
This PR adds following improvements: 
* Remove unused tracker entries while processing sync data
* Take into account filter selection & terms when performing 'Start/stop/delete' context actions in filter lists
  Now, only filtered torrents will be affected by them, just like in the GUI.
* Provide better feedback when performing 'Start/stop/delete' context actions in filter lists
  Small improvement over GUI - now these actions will be disabled if it's not possible to use them.
* Add context menu to status filter list
* Fix error when toggling filter title
  Fixup for small bug introduced in #21269

PR #21438.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebUI WebUI-related issues/changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants