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
Fix #7631: 16 out cargo support for industry directory #7809
Conversation
Would it be possible to implement one of the following systems instead of just going to cargo sprites and hiding a bunch of information from players? This will retain more information in the industry directory window. Plan A: Plan B: |
IMO this change makes industry window completely pointless as amount of produced cargo was the only useful info there. Also default cargo icons are terrible in this context. They're too small and not very recognizable. |
Why? Because currently the window hard crashes the game if your industry has more than 7 cargos produced/accepted. I do agree that this isn't the correct solution though... |
Could we do this:
?? The rationale is that it's high production amounts that are looked for in this window. |
Given that we have a hard-crash currently, I think that looks like a pretty good fix 👍 Long-term, maybe we should change string handling so we can show all the cargos, but that also raises UI questions (horizontal overflow). Let's fix the immediate crash. |
d42620d
to
2deb22d
Compare
2deb22d
to
198c8f9
Compare
Oups I should have checked with mingw before pushing |
198c8f9
to
b0cabae
Compare
It's just a basic change. The window may need to be rewritten.
English.txt needs to be cleaned up