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

[Bug]: Clicking the land info button in the main toolbar no longer activates the land info tool without using the drop down #10987

Closed
JGRennison opened this issue Jun 11, 2023 · 2 comments

Comments

@JGRennison
Copy link
Contributor

Version of OpenTTD

master

Expected result

Clicking the land info button in the main toolbar activates the land info tool without using the drop down

Actual result

Clicking the land info button in the main toolbar no longer activates the land info tool without using the drop down

Steps to reproduce

Clicking the land info button in the main toolbar

@glx22
Copy link
Contributor

glx22 commented Jun 11, 2023

Seems to happen in 13.3 too (it's fine in 13.1)

@PeterN
Copy link
Member

PeterN commented Jun 11, 2023

Hmm, it calls SetObjectToPlace, and then ResetObjectToPlace straight after.

PeterN added a commit to PeterN/OpenTTD that referenced this issue Jun 12, 2023
…rking as default.

Clicking and releasing on the query toolbar icon is meant to select the land-info tool.

This did not work as during closing a window, OnFocusLost() is called, which then closes the window again. These two calls toggled the land-info tool one and off in the same action.

Resolve by not calling Window::Close in OnFocusLost() if the window is already closing.
@PeterN PeterN closed this as completed in ebc451b Jun 12, 2023
LordAro pushed a commit to LordAro/OpenTTD that referenced this issue Jun 28, 2023
…rking as default. (OpenTTD#11000)

Clicking and releasing on the query toolbar icon is meant to select the land-info tool.

This did not work as during closing a window, OnFocusLost() is called, which then closes the window again. These two calls toggled the land-info tool one and off in the same action.

Resolve by not calling Window::Close in OnFocusLost() if the window is already closing.
LordAro pushed a commit that referenced this issue Jun 28, 2023
…s default. (#11000)

Clicking and releasing on the query toolbar icon is meant to select the land-info tool.

This did not work as during closing a window, OnFocusLost() is called, which then closes the window again. These two calls toggled the land-info tool one and off in the same action.

Resolve by not calling Window::Close in OnFocusLost() if the window is already closing.
mrmbernardi pushed a commit to mrmbernardi/OpenTTD that referenced this issue Jul 2, 2023
…rking as default. (OpenTTD#11000)

Clicking and releasing on the query toolbar icon is meant to select the land-info tool.

This did not work as during closing a window, OnFocusLost() is called, which then closes the window again. These two calls toggled the land-info tool one and off in the same action.

Resolve by not calling Window::Close in OnFocusLost() if the window is already closing.
shoter pushed a commit to shoter/OpenTTD that referenced this issue Jul 16, 2023
…rking as default. (OpenTTD#11000)

Clicking and releasing on the query toolbar icon is meant to select the land-info tool.

This did not work as during closing a window, OnFocusLost() is called, which then closes the window again. These two calls toggled the land-info tool one and off in the same action.

Resolve by not calling Window::Close in OnFocusLost() if the window is already closing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants