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

Add show_unfiltered to allow display of things that *do* match a filter #2280

Merged
merged 3 commits into from
Nov 15, 2024

Conversation

chazlarson
Copy link
Contributor

Description

Add show_unfiltered setting to allow display of the [possibly] few items that make it through a filter rather than the [possibly] thousands that don't

Type of Change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Checklist

Please delete options that are not relevant.

  • Updated Documentation to reflect changes
  • Updated the CHANGELOG with the changes

@chazlarson
Copy link
Contributor Author

This PR also gets rid of a bunch of duplication on the Filters page.

@YozoraXCII
Copy link
Contributor

YozoraXCII commented Oct 17, 2024

Should we make the default behaviour be true for show unfiltered? Or is this not related to the discussion about overlays last week

@chazlarson
Copy link
Contributor Author

Should we make the default behaviour be true for show unfiltered? Or is this not related to the discussion about overlays last week

It's related to that discussion , but leaving the default as false won't put anything new in people's logs. Personally I think the default should be:

show_filtered: false
show_unfiltered: true

But I left it as is in the interest of least-changes.

@YozoraXCII
Copy link
Contributor

Should we make the default behaviour be true for show unfiltered? Or is this not related to the discussion about overlays last week

It's related to that discussion , but leaving the default as false won't put anything new in people's logs. Personally I think the default should be:

show_filtered: false
show_unfiltered: true

But I left it as is in the interest of least-changes.

I believe Soh was in agreement about switching to show items that do meet the filter criteria so that logs aren't hammered by things that don't meet it. I would support this change

@chazlarson
Copy link
Contributor Author

Turning it on will cause a bunch of new logging for people who have filtering off. I'm not sure we want that. Like a 4K overlay in a 4K library; suddenly the log has thousands more lines.

Maybe leave it off to prevent that. The reason to add it is to get useful information if desired

@chazlarson
Copy link
Contributor Author

This change leaves "show filtered items" off, as has always been the default, but enables the user to turn on "just show me the things that made it through" if they want.

@YozoraXCII
Copy link
Contributor

I have tested this locally and confirm it works as expected.

When show_filtered: true:

image

when show_unfiltered: true:

image

@YozoraXCII YozoraXCII added the status:ready-to-merge PR has been reviewed and is ready to be merged label Oct 31, 2024
@YozoraXCII YozoraXCII merged commit cd37a77 into nightly Nov 15, 2024
4 checks passed
@YozoraXCII YozoraXCII deleted the show-positive-filtering branch November 15, 2024 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:ready-to-merge PR has been reviewed and is ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants