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

Empty control nodes (for example Control) should default to pass the mouse event. #24083

Closed
QbieShay opened this issue Nov 30, 2018 · 3 comments

Comments

@QbieShay
Copy link
Contributor

Godot version:
11d7738

Issue description:
When creating interfaces, it happened to me quite often to create full screen controls and instantiate other elements inside of this full size controls.
This leads to an unexpected behaviour which is that even an empty control by default stops the mouse.
I would expect this behaviour from a solid Control element(buttons, panels, etc) but not from the 'vanilla' Control node.

This behaviour led me to spend quite some time debugging why some gui functionalities were broken, when it was just the control filtering mouse events

Proposal:
All the empty control nodes (so far i can think only of the base control node) should pass mouse events by default.

@Zylann
Copy link
Contributor

Zylann commented Nov 30, 2018

I agree, I went into the same situations. Controls that have no visuals by default should ignore the mouse (Control and some of the Container nodes).

@KoBeWi
Copy link
Member

KoBeWi commented Aug 17, 2019

Yeah, the problem is that MOUSE_FILTER_STOP is inherited by pretty much every Control-derived node and none of them change this. So this should be changed for each of these nodes individually, unless there's some possibility to change it only for base class.

@akien-mga
Copy link
Member

Closing as duplicate of godotengine/godot-proposals#788 since we now have those discussions in the proposal repo.

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

4 participants