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

Can't resize window on Linux #15

Closed
davidmartos96 opened this issue Mar 26, 2021 · 8 comments
Closed

Can't resize window on Linux #15

davidmartos96 opened this issue Mar 26, 2021 · 8 comments

Comments

@davidmartos96
Copy link

davidmartos96 commented Mar 26, 2021

Hello! I have tried the Linux app and it works great, nice job!
I found an issue though, I'm not able to resize the window. I figure this is not intended because you set Size.infinite for the max window size.
It looks like that is disabling the ability to resize at all.

As you can see here the Maximize and Resize buttons are grayed out.

image

@aguilaair
Copy link
Member

I've experienced the same issue too, I'll take a look a little later 👍

@aguilaair
Copy link
Member

I've been experimenting with this and it is clear that setWindowMinSize & setWindowMaxSize are causing issues when resizing in linux, and setting either one will cause the bug to appear. The only workaround at this point in time I think is simply not setting these parameters in Linux

@aguilaair
Copy link
Member

This is a known issue at google/flutter-desktop-embedding#844

@davidmartos96
Copy link
Author

@aguilaair Interesting. So does it fail also by commenting out the setMaxSize line? I thought the problem was about Size.inifnite.

@aguilaair
Copy link
Member

@aguilaair Interesting. So does it fail also by commenting out the setMaxSize line? I thought the problem was about Size.inifnite.

Yep, still fails AKAIK

@aguilaair
Copy link
Member

This issue has now been solved. @leoafarias, mind closing it?

@leoafarias
Copy link
Member

@aguilaair do you know which release channel is on? Want to make sure the lates build uses that version.

@aguilaair
Copy link
Member

It's resolved on master, so there should be no change required. Just in case you can change https://github.com/leoafarias/sidekick/blob/main/pubspec.lock#L878 to 57a2cd88486105d3d813583339eb0e23c9bfd6d2

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

No branches or pull requests

3 participants