transmission
transmission copied to clipboard
Add privacy filter for webui (filter torrents by private/public status)
Originally from #4423. This is mostly from @WillDaSilva, but I changed a few things:
- slight UI change (
 
to--
) as separator - deduplicate 'All'
- filterTracker expression was wrong
I can’t seem to tag @WillDaSilva as coauthor, for some reason.
@dareiff Thank you for making this!
If you would like to add me as a co-author, you can rebase the first commit of this PR so as to edit its commit message to end with:
Co-authored-by: Will Da Silva <[email protected]>
Thanks. Don't forget to undraft your pull request when ready for review.
@dareiff Thank you for making this!
If you would like to add me as a co-author, you can rebase the first commit of this PR so as to edit its commit message to end with:
Co-authored-by: Will Da Silva <[email protected]>
Thanks for that! Got it in.
OK, ready for review @tearfur.
Styling items in dropdown is not supported under some browsers. You can just remove styles resulting in only hyphens in the dropdown.
For some sites that may be using "private" and "public" in their domain, mai I suggest we prepend them with "is:", as so: "is:private", "is:public", my custom client has "is:" to all status dropdown while the "tr:" is given to trackers so these can be combined into a single dropdown and I'll have no problem segregating them when filtering torrents.
Styling items in dropdown is not supported under some browsers. You can just remove styles resulting in only hyphens in the dropdown.
Yeah I’m not sure exactly why that inline style was there, but everything looked normal in my browsers so I didn’t even think to remove it.
For some sites that may be using "private" and "public" in their domain, mai I suggest we prepend them with "is:", as so: "is:private", "is:public"
I like it 👍🏻
@dareiff Unless you have something else to say/suggest, please update this PR according to Rukario's suggestion.
@Rukario AFAIK All modern browsers support some level of styling select boxes. But inline styles should be avoided anyway.
Hello, it looks like this PR is in limbo of not updating, are you going to do about it? This is great feature BTW.
I can see the benefit of using both private flag and torrent status to each other but I think the private flag should be moved to status select box to make more sense code-wise and for one of the future PRs aiming to tackle the obstacle of limited options in filtering torrents. It will be harmonious with using "is:" on all status from different places of a torrent. Private flag is just another status type belonging to private trackers.