Re: [PATCH spice-gtk 0/2] Make sure to set initial cursor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

----- Original Message -----
> Hi,
> 
> It can happen that the initial cursor is not set for some monitors
> (SpiceDisplays)
> in a multimonitor linux guest env (there is a single cursor channel). To fix
> the problem a new class is created to handle gtk/gdk properties of the remote
> cursor. There is a 1:1 relation between SpiceChannelCursorGtk and
> SpiceChannelCursor
> (similar as SpiceSession and SpiceGtkSession).
> 
> Maybe it is a too heavy approach. The other approach I though about is
> setting
> the cursor GdkPixbuf as data to the cursor channel. In the end I decided to
> go
> for the class, imho it is more clean.

Why not just save the last display_cursor in the cursor channel and add a "cursor" property? It would need to be a public cursor struct, introspectable etc.

Tbh, I think the cursor channel should have been designed that way (and not just reflect the protocol events). We can then later think about deprecating SET/RESET/HIDE events.

Advantage is that this could also be useful to other UI frontends (qt/e/android/etc)


> 
> Thanks for comments,
> Pavel
> 
> Resolves:
>   https://bugzilla.redhat.com/show_bug.cgi?id=1411380
> 
> git branch cursor_gtk:
>   https://github.com/xerus/spice-gtk/tree/cursor_gtk
> 
> initial approach:
>   https://lists.freedesktop.org/archives/spice-devel/2017-January/034972.html
> 
> Pavel Grunt (2):
>   Add helper class to handle cursor image
>   widget: Set cursor during construction
> 
>  src/Makefile.am          |   2 +
>  src/channel-cursor-gtk.c | 269
>  +++++++++++++++++++++++++++++++++++++++++++++++
>  src/channel-cursor-gtk.h |  52 +++++++++
>  src/spice-widget.c       |  32 +++---
>  4 files changed, 340 insertions(+), 15 deletions(-)
>  create mode 100644 src/channel-cursor-gtk.c
>  create mode 100644 src/channel-cursor-gtk.h
> 
> --
> 2.12.2
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]