From: Erico Nunes <ernunes@redhat.com>
To: "Marc-André Lureau" <marcandre.lureau@gmail.com>,
"Kasireddy, Vivek" <vivek.kasireddy@intel.com>
Cc: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Subject: Re: [PATCH 1/2] ui/gtk: use widget size for cursor motion event
Date: Thu, 30 Mar 2023 16:08:35 +0200 [thread overview]
Message-ID: <27c3c14c-6d91-a01f-94c7-c874f096665d@redhat.com> (raw)
In-Reply-To: <CAJ+F1CK28C4EL=abcJWo3TOZqsMxdTnrbE0Y40Jabn-vFXaBbg@mail.gmail.com>
On 23/03/2023 15:41, Marc-André Lureau wrote:
> Should we queue the 2 patches from this series? (note that they were
> not correctly handled by patchew, probably because you dropped the
> cover letter).
>
> For me -display gtk is unusable on hidpi & wayland anyway, because the
> cursor position given to the guest does not match the dimensions given
> for the monitor.
>
> Also relative mouse support is broken as well (mouse wrapping and
> confinement/grab is not supported by gdk/gtk on wayland).
>
> I am not actively looking at these problems, they are "solved" with
> spice (use -display spice-app). And I am also regularly working on a
> gtk4/rust widget, using -display dbus
> (https://gitlab.gnome.org/malureau/rdw). There is also
> https://gitlab.gnome.org/chergert/libmks as a gtk4/C alternative. I am
> not sure we should keep maintaining the gtk3 backend going forward.
> And as a Gtk4/-display dbus client mature, I hope we can offer a
> better alternative than ui/sdl or ui/cocoa on other platforms as well.
To answer this: well since we already have the fixes, if it gets reviews
I think we could merge them anyway.
But realizing that the built-in gtk UI is in this best-effort state, I
probably won't be spending much time on it anymore either.
It seems to be more productive to focus the effort on improving the
experience with the dbus backend and its UIs going forward, then.
Thanks
Erico
prev parent reply other threads:[~2023-03-30 14:08 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-20 16:08 [PATCH 1/2] ui/gtk: use widget size for cursor motion event Erico Nunes
2023-03-20 16:08 ` [PATCH 2/2] ui/gtk-egl: fix scaling for cursor position in scanout mode Erico Nunes
2023-03-21 3:29 ` [PATCH 1/2] ui/gtk: use widget size for cursor motion event Kasireddy, Vivek
2023-03-22 16:10 ` Erico Nunes
2023-03-23 5:01 ` Kasireddy, Vivek
2023-03-23 14:41 ` Marc-André Lureau
2023-03-30 14:08 ` Erico Nunes [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=27c3c14c-6d91-a01f-94c7-c874f096665d@redhat.com \
--to=ernunes@redhat.com \
--cc=marcandre.lureau@gmail.com \
--cc=qemu-devel@nongnu.org \
--cc=vivek.kasireddy@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).