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

Extend the message window to cover the whole available vertical space. #9904

Merged
merged 2 commits into from
Nov 8, 2014

Conversation

BevapDin
Copy link
Contributor

@BevapDin BevapDin commented Nov 8, 2014

Fixes #2091 (it's the solution proposed there), but also fixes #9834 as message window be will redrawn and override the stuff from the previous window. The HP window is also expanded to cover the space between message window and terrain window that is below the HP window.

I recognized the window w_status2 (which is unused with the wide sidebar) causes flickering as it overlays with the other windows, so I excluded it from being redrawn.

This occupies the previously unused space and erases / redraws it when the sidebar is redrawn.
The window w_hp goes also down to the same line as w_mesages, so it occupies and erases the space between the messages window and the terrain window.
Don't erase nor refresh the unused w_status2 window. Nothing is drawn to it anyway.
@KA101 KA101 self-assigned this Nov 8, 2014
@KA101
Copy link
Contributor

KA101 commented Nov 8, 2014

Hmm.
The reason I use the wide sidebar is that it has all the handy info in the middle of the screen where I can see it, rather than shunting it into the corner. I imagine having the log below the bar would be a PitA to implement though, so yeah.

@KA101 KA101 merged commit b87a697 into CleverRaven:master Nov 8, 2014
@BevapDin BevapDin deleted the ui-problems branch November 8, 2014 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants