-
Beta Was this translation helpful? Give feedback.
Replies: 3 comments
-
Hi @qskousen Thank you for pointing this out. |
Beta Was this translation helpful? Give feedback.
-
I see that this is this code here in
If I am correct, this whole function (and references to it) can be removed, and there will be no more of that margin. Would it be alright if I made a PR for that? Edit: I did try out just that and it seems to work correctly with the demo app. I am guessing in this case the style for the border width to jump to 2 when maximized in |
Beta Was this translation helpful? Give feedback.
-
Sure. Feel free to create a PR for this
…On Fri, Jun 27, 2025, 2:09 AM qskousen ***@***.***> wrote:
I see that this is this code here in Window.axaml.cs:OnWindowStateChanged:
case WindowState.Maximized:
ToggleMaxButtonVisibility(CanMaximize);
Margin = new Thickness(7);
break;
If I am correct, this whole function (and references to it) can be
removed, and there will be no more of that margin. Would it be alright if I
made a PR for that?
—
Reply to this email directly, view it on GitHub
<#24 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEKE6B2RRHL6TMVJAG7WOSL3FQZONAVCNFSM6AAAAACAGYLDRCVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTGNJYHEYDMNA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
Hi @qskousen
Thank you for pointing this out.
When I started this project, I thought this border is better to be shown on maximize.😂. Anyway, after observing other apps, this is best to be remove.