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

Add setInteractive #9055

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add setInteractive #9055

wants to merge 2 commits into from

Conversation

someonewithpc
Copy link

Closes #5442

I'll note the I couldn't get the popup to trigger the event handler (UIEventSimulator.fire('click', popup._container);). I'm also not sure if the line endings are correct, as I tried messing with that in many ways and some diffs show some ^M, regardless of what I did

@someonewithpc
Copy link
Author

@Falke-Design Could you take a look at this? I had to publish a fork so my company's project could proceed, and I'd rather have it merged. Let me know if something is missing or isn't the best approach

@someonewithpc
Copy link
Author

@Malvoz Any chance you could review this? I'm currently applying a patch, to use it at my company, and I'd rather move away from that

@heyman
Copy link
Contributor

heyman commented Jan 10, 2024

Just chiming in that this feature would be useful 😀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New method setInteractive(interactive) for interactive layers
3 participants