louib authored 3 years ago and Sebastian Krzyszkowiak committed 3 years ago

This commit adds the `build-dep` one-liner to install the dependencies
by leveraging the debian control file. The instructions are from
the same section in phosh's README. Also fixes a typo in the building
section, which seems to be from phosh.

See https://github.com/swaywm/wlroots/pull/1919

Move the cursor under the finger until touch points receive their own
cursors.

Otherwise the old cursor stays there until the pointer gets moved.

This also removes the destroy signal listener of the pending surface.

There's no need to redraw unactivated clients since they're
hidden anyway, so that just wastes resources. It also looks
bad in phosh activity chooser.

Sebastian Krzyszkowiak authored 4 years ago and Guido Gunther committed 4 years ago

The actual window geometry should stay in the same position
regardless of surface geometry being changed.

Sebastian Krzyszkowiak authored 4 years ago and Guido Gunther committed 4 years ago

We're interested in restoring the actual window into the same
position, regardless of whether the shadows around it happened
to change their size meanwhile.

Sebastian Krzyszkowiak authored 4 years ago and Guido Gunther committed 4 years ago

This makes it consistent with PHOC_EDGE_SNAP_THRESHOLD and actually makes
more sense.

This follows the convention used by other projects like phosh.

We've been using proper wlroots packaging for a long time now.

This allows them to run as soon as the dependency is fulfilled, before
the previous stage has been completed.

This ensures that the view is still on the list while it may get
damaged by children handlers - otherwise phoc_desktop_view_is_visible
won't be able to work correctly.

Fixes #191

If the view is unmapped, we can tell that it's not visible right
away. Later assertion assumes that there's at least one view on
the desktop->views list, which is a valid assumption to make only
if the view we got is actually mapped.

Should fix #191

Sebastian Krzyszkowiak authored 4 years ago and Guido Gunther committed 4 years ago

Every wlr_output is going to have an associated PhocOutput, but
wlr_output_layout_output_at is not guaranteed to always return
a non-NULL value. Therefore it doesn't make much sense to check
for `output` being NULL right after unchecked dereference
of `wlr_output`.

It's useful to have when testing custom wlroots builds on the phone.

We still want to be able to reveal shell when there's no surface
at the edge of the screen - which can happen when the fullscreen
surface's size doesn't cover the whole output.

Some apps open windows with preferred size that are larger than
the phone screen, but that can be resized down manually to fit.
There's no reason not to try to make them fit on the screen
automatically, so do just that when centering views.

It's simpler and does the same thing :)

Sebastian Krzyszkowiak authored 4 years ago and Guido Gunther committed 4 years ago

It's not really a useful feature to have in a mobile compositor and just
makes positioning code more complicated for no good reason. It wasn't
even complete, as evidenced by TODOs being removed with it. Support for
iterating over and drawing rotated surfaces is retained as it may still
be useful, but we sure don't need it at the view level.

Sebastian Krzyszkowiak authored 4 years ago and Guido Gunther committed 4 years ago

Setting force_shell_reveal changes the contents of the output, so
it needs to be damaged. It is already done in all the other places
where it's changed.

Sebastian Krzyszkowiak authored 4 years ago and Guido Gunther committed 4 years ago

For consistency.