-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add inner rounding of corners #54
Comments
Yep, I am waiting for that, the only reason I don't use sway, is no way of making window corners rounded and making that with border and images is not good, that doesn't look nice on all apps |
That looks so good, are you planning on adding anti aliasing to it ? |
Which branch are you working in? This looks sick! |
Yes.
Haven't pushed any commits yet; this is literally editing the shaders used by the GLES2 renderer in wlroots. I'll be creating a custom renderer with its own shaders so a different wlroots version won't be required. I've yet to do much thinking on how containers are going to work though. |
Are you talking about stacked and tabbed windows? |
Precisely. |
Yeah, you don't want something like this: https://www.reddit.com/r/i3wm/comments/ndo9iy/i_am_using_i3gaps_with_picom_and_this_is_how_my/ |
Well that is exactly what I'm trying to avoid, but given picom does that right now, maybe I don't need to worry about it just yet. |
By default the titlebar text is centered right? Why not make it rounded too and add some padding to the text so that it won't get clipped |
Any updates on this? |
I don't wanna be annoying, but have you got a branch or something we could play around with? I personally would not care about any bleeding edge stuff as long as it means I can use rounded corners. I need that shit. |
I've pushed https://github.com/fluix-dev/sway-borders/tree/beta so you can play around with. It's based on upstream Sway, not this fork so it's super bare and limited, but you can set inner, rounded borders. You can set the color here: sway-borders/sway/desktop/renderer.c Line 45 in 970b52a
With the format The radius can be set by replacing the sway-borders/sway/desktop/render.c Line 184 in 970b52a
My main issue is figuring out how to have everything (inner rounding, outer padding, and future work like blur) work together, but I think I've been overthinking it. I'm going to try and just use the existing code + adding on the work in this branch for inner padding, and configuration changes and see how that works out. Maybe I'll have to rewrite it again when adding blur, but that's better than having nothing for months/years at a time. |
Hmmm. Won't compile for me due to: #include <render/gles2.h>
#include <render/egl.h> Am I missing something? Edit: Seems like |
You are amazing dude! Trying it out today! |
@ErikReider you were correct, those functions were made private some time ago. I've pushed some more WIP commits to the beta branch, so please give those a try at compilation. The branch has (for now) lost its border colours (as I work on putting them on the outside of windows) but gained anti-aliasing (see below)! Radius is hardcoded to |
any hope for inner round corners yet ? it will be awesome to have round corners for floating windows without any border
The text was updated successfully, but these errors were encountered: