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

Feature: Town name filtering #7621

Merged
merged 2 commits into from Jul 22, 2019
Merged

Conversation

telk5093
Copy link
Contributor

@telk5093 telk5093 commented Jun 12, 2019

This is a re-PR of #6983, which implements the filtering feature for town name list like below:
(I reopen a PR since I have no idea how to change branch of previous PR)
image

Now it does not blink and seems stable unlike #6983 as @LordAro said.
Nevertheless, please let me know if there any problem with my commit and also please aware that I'm very new to both C/C++ and OpenTTD's codes.

src/town_gui.cpp Outdated Show resolved Hide resolved
@telk5093 telk5093 force-pushed the town_name_filtering branch 2 times, most recently from ff43701 to e75f752 Compare June 13, 2019 03:40
@nielsmh
Copy link
Contributor

nielsmh commented Jun 13, 2019

Remember to run the generate_widget and squirrel_export scripts after adding a new widget ID, and commit the updated script API files.

src/town_gui.cpp Outdated Show resolved Hide resolved
@telk5093
Copy link
Contributor Author

@nielsmh I had no idea I should run squirrel_export additionally. Thanks for noticing me and has done.

@LordAro LordAro merged commit 196d586 into OpenTTD:master Jul 22, 2019
douiwby pushed a commit to douiwby/OpenTTD that referenced this pull request Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants