-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
fix(runtime-wry): window edge not working after setting resizable false and decorated false dynamically #10211
fix(runtime-wry): window edge not working after setting resizable false and decorated false dynamically #10211
Conversation
after setting resziable false and decorated false dynamically
I'm not sure if we also want to disable it on fullscreen and maximize or is it something the user should do manually |
I think we used to do that 🤔 |
Looking at the code it seems to be handled by amr's code already, |
"tauri-runtime-wry": "patch:bug" | ||
--- | ||
|
||
Fix window edge not working after setting resizable false and decorated false dynamically |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure I understand the problem, if decoration is false and resizable is false, window edge should not work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't the edges of an undercoated window with resize disabled receive events and being clickable?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah it should, is it not receiving these events?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, that part is blocked by the top level resize window and not interactable at all
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah I see where the bug is, I will push a better fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Legend-Master could you give 14af33b a test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't seem to work, after setting both resize and decoration to false, the entire window becomes not interactable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I retested and seemed like I can click just fine, outside and inside the resize region.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nvm, I was mislead by the hover effect, I reverted my changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kinda weird that HTTRANSPARENT
didn't work, will need to research it more I guess. Your solution is fine for now.
e0725d0
to
7634cd7
Compare
Not sure if it's related with #10205