| Age | Commit message (Collapse) | Author | Files | Lines |
|
We now send some protocol errors that wlroots 0.16 is missing [1].
This also allows us to access the xdg decoration from a view, which will
be necessary for some future changes.
[1]: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/4051
|
|
|
|
How river currently sets this isn't really in accordance with the spirit
of the protocol. It was originally done this way to get gtk3 windows to
look a little bit better with borders drawn around them. However, I've
come to believe that river shouldn't just ignore standards like this.
The right way to do things would be to either implement the
xdg-decoration protocol for gtk properly or to be pragmatic and accept
some programs are intended to be used with CSD and that's OK.
|
|
River does not currently recognize Ctrl as a modifier, only Control.
|
|
|
|
|
|
This is some dead code that is left over from before the scene graph
changes.
|
|
This brings the behavior closer to what we had before the scene
graph refactor.
The main difference now is that the order has changed from background to
overlay instead of from overlay to background. This ordering seems to
make more sense in the cases I've tested and the old ordering was just
cargo-cult anyways.
|
|
|
|
|
|
Well, at least as much of it as was implemented before the scene graph
refactor.
|
|
|
|
Now with 50% less pointer warping!
The new implementation requires the user to move the cursor into the
constraint region before the constraint is activated in order to keep
behavior more predictable.
|
|
This looks to be a typo made way back in the early days of river.
|
|
|
|
|
|
This was previously used to create a noop output, but we no longer rely
on that hack.
|
|
This replaces the old View.fromWlrSurface function and is more general.
This commit also moves the xdg activation request_activate listener to
Server as it has no reason to be in View.
|
|
|
|
|
|
This is more reliable since it uses absolute coordinates instead of a
relative movement which could cause the cursor position to get out of
sync with the view.
This is the same approach used for resize.
|
|
- Move the decision whether a configure should be tracked or not into
the xdg toplevel/xwayland code.
- Only track configures for xdg toplevels with the transaction system
if the dimensions of the view are affected.
|
|
|
|
Co-authored-by: Leon Henrik Plickat <leonhenrik.plickat@stud.uni-goettingen.de>
|
|
|
|
|
|
Currently we may resize fullscreen views when they become visible/not
visible when switching tags even if their fullscreen state remains
constant. This is suboptimal, and as it turns out also much more complex
to implement.
|
|
We need to initialize the geometry on map to ensure the first commit is
handled correctly.
Also we don't care about the x/y of the geometry, only the width/height.
|
|
|
|
|
|
|
|
|
|
This reduces the number of separate configure events sent to clients
through better batching and is also more correct.
|
|
|
|
In commitTransaction() we currently the current view state to determine
whether or not to enable the view's scene tree. However we don't update
the view's current state until after that check.
|
|
|
|
This state affects rendering, so it should pass through the transaction
system like the rest.
|
|
Moving fullscreen views between outputs now works properly.
A case in which we did not inform the client that it is no longer
fullscreen has been fixed as well.
|
|
The race is as follows:
1. Output A commits and sets render state to pending_lock_surface
2. Output B commits and sets render state to pending_lock_surface
3. Output A presents and sets render state to lock_surface
4. maybeLock() does not lock because waiting on output B
5. Output A commits and sets render state to pending_lock_surface
6. Output B presents and sets render state to lock_surface
4. maybeLock() does not lock because waiting on output A
|
|
Having this in a separate file no longer makes sense since the switch to
the scene graph.
|
|
|
|
A transaction is now necessary to update border state.
|
|
|
|
The scene_layer_surface may be destroyed before handleDestroy is called,
which means we can't rely on it to access the wlr_layer_surface in
destroyPopups().
|
|
This was motivated by the view tags not being proplerly updated by
Root.commitTransaction() when there were no views on an output.
|
|
|
|
|
|
|
|
|
|
|