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

disable rounded corners macos #2182

Open
jakenvac opened this issue Jun 26, 2022 · 11 comments · May be fixed by #6587
Open

disable rounded corners macos #2182

jakenvac opened this issue Jun 26, 2022 · 11 comments · May be fixed by #6587
Labels
enhancement New feature or request PR-welcome Wez doesn't have plans to work on this, but will accept a PR if someone feels motivated!

Comments

@jakenvac
Copy link
Contributor

Hi, I've seen discussion about making the macos build preserve the rounded corners when the title bar is disabled here: #1034, I'm assuming the squared corners were originally considered a bug?

Is it possible to restore the squared corners in macos via a setting? If not, I think it would be a good idea to extend the window decorations setting to include a corners option.

@jakenvac jakenvac added the enhancement New feature or request label Jun 26, 2022
@wez wez added the PR-welcome Wez doesn't have plans to work on this, but will accept a PR if someone feels motivated! label Jul 19, 2022
@wez
Copy link
Owner

wez commented Jul 19, 2022

Yeah, all the windows are rounded on macOS, so I considered it a bug. I'll accept a PR for something around this. I'm not super sure what the config should look like. Maybe add a MACOS_SQUARE_CORNERS to window decorations?

@rockyzhang24
Copy link

I think we can introduce SQUARE_CORNER. The existing flags NONE, TITLE, RESIZE and TITLE | RESIZE can be preserved, i.e., the rounded corner is the default. If users need the square corner, they can just append it to the corresponding flags. E.g., NONE | SQUARE_CORNER, RESIZE | SQUARE_CORNER, etc.

@wez
Copy link
Owner

wez commented Jul 29, 2022

An issue with just using SQUARE_CORNER is that it implies that not specifying it will give something other than square corners, but most systems only have square corners, and there is not a portable way to know or specify which should be used.

I think MACOS_FORCE_SQUARE_CORNERS is a more appropriately scoped name and effect.

@rockyzhang24
Copy link

Oh yes, exactly. MACOS_FORCE_SQUARE_CORNERS is proper flag. I second it.

@benfrain
Copy link

Another vote for MACOS_FORCE_SQUARE_CORNERS flag. Afraid I have zero Rust chops to help out though, sorry!

@wez
Copy link
Owner

wez commented Feb 1, 2023

FWIW, I think this is more a matter of figuring out how to get macos to disable rounded corners with OpenGL and Metal views.

These are relevant questions with no obvious answer:

https://github.com/lukakerr/NSWindowStyles#6-vibrant-background-with-custom-border-radius-and-no-titlebar might be adaptable for this purpose by setting the corner radius to 0.

I don't have time to actually sit down and play with this; I just happened to be looking at window_decorations tonight as part of #2669

@karambaq
Copy link

Is there a way to made look wez like iterm? Without title bar but with close/expand buttons? Like this:

image

@wez
Copy link
Owner

wez commented Mar 21, 2023

@karambaq not yet, but keep you eye on this PR:

@IdoEsports
Copy link

hey @wez any update on the corner radius setting?

@dfl0
Copy link

dfl0 commented May 24, 2024

Hey @wez, I was going to create an issue for this but noticed this one was asking exactly the same thing I was going to. Any plans to implement this at some point now? Is it a matter of difficulty or necessity?

@wez
Copy link
Owner

wez commented May 25, 2024

the PR-welcome tag indicates that I'm happy to review a PR produced by someone in the community, but that I have no plans to work directly on this myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request PR-welcome Wez doesn't have plans to work on this, but will accept a PR if someone feels motivated!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants