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

Bionic menu clean up #35221

Merged
merged 3 commits into from
Nov 18, 2019
Merged

Bionic menu clean up #35221

merged 3 commits into from
Nov 18, 2019

Conversation

Fris0uman
Copy link
Contributor

@Fris0uman Fris0uman commented Nov 1, 2019

Summary

SUMMARY: Interface "Display name of perpetual fuel"

Purpose of change

Display name of perpetual fuel
Don't display Fuel Available: if no fuel is available or if the current mode is reassigning
Replace ostringstream with string

Describe the solution

Describe alternatives you've considered

Testing

With Perpetual fuel:
image

Without any Fuel:
image

Additional context

@curstwist curstwist added [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. labels Nov 1, 2019
src/bionics_ui.cpp Outdated Show resolved Hide resolved
src/bionics_ui.cpp Outdated Show resolved Hide resolved
src/bionics_ui.cpp Outdated Show resolved Hide resolved
@tinukedaya
Copy link
Contributor

From the pictures it looks like it's hiding the bionics if you don't have the power to activate them. Is that so?
If yes, I'd suggest instead of hiding, grey them out and put them on the bottom of the list.
If not, just ignore me...

@Fris0uman
Copy link
Contributor Author

It's two screen shot of two different characters, the second one has no bionics.

@ZhilkinSerg ZhilkinSerg merged commit a3b0f0f into CleverRaven:master Nov 18, 2019
@Fris0uman Fris0uman deleted the biomenu branch November 18, 2019 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants