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

fix(core/wry): implement resizing natively on Windows #9862

Merged
merged 4 commits into from
Jun 5, 2024

Conversation

amrbashir
Copy link
Member

@amrbashir amrbashir commented May 23, 2024

closes #7388
closes #9510
closes #9464
closes #9881

ref #9268
ref #9053
ref #8770
ref #8750
ref #4012

@FabianLars
Copy link
Member

Sorry for the long delay on the review here...

The resizing itself works really well, good job!

You said you wanted to add support for the controls too, right? Asking because if users imitate native controls then the resize handles will conflict with it at the border a bit.
Even if not, imo it'd be a good tradeoff (especially if we add bordered mode too idk), just wanna be upfront about it.

@amrbashir
Copy link
Member Author

You said you wanted to add support for the controls too, right? Asking because if users imitate native controls then the resize handles will conflict with it at the border a bit.
Even if not, imo it'd be a good tradeoff (especially if we add bordered mode too idk), just wanna be upfront about it.

That should be in another PR, and most likely resize handles will take priority over buttons

@FabianLars
Copy link
Member

That should be in another PR

Of course

and most likely resize handles will take priority over buttons

Yeah, that's how this PR already behaves with html/js buttons which is why i'm asking, cause this is different to native/electron. But again, i'm fine with that, just wanted to talk about it once so we're all on the same page.

@amrbashir
Copy link
Member Author

cause this is different to native/electron.

Sure native behaves differently but that's out of scope of this PR since we can't achieve similar things as native (not yet at least) but I am curious how electron behaves with WCO or custom HTML buttons.

@FabianLars
Copy link
Member

FabianLars commented Jun 4, 2024

in vscode for example (and other random electron apps I have installed) the buttons behave like native == only have a 1px resize handle (feels less than 1px tbh). Basically as if the buttons float above the resize handles.

@amrbashir
Copy link
Member Author

Just tested electron and I don't see them doing anything special, the resize border size is consistent whether there is a button or not so it is safe to say they have the same behavior as us (maybe our resize border size is a little bit bigger?)

@FabianLars
Copy link
Member

FabianLars commented Jun 5, 2024

Maybe the apps i tested used WCO 🤷

maybe our resize border size is a little bit bigger?

Now that you mention it, yes that's also the case. (imo good because for me electron's handles are too small)

@amrbashir
Copy link
Member Author

Can we merge this and get it in the hand of users to test so we can see if there is any bugs or if the border size needs a bit of tweaking?

@FabianLars
Copy link
Member

fine with me

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