-
Notifications
You must be signed in to change notification settings - Fork 57
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
1.9 merge #277
1.9 merge #277
Conversation
Thanks for the work here! We'd like to use the 1.9 release to transition our fx_renderer to use the fx_renderer we've built in scenefx, which will have feature parity with the fx_renderer bundled in swayfx. As part of that, we will simply just have to import fx_renderer from scenefx and it should be plug and play for that portion of the rebase. |
I'm going to make a few additions to this branch to move along some of the WIP parts, I hope you don't mind :) |
Should just be render.c that needs fixing now |
Relies on the |
right now it is sort of functional but black screens a lot with
|
Once the buffer slot issue is fixed, we can start adding back some of the eye candy (lots of TODOs) |
Seems like there's a lot of missing code from sway 1.9, like the root.c file not compiling without manually copy/pasting the contents from sway 1.9 |
New fun bug! |
|
All that's left now are adding back the compositor effects in render.c (see the many TODOs), and scenefx |
Found another issue, when moving a floating window the bottom border can be 1 px off in some positions - likely a rounding issue EDIT: probably just #113 |
We need to have a discussion on if we should keep scaling in the scene side or make it a compositor responsibility |
fa35b26
to
6a6064f
Compare
With that in mind I think we have feature parity with upstream swayfx now, @ErikReider do you mind giving the render logic another look through? I'll do the same before merging |
Maybe a little bit off-topic, but will scenefx be tagged soon after or before this PR is merged? It will help a lot for distribution packagers. |
Yup! As soon as the corresponding scene PR is merged I will tag a release |
cc80da3
to
d99ae6d
Compare
render layoer_shell, render, and seatop_move_tiling and fx_renderer are currentlyt he main things that seem to require major changes.