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

Mouse Pickable on Area2D doesn't work on SubViewport (broke my game from 4.1 to 4.2) #85659

Closed
stebulba opened this issue Dec 2, 2023 · 8 comments

Comments

@stebulba
Copy link
Contributor

stebulba commented Dec 2, 2023

Godot version

4.2-stable

System information

Ubuntu 22.04

Issue description

Also the $SubViewportContainer.set_default_cursor_shape(Input.CURSOR_POINTING_HAND) doesn't work anymore on 4.2.

Steps to reproduce

make sure mouse_pickable is true on Aera2D
image

Minimal reproduction project

Mouse_pickable-subviewport-4.2.zip

@AThousandShips
Copy link
Member

Please open one report for one issue, the second of your issues already has a report so there's no need to open a new one

@stebulba stebulba changed the title Mouse Pickable on Aera2D doesn't work on SubViewport + set_default_cursor_shape (broke my game from 4.1 to 4.2) Mouse Pickable on Aera2D doesn't work on SubViewport (broke my game from 4.1 to 4.2) Dec 2, 2023
@Sauermann
Copy link
Contributor

The two issues, that you describe are already reported:

Consolidating as duplicate with these dedicated bug-reports.

@Sauermann Sauermann closed this as not planned Won't fix, can't repro, duplicate, stale Dec 2, 2023
@stebulba
Copy link
Contributor Author

stebulba commented Dec 2, 2023

Consolidating as duplicate with these dedicated bug-reports.

My main point is that there is no release information addressing these well-known issues that completely break compatibility. Closing my duplicate without modifying the godot release page doesn't help anyone. On the contrary, game developers will waste their time duplicating the issues again.

@AThousandShips
Copy link
Member

AThousandShips commented Dec 2, 2023

This should be tracked in the respective issues or the original PRs instead IMO, this is still two different, unrelated issues, I'm also not sure the second issue can be considered a critical change for these purposes, unlike the first which does have a workaround that can be mentioned

@stebulba
Copy link
Contributor Author

stebulba commented Dec 2, 2023

I agree with you regarding the second issue. I have, in fact, adjusted the title accordingly. It's not the first time I've wasted a lot of time diagnosing a bug that already existed. In this case, the problem stems primarily from an early release of version 4.2 or simply a lack of information in the release notes.

@AThousandShips
Copy link
Member

AThousandShips commented Dec 2, 2023

Then there's no need for this, as only one of the issues has this concern, so this is still a duplicate, and 4.2 was not released too early, but additional notes would be useful

It's not the first time I've wasted a lot of time diagnosing a bug that already existed.

That's not the fault of the missing note, you could have searched for it instead and found it, as you did find it after "wasting" that time

It feels to me like the only reason this issue report is a separate report as opposed to a comment on the existing one is frustration with having spent time trying to identify the bug, and your opinion that 4.2 was released prematurely, but to me it's not constructive to make an issue report for these reasons, I'd say that a note should be left on that report suggesting it be added to the release post as a critical change and this be closed

@AThousandShips AThousandShips reopened this Dec 2, 2023
@AThousandShips AThousandShips changed the title Mouse Pickable on Aera2D doesn't work on SubViewport (broke my game from 4.1 to 4.2) Mouse Pickable on Area2D doesn't work on SubViewport (broke my game from 4.1 to 4.2) Dec 2, 2023
@stebulba
Copy link
Contributor Author

stebulba commented Dec 2, 2023

and your opinion that 4.2 was released prematurely, but to me it's not constructive to make an issue report for these reasons

My opinion is NOT the reason of my post. A real issue is existing.
The release talk about Viewport mouse handling work from Markus Sauermann
How can we know that the bug I got is an old bug since the release is supposed to be functional?
If there is an existing old tracking about this bug, for general users, this looks like a new bug separate of the old tracking. And if this bug is not reported after the release of version 4.2, the confusion persists.
The main issue is the lack of information in the release notes. With more information, I wouldn't have downloaded this version.
I think my post can do the job for now. When this lack is fixed, for sure my post can be close.

I hope it is understandable.

@AThousandShips
Copy link
Member

If there is an existing old tracking about this bug, for general users, this looks like a new bug separate of the old tracking.

That's not helpful, we can't make new bug reports for all bugs found during 4.2 development once 4.2 releases, it says it's for 4.2 in the description

But sure, we can keep this open, but I'd suggest in the future to comment on the existing bug report to suggest this instead of adding extra bug reports for the same issue, adding work for bug triagers and clutter

@stebulba stebulba closed this as completed Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants