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

Debug text overlap #7100

Closed
Toranks opened this issue Nov 6, 2022 · 4 comments
Closed

Debug text overlap #7100

Toranks opened this issue Nov 6, 2022 · 4 comments
Labels
Bug Issues involving unexpected behavior. Confirmed Issues that have been successfully reproduced by at least one developer. Regression Issues that were not present in previous releases. UI User interface issues, including both back-end and front-end issues.

Comments

@Toranks
Copy link
Contributor

Toranks commented Nov 6, 2022

Game and System Information

The Battle for Wesnoth version 1.17.9 x86_64
Running on Microsoft Windows 7 Service Pack 1 (6.1.7601) x86_64
Distribution channel: Steam

Current video settings

SDL video drivers: [windows] dummy
Window size: 1920x1080
Game canvas size: 1920x1080
Final render target size: 1920x1080
Screen refresh rate: 60

Description of the bug

The debug console informational text (and any other text above) overlaps between different outputs (but not lines inside the same outputs), this was not the case in 1.16 with the same settings and resolution.
Sin título

Steps to reproduce the behavior

  1. Write :debug
  2. Do different things or run different commands that output the text above.

Expected behavior

The spacing between lines should be the same in one output as it is between different outputs.

Additional context

Probably Windows-only

@Toranks Toranks added the Bug Issues involving unexpected behavior. label Nov 6, 2022
@Wedge009 Wedge009 added UI User interface issues, including both back-end and front-end issues. Confirmed Issues that have been successfully reproduced by at least one developer. labels Nov 6, 2022
@Wedge009
Copy link
Member

Wedge009 commented Nov 6, 2022

Doesn't appear to be OS-specific.

@Wedge009
Copy link
Member

Wedge009 commented Nov 7, 2022

This appears to have been caused by a5b1540 (@Vultraz), just before 1.17.7 was tagged.

@Wedge009
Copy link
Member

Wedge009 commented Nov 7, 2022

I'm trying to understand the details of what changed. At a guess, the previous surface-based implementation aligned nicely so that the text boxes did not overlap. But using textures to implement the text boxes, the alignment appears to only be done at the text level, ignoring the padding of the boxes surrounding the text, hence the overlap.

BTW, this issue applies to chat messages as well as debug output.

@Wedge009 Wedge009 added the Regression Issues that were not present in previous releases. label Nov 7, 2022
@Toranks
Copy link
Contributor Author

Toranks commented Nov 25, 2022

I have noticed that the overlap depends on the zoom level, in case it helps you to find the problem. Text boxes should not be affected by zoom.

Sin título

image

Wedge009 added a commit to Wedge009/wesnoth that referenced this issue Nov 26, 2022
Resolves wesnoth#7100.

Since a5b1540, chat messages are overlapping with each other. It appears the old SDL Surface implementation handles co-ordinates such that message borders were already accounted for.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues involving unexpected behavior. Confirmed Issues that have been successfully reproduced by at least one developer. Regression Issues that were not present in previous releases. UI User interface issues, including both back-end and front-end issues.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants