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.

Sign up for GitHub

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

Jump to bottom

RSS: Performance - download rules perform match after each keypress #6166

Open
magao opened this issue Dec 30, 2016 · 2 comments
Open

RSS: Performance - download rules perform match after each keypress #6166

magao opened this issue Dec 30, 2016 · 2 comments
Labels
Performance RSS RSS-related issues/changes

Comments

@magao
Copy link
Contributor

magao commented Dec 30, 2016 •
edited by xavier2k6
Loading

When modifying the RSS download filter rules, the filter will be matched against the current articles after each keypress in the "Must Contain", "Must Not Contain" and "Episode Filter" fields.

When there are a large number of articles to match this can result in very unresponsive UX.
@magao
Copy link
Contributor Author

magao commented Dec 30, 2016

I will be submitting a pull request shortly which delays matching until there has been no input for at least one second.

@magao magao mentioned this issue Jan 1, 2017
Closed
magao added a commit to magao/qBittorrent that referenced this issue Jan 28, 2017
@magao
RSS defer matching articles for rules until idle. Closes qbittorrent#…
bd643b4 
…6166.

--HG--
branch : magao-dev
@magao magao mentioned this issue Feb 11, 2017
Closed
@FranciscoPombal FranciscoPombal added Performance RSS RSS-related issues/changes labels Feb 19, 2020
@luzpaz
Copy link
Contributor

luzpaz commented Oct 25, 2023

@xavier2k6 what's the verdict on this ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Performance RSS RSS-related issues/changes
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@magao @luzpaz @FranciscoPombal