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 option to change position of look around windows #36804

Merged
merged 4 commits into from
Apr 11, 2020

Conversation

Kilvoctu
Copy link
Contributor

@Kilvoctu Kilvoctu commented Jan 8, 2020

Summary

SUMMARY: Interface "Add option to change position of look around windows"

Purpose of change

First part of my own request #36327.

Describe the solution

Adds "Look around position" in the interface options, where the options are left and right (right is default).

Describe alternatives you've considered

Do nothing and wait for someone else to do it.

Testing

Play the game with sidebar on right and look around window on right.
Play game with sidebar on right and look around window on left.
Play with sidebar on left and look around window on right.
With sidebar on left and look around window on left.
Tested on bordeless fullscreen, maximized and windowed.

Additional context

See original issue for screenshots of how it looks now.
Used the pickup position code as reference.
Will add options for the aim window and list items window as separate PRs if nothing goes terribly wrong with this one.

Adds UI function in options to change look around window between left and right
@Night-Pryanik
Copy link
Contributor

Please run your changes through the astyle.

@Kilvoctu
Copy link
Contributor Author

Kilvoctu commented Jan 8, 2020

Yeah, my fault. When I was following the compiling guide, I didn't realize the astyle test was off. As a non-programmer, I'm not familiar with the concept and was already learning what it is and looking through the files.

astyle attempt
src/game.cpp Outdated Show resolved Hide resolved
la window is defined on right-hand side, and cleanup conditional
@ZhilkinSerg ZhilkinSerg added [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. labels Jan 10, 2020
@ZhilkinSerg
Copy link
Contributor

@kevingranade: New option suggested

Fix la window always redisplaying on right side when toggling minimap
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.

6 participants