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

Wlroots 0.18.1 #55

Merged
merged 14 commits into from
Nov 12, 2024
Merged

Wlroots 0.18.1 #55

merged 14 commits into from
Nov 12, 2024

Conversation

ErikReider
Copy link
Member

@ErikReider ErikReider commented Jul 29, 2024

TinyWL runs. Can't really test Sway yet though.

  • Update to 0.18.1

Requires: #54

Working implementation:
ErikReider/fx-comp#9

@ErikReider ErikReider added the enhancement New feature or request label Jul 29, 2024
@ErikReider ErikReider requested a review from WillPower3309 July 29, 2024 19:26
@ErikReider ErikReider self-assigned this Jul 29, 2024
@ErikReider ErikReider marked this pull request as ready for review July 30, 2024 21:08
protocol/meson.build Outdated Show resolved Hide resolved
@ErikReider
Copy link
Member Author

ErikReider commented Sep 11, 2024

@WillPower3309 Should we also bump the scenefx version as well? (To make it easier for packagers and comp that use older versions of wlroots)

@WillPower3309
Copy link
Member

@WillPower3309 Should we also bump the scenefx version as well? (To make it easier for packagers and comp that use older versions of wlroots)

Yup! Need to bump for the last wlroots bump then will merge this

@ErikReider
Copy link
Member Author

Tested it in ErikReider/fx-comp#9 and everything seems to be working as expected! :D

@ErikReider ErikReider marked this pull request as draft September 21, 2024 00:34
@ErikReider ErikReider changed the title Wlroots 0.18.0 Wlroots 0.18.1 Sep 21, 2024
@ErikReider ErikReider marked this pull request as ready for review October 2, 2024 12:46
@WillPower3309 WillPower3309 linked an issue Oct 15, 2024 that may be closed by this pull request
@0xNULLderef
Copy link

i know this is well ahead of the curve, but trying on the latest wlroots 0.19 seems like removal of XDG clipping makes all the shadows and rounded corners wonky on gtk like the comment on it mentioned... did a bad job backporting on https://github.com/0xNULLderef/scenefx/tree/wlroots-0.19 for the sake of my own config but are there any plans to tackle this? i see the opaque region being defined but i have no idea how it relates to the actual rendered window contents

tested apps are meld and waterfox (should be the same as firefox) that show a padding around the window and no rounded corners (tinywl from src tree)
eglgears_wayland and kitty did not have such issues as i think they use the raw wl egl surface

@ErikReider
Copy link
Member Author

i know this is well ahead of the curve, but trying on the latest wlroots 0.19 seems like removal of XDG clipping makes all the shadows and rounded corners wonky on gtk like the comment on it mentioned... did a bad job backporting on https://github.com/0xNULLderef/scenefx/tree/wlroots-0.19 for the sake of my own config but are there any plans to tackle this? i see the opaque region being defined but i have no idea how it relates to the actual rendered window contents

So we don't only officially support their main branch due to them not actually having a release of 0.19 yet, so we haven't checked for compatibility as of yet. So you'll kinda be on your own on this front until we port it to 0.19 in the coming months

I read that they didn't remove the geometry, but moved it instead. Have you tried using that property?

eglgears_wayland and kitty did not have such issues as i think they use the raw wl egl surface

This is due to client side decoration. The GTK toplevels have decorations like shadows and grabbing edges that affect the size of the toplevel if not compensated for, using it's geometry field :)

@WillPower3309
Copy link
Member

needs rebase / merge from main

@WillPower3309 WillPower3309 merged commit 398761e into main Nov 12, 2024
@WillPower3309 WillPower3309 deleted the wlroots-0.18 branch November 12, 2024 23:18
WillPower3309 pushed a commit that referenced this pull request Nov 14, 2024
WillPower3309 pushed a commit that referenced this pull request Nov 14, 2024
ErikReider added a commit that referenced this pull request Nov 15, 2024
* Add wlr_scene blur impl

* Added initial implementation of per output optimized blur

* Added ability to only re-render optimized blur on a specific output

* Fixed segfault if scene_buffer doesn't contain a scene_surface

* Added support for wlr_scene_subsurface_tree_set_clip. Fixes blur not being clipped after #56

* Added optimized blur example to tinywl

* Fixed blur opaque region not being correct for non scene_surfaces

* Renamed wlr_scene_blur to wlr_scene_optimized_blur

* Fixed optimized blur not being clipped

* Renamed enum WLR_SCENE_NODE_BLUR -> WLR_SCENE_NODE_OPTIMIZED_BLUR

* Fixed silly bug where setting num_passes to the default value wouldn't update

* Translate toplevel opaque_region to compensate for GTK CSD

* Set optimized blur node to always visible

* Fixed not optimized blur being restricted to damage

* Fixed blur corner rounding not matching scene_buffer

* Renamed should_blur -> scene_buffer_is_blur_enabled

* Simplified method of getting buffer opaque region

* Don't always update the optimized blur when setting buffer blur state

* Skip re-rendering optimized blur when the blur node is disabled

* Fixed blur detection only working for wlr_scene_surface nodes

* Use transformed output size when extending blur damage

* Don't update the scene node twice when updating optimized size

* TinyWL: skip commit on initial_commit

Fixes setting size of border and shadow nodes too early, leading to the
optimized blur example not working with GTK4 toplevels.

* Rebase: wlroots 0.18.1 (#55)

* updated flake for wlroots 0.18

* rebase: wlroots 0.18.1 (#55)

* rebase fix

* fix2

* Lowered version to 0.1

I'll let Will bump the version instead :)

---------

Co-authored-by: William McKinnon <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wlroots 0.18
3 participants