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 search/filter functionality to object selection window #1837

Merged
merged 19 commits into from Feb 16, 2023

Conversation

AaronVanGeffen
Copy link
Member

@AaronVanGeffen AaronVanGeffen commented Feb 12, 2023

This PR introduces search/filter functionality to the object selection window, performing case-insensitive string matching on the visible name and object filenames.

Matches by visible name:
Unnamed (1)

Or by object filename:
Unnamed (2)

@AaronVanGeffen AaronVanGeffen added the changelog Should get a changelog entry label Feb 12, 2023
@AaronVanGeffen AaronVanGeffen linked an issue Feb 12, 2023 that may be closed by this pull request
@AaronVanGeffen AaronVanGeffen marked this pull request as ready for review February 12, 2023 21:53
@AaronVanGeffen AaronVanGeffen enabled auto-merge (squash) February 15, 2023 20:02
@@ -342,6 +342,16 @@ namespace OpenLoco::Input
}
}

ti = WindowManager::find(WindowType::objectSelection);
if (ti != nullptr)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should start handling these through the window manager, ideally passing through to an event for the active window only. Something for a future PR though.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah i was a bit confused as to what this was about.

@AaronVanGeffen AaronVanGeffen merged commit b5c1ad8 into OpenLoco:master Feb 16, 2023
@AaronVanGeffen AaronVanGeffen deleted the objsel/filter branch February 16, 2023 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog Should get a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow searching/filtering objects in object selection window
2 participants