view.c: refactor

This commit is contained in:
Johan Malm 2020-05-16 12:04:12 +01:00
parent fdc7b4f1f3
commit 778e4e890b
6 changed files with 83 additions and 84 deletions

View file

@ -141,13 +141,13 @@ void xwl_surface_new(struct wl_listener *listener, void *data);
bool view_want_deco(struct view *view);
void view_focus_last_toplevel(struct server *server);
void focus_view(struct view *view, struct wlr_surface *surface);
void view_focus_next_toplevel(struct server *server);
void view_focus_next_toplevel(struct view *current);
void begin_interactive(struct view *view, enum cursor_mode mode,
uint32_t edges);
bool is_toplevel(struct view *view);
struct view *desktop_view_at(struct server *server, double lx, double ly,
struct wlr_surface **surface, double *sx,
double *sy, int *view_area);
struct view *view_at(struct server *server, double lx, double ly,
struct wlr_surface **surface, double *sx, double *sy,
int *view_area);
/* TODO: try to refactor to remove from header file */
struct view *first_toplevel(struct server *server);

View file

@ -21,7 +21,8 @@ static void render_decorations(struct wlr_output *output, struct view *view)
box = deco_box(view, LAB_DECO_PART_TOP);
float color2[] = { 0.7, 0.2, 0.2, 0.9 };
wlr_render_rect(view->server->renderer, &box, color2, output->transform_matrix);
wlr_render_rect(view->server->renderer, &box, color2,
output->transform_matrix);
}
static void render_surface(struct wlr_surface *surface, int sx, int sy,

View file

@ -266,9 +266,9 @@ static void process_cursor_motion(struct server *server, uint32_t time)
struct wlr_seat *seat = server->seat;
struct wlr_surface *surface = NULL;
int view_area;
struct view *view = desktop_view_at(server, server->cursor->x,
server->cursor->y, &surface, &sx,
&sy, &view_area);
struct view *view = view_at(server, server->cursor->x,
server->cursor->y, &surface, &sx, &sy,
&view_area);
if (!view) {
/* If there's no view under the cursor, set the cursor image to
* a default. This is what makes the cursor image appear when
@ -353,9 +353,9 @@ void server_cursor_button(struct wl_listener *listener, void *data)
double sx, sy;
struct wlr_surface *surface;
int view_area;
struct view *view = desktop_view_at(server, server->cursor->x,
server->cursor->y, &surface, &sx,
&sy, &view_area);
struct view *view = view_at(server, server->cursor->x,
server->cursor->y, &surface, &sx, &sy,
&view_area);
if (event->state == WLR_BUTTON_RELEASED) {
/* If you released any buttons, we exit interactive move/resize
* mode. */

137
view.c
View file

@ -1,24 +1,21 @@
#include "labwc.h"
bool view_want_deco(struct view *view)
bool is_toplevel(struct view *view)
{
if (view->type != LAB_XWAYLAND_VIEW)
switch (view->type) {
case LAB_XDG_SHELL_VIEW:
return view->xdg_surface->role == WLR_XDG_SURFACE_ROLE_TOPLEVEL;
case LAB_XWAYLAND_VIEW:
return xwl_nr_parents(view) > 0 ? false : true;
}
return false;
if (!is_toplevel(view))
return false;
if (view->xwayland_surface->override_redirect)
return false;
if (view->xwayland_surface->decorations !=
WLR_XWAYLAND_SURFACE_DECORATIONS_ALL)
return false;
return true;
}
static struct view *last_toplevel(struct server *server)
struct view *first_toplevel(struct server *server)
{
struct view *view;
wl_list_for_each_reverse (view, &server->views, link) {
wl_list_for_each (view, &server->views, link) {
if (!view->been_mapped) {
continue;
}
@ -30,16 +27,53 @@ static struct view *last_toplevel(struct server *server)
return NULL;
}
static struct view *last_toplevel(struct server *server)
{
struct view *view;
wl_list_for_each_reverse (view, &server->views, link) {
if (!view->been_mapped)
continue;
if (is_toplevel(view))
return view;
}
/* no top-level view */
return NULL;
}
void view_focus_last_toplevel(struct server *server)
{
/* TODO: write view_nr_toplevel_views() */
if (wl_list_length(&server->views) < 2) {
if (wl_list_length(&server->views) < 2)
return;
}
struct view *view = last_toplevel(server);
focus_view(view, view->surface);
}
static struct view *next_toplevel(struct view *current)
{
struct view *tmp = current;
do {
tmp = wl_container_of(tmp->link.next, tmp, link);
} while (!tmp->been_mapped || !is_toplevel(tmp));
return tmp;
}
/* Original function name - suggest we keep it */
void view_focus_next_toplevel(struct view *current)
{
struct view *view;
view = next_toplevel(current);
focus_view(view, view->surface);
}
static void move_to_front(struct view *view)
{
wl_list_remove(&view->link);
wl_list_insert(&view->server->views, &view->link);
}
static void activate_view(struct view *view)
{
if (view->type == LAB_XDG_SHELL_VIEW) {
@ -58,6 +92,8 @@ static void activate_view(struct view *view)
*/
static void set_activated(struct wlr_surface *s, bool activated)
{
if (!s)
return;
if (wlr_surface_is_xdg_surface(s)) {
struct wlr_xdg_surface *previous;
previous = wlr_xdg_surface_from_wlr_surface(s);
@ -69,18 +105,12 @@ static void set_activated(struct wlr_surface *s, bool activated)
}
}
static void move_to_front(struct view *view)
{
wl_list_remove(&view->link);
wl_list_insert(&view->server->views, &view->link);
}
/* FIXME: why have we got both view and surface here???? */
void focus_view(struct view *view, struct wlr_surface *surface)
{
/* Note: this function only deals with keyboard focus. */
if (view == NULL) {
if (!view)
return;
}
struct server *server = view->server;
struct wlr_seat *seat = server->seat;
struct wlr_surface *prev_surface;
@ -115,22 +145,18 @@ void focus_view(struct view *view, struct wlr_surface *surface)
&keyboard->modifiers);
}
static struct view *next_toplevel(struct view *current)
bool view_want_deco(struct view *view)
{
struct view *tmp = current;
do {
tmp = wl_container_of(tmp->link.next, tmp, link);
} while (!tmp->been_mapped || !is_toplevel(tmp));
return tmp;
}
void view_focus_next_toplevel(struct server *server)
{
struct view *view;
view = first_toplevel(server);
view = next_toplevel(view);
focus_view(view, view->surface);
if (view->type != LAB_XWAYLAND_VIEW)
return false;
if (!is_toplevel(view))
return false;
if (view->xwayland_surface->override_redirect)
return false;
if (view->xwayland_surface->decorations !=
WLR_XWAYLAND_SURFACE_DECORATIONS_ALL)
return false;
return true;
}
void begin_interactive(struct view *view, enum cursor_mode mode, uint32_t edges)
@ -175,34 +201,7 @@ void begin_interactive(struct view *view, enum cursor_mode mode, uint32_t edges)
}
}
bool is_toplevel(struct view *view)
{
switch (view->type) {
case LAB_XWAYLAND_VIEW:
return xwl_nr_parents(view) > 0 ? false : true;
case LAB_XDG_SHELL_VIEW:
return view->xdg_surface->role == WLR_XDG_SURFACE_ROLE_TOPLEVEL;
}
return false;
}
struct view *first_toplevel(struct server *server)
{
struct view *view;
wl_list_for_each (view, &server->views, link) {
if (!view->been_mapped) {
continue;
}
if (is_toplevel(view)) {
return view;
}
}
fprintf(stderr, "warn: found no toplevel view (%s)\n", __func__);
return NULL;
}
static bool view_at(struct view *view, double lx, double ly,
static bool _view_at(struct view *view, double lx, double ly,
struct wlr_surface **surface, double *sx, double *sy)
{
/*
@ -241,9 +240,9 @@ static bool view_at(struct view *view, double lx, double ly,
return false;
}
struct view *desktop_view_at(struct server *server, double lx, double ly,
struct wlr_surface **surface, double *sx,
double *sy, int *view_area)
struct view *view_at(struct server *server, double lx, double ly,
struct wlr_surface **surface, double *sx, double *sy,
int *view_area)
{
/*
* This iterates over all of our surfaces and attempts to find one under
@ -252,7 +251,7 @@ struct view *desktop_view_at(struct server *server, double lx, double ly,
*/
struct view *view;
wl_list_for_each (view, &server->views, link) {
if (view_at(view, lx, ly, surface, sx, sy))
if (_view_at(view, lx, ly, surface, sx, sy))
return view;
if (!view_want_deco(view))
continue;

2
xdg.c
View file

@ -61,7 +61,7 @@ void xdg_surface_unmap(struct wl_listener *listener, void *data)
{
struct view *view = wl_container_of(listener, view, unmap);
view->mapped = false;
view_focus_next_toplevel(view->server);
view_focus_next_toplevel(view);
}
void xdg_surface_destroy(struct wl_listener *listener, void *data)

3
xwl.c
View file

@ -49,8 +49,7 @@ void xwl_surface_unmap(struct wl_listener *listener, void *data)
{
struct view *view = wl_container_of(listener, view, unmap);
view->mapped = false;
if (is_toplevel(view))
view_focus_next_toplevel(view->server);
view_focus_next_toplevel(view);
}
void xwl_surface_destroy(struct wl_listener *listener, void *data)