Merge pull request #414 from emersion/fix-hidpi-rotated-input

Fix rotated views on HiDPI outputs
This commit is contained in:
Drew DeVault 2017-11-12 09:56:13 -05:00 committed by GitHub
commit 38e14de240
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 28 additions and 24 deletions

View file

@ -229,7 +229,7 @@ void view_teardown(struct roots_view *view) {
struct roots_view *view_at(struct roots_desktop *desktop, double lx, double ly, struct roots_view *view_at(struct roots_desktop *desktop, double lx, double ly,
struct wlr_surface **surface, double *sx, double *sy) { struct wlr_surface **surface, double *sx, double *sy) {
for (int i = desktop->views->length - 1; i >= 0; --i) { for (ssize_t i = desktop->views->length - 1; i >= 0; --i) {
struct roots_view *view = desktop->views->items[i]; struct roots_view *view = desktop->views->items[i];
if (view->type == ROOTS_WL_SHELL_VIEW && if (view->type == ROOTS_WL_SHELL_VIEW &&
@ -241,11 +241,12 @@ struct roots_view *view_at(struct roots_desktop *desktop, double lx, double ly,
double view_sx = lx - view->x; double view_sx = lx - view->x;
double view_sy = ly - view->y; double view_sy = ly - view->y;
struct wlr_surface_state *state = view->wlr_surface->current;
struct wlr_box box = { struct wlr_box box = {
.x = 0, .x = 0,
.y = 0, .y = 0,
.width = view->wlr_surface->current->buffer_width, .width = state->buffer_width / state->scale,
.height = view->wlr_surface->current->buffer_height, .height = state->buffer_height / state->scale,
}; };
if (view->rotation != 0.0) { if (view->rotation != 0.0) {
// Coordinates relative to the center of the view // Coordinates relative to the center of the view
@ -259,7 +260,6 @@ struct roots_view *view_at(struct roots_desktop *desktop, double lx, double ly,
} }
if (view->type == ROOTS_XDG_SHELL_V6_VIEW) { if (view->type == ROOTS_XDG_SHELL_V6_VIEW) {
// TODO: test if this works with rotated views
double popup_sx, popup_sy; double popup_sx, popup_sy;
struct wlr_xdg_surface_v6 *popup = struct wlr_xdg_surface_v6 *popup =
wlr_xdg_surface_v6_popup_at(view->xdg_surface_v6, wlr_xdg_surface_v6_popup_at(view->xdg_surface_v6,
@ -274,7 +274,6 @@ struct roots_view *view_at(struct roots_desktop *desktop, double lx, double ly,
} }
if (view->type == ROOTS_WL_SHELL_VIEW) { if (view->type == ROOTS_WL_SHELL_VIEW) {
// TODO: test if this works with rotated views
double popup_sx, popup_sy; double popup_sx, popup_sy;
struct wlr_wl_shell_surface *popup = struct wlr_wl_shell_surface *popup =
wlr_wl_shell_surface_popup_at(view->wl_shell_surface, wlr_wl_shell_surface_popup_at(view->wl_shell_surface,

View file

@ -20,30 +20,33 @@ static void render_surface(struct wlr_surface *surface,
struct roots_desktop *desktop, struct wlr_output *wlr_output, struct roots_desktop *desktop, struct wlr_output *wlr_output,
struct timespec *when, double lx, double ly, float rotation) { struct timespec *when, double lx, double ly, float rotation) {
if (surface->texture->valid) { if (surface->texture->valid) {
float scale_factor = (float)wlr_output->scale / surface->current->scale; double surface_scale = surface->current->scale;
int width = surface->current->buffer_width * scale_factor; double width = (double)surface->current->buffer_width / surface_scale;
int height = surface->current->buffer_height * scale_factor; double height = (double)surface->current->buffer_height / surface_scale;
int render_width = width * wlr_output->scale;
int render_height = height * wlr_output->scale;
double ox = lx, oy = ly; double ox = lx, oy = ly;
wlr_output_layout_output_coords(desktop->layout, wlr_output, &ox, &oy); wlr_output_layout_output_coords(desktop->layout, wlr_output, &ox, &oy);
ox *= wlr_output->scale; ox *= wlr_output->scale;
oy *= wlr_output->scale; oy *= wlr_output->scale;
if (wlr_output_layout_intersects(desktop->layout, wlr_output, if (wlr_output_layout_intersects(desktop->layout, wlr_output,
lx, ly, lx + width, ly + height)) { lx, ly, lx + render_width, ly + render_height)) {
float matrix[16]; float matrix[16];
float translate_origin[16]; float translate_origin[16];
wlr_matrix_translate(&translate_origin, wlr_matrix_translate(&translate_origin,
(int)ox + width / 2, (int)oy + height / 2, 0); (int)ox + render_width / 2, (int)oy + render_height / 2, 0);
float rotate[16]; float rotate[16];
wlr_matrix_rotate(&rotate, rotation); wlr_matrix_rotate(&rotate, rotation);
float translate_center[16]; float translate_center[16];
wlr_matrix_translate(&translate_center, -width / 2, -height / 2, 0); wlr_matrix_translate(&translate_center, -render_width / 2,
-render_height / 2, 0);
float scale[16]; float scale[16];
wlr_matrix_scale(&scale, width, height, 1); wlr_matrix_scale(&scale, render_width, render_height, 1);
float transform[16]; float transform[16];
wlr_matrix_mul(&translate_origin, &rotate, &transform); wlr_matrix_mul(&translate_origin, &rotate, &transform);
@ -51,8 +54,8 @@ static void render_surface(struct wlr_surface *surface,
wlr_matrix_mul(&transform, &scale, &transform); wlr_matrix_mul(&transform, &scale, &transform);
wlr_matrix_mul(&wlr_output->transform_matrix, &transform, &matrix); wlr_matrix_mul(&wlr_output->transform_matrix, &transform, &matrix);
wlr_render_with_matrix(desktop->server->renderer, wlr_render_with_matrix(desktop->server->renderer, surface->texture,
surface->texture, &matrix); &matrix);
struct wlr_frame_callback *cb, *cnext; struct wlr_frame_callback *cb, *cnext;
wl_list_for_each_safe(cb, cnext, wl_list_for_each_safe(cb, cnext,
@ -64,19 +67,20 @@ static void render_surface(struct wlr_surface *surface,
struct wlr_subsurface *subsurface; struct wlr_subsurface *subsurface;
wl_list_for_each(subsurface, &surface->subsurface_list, parent_link) { wl_list_for_each(subsurface, &surface->subsurface_list, parent_link) {
double sx = subsurface->surface->current->subsurface_position.x, struct wlr_surface_state *state = subsurface->surface->current;
sy = subsurface->surface->current->subsurface_position.y; double sx = state->subsurface_position.x;
double sw = subsurface->surface->current->buffer_width, double sy = state->subsurface_position.y;
sh = subsurface->surface->current->buffer_height; double sw = state->buffer_width / state->scale;
double sh = state->buffer_height / state->scale;
if (rotation != 0.0) { if (rotation != 0.0) {
// Coordinates relative to the center of the subsurface // Coordinates relative to the center of the subsurface
double ox = sx - (double)width/2 + sw/2, double ox = sx - width/2 + sw/2,
oy = sy - (double)height/2 + sh/2; oy = sy - height/2 + sh/2;
// Rotated coordinates // Rotated coordinates
double rx = cos(-rotation)*ox - sin(-rotation)*oy, double rx = cos(-rotation)*ox - sin(-rotation)*oy,
ry = cos(-rotation)*oy + sin(-rotation)*ox; ry = cos(-rotation)*oy + sin(-rotation)*ox;
sx = rx + (double)width/2 - sw/2; sx = rx + width/2 - sw/2;
sy = ry + (double)height/2 - sh/2; sy = ry + height/2 - sh/2;
} }
render_surface(subsurface->surface, desktop, wlr_output, when, render_surface(subsurface->surface, desktop, wlr_output, when,
@ -103,7 +107,8 @@ static void render_xdg_v6_popups(struct wlr_xdg_surface_v6 *surface,
popup->popup_state->geometry.y - popup->geometry->y; popup->popup_state->geometry.y - popup->geometry->y;
render_surface(popup->surface, desktop, wlr_output, when, popup_x, render_surface(popup->surface, desktop, wlr_output, when, popup_x,
popup_y, rotation); popup_y, rotation);
render_xdg_v6_popups(popup, desktop, wlr_output, when, popup_x, popup_y, rotation); render_xdg_v6_popups(popup, desktop, wlr_output, when, popup_x, popup_y,
rotation);
} }
} }