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

Hiding lone titlebar scenegraph #8076

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

svalaskevicius
Copy link

this is a WIP PR, for comments and hopefully others joining in taking this feature onwards :)

the render.c file was removed and the logic is switching to the scenegraph based implementation.

during my brief testing this seems to have worked, but the testing was limited as I got stumbled on another issue of the master branch - waybar was not showing after I create windows (#8075) so I reverted to the previous version for now.

a step towards resolving #8072 - hopefully :)

nim65s added a commit to nim65s/dotfiles that referenced this pull request Mar 19, 2024
Copy link

@nim65s nim65s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @svalaskevicius ! This is working for me, so I guess we could close #7889 in favor of this one.

include/sway/tree/view.h Outdated Show resolved Hide resolved
sway/tree/view.c Outdated Show resolved Hide resolved
@svalaskevicius svalaskevicius force-pushed the hiding-lone-titlebar-scenegraph branch from ecc2e16 to 07ec101 Compare March 21, 2024 18:38
@nim65s
Copy link

nim65s commented Mar 25, 2024

I found a little bug: when I open a first transparent windows (my terminal, kitty, in the current case), it doesn't have a title bar, which is expected. Then, I open a second one, and I get 2 side-by-side windows, both with titlebar, which is still expected. When I close one of the 2, the title bar of the lone window disappear, which is still the correct behavior.

But the space taken by the titlebar of the left window doesn't get its background back, and I have a black rectangle in place of my wallpaper there, on the top left corner of my screen.

It's minor, as the windows itself is behaving as expected, and only the transparent background is not updated, but we might be missing something here. I guess it's not kitty's fault, because this issues did not appear on #7889.

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

Successfully merging this pull request may close these issues.

None yet

2 participants