[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 0/6] ui: rework -show-cursor option
From: |
Gerd Hoffmann |
Subject: |
Re: [PATCH v2 0/6] ui: rework -show-cursor option |
Date: |
Thu, 6 Feb 2020 14:20:02 +0100 |
On Thu, Feb 06, 2020 at 11:52:05AM +0000, Peter Maydell wrote:
> On Thu, 6 Feb 2020 at 11:29, Gerd Hoffmann <address@hidden> wrote:
> >
> >
>
> This cover letter is missing a description of what the patchset does...
>
> The closest thing the patchset seems to get to documentation is the
> oneliner in ui.json:
> +# @show-cursor: Force showing the mouse cursor (default: off).
>
> but looking at the ui/cocoa.m implementation that isn't what it
> actually does -- it just seems to mean "default to shown on
> startup", because the logic that unconditionally hides the host
> cursor on mousegrab and unhides it on ungrab remains
> unchanged. This doesn't on the face of it sound like very
> useful behaviour, since the option will only have an effect for
> the short period of time between QEMU startup and the first
> mouse-grab, but without documentation of what the option
> is intended to do and in particular how it's intended to
> interact with grab/ungrab I don't know what your intention
> for the behaviour was.
Well, it doesn't change actual behavior for SDL and cocoa. It only adds
"-display {sdl,cocoa},show-cursor=on" as replacement for the global
"-show-cursor" option. Guess I should reorder the patches (move 5/6
before the individual UI patches) and reword the commit messages.
If you think cocoa behavior isn't useful we can revert commit
13aefd303cf996c2d183e94082413885bf1d15bf instead, or drop the
cursor_hide check in hideCursor() + unhideCursor(). Your call.
It also adds gtk support (based on a patch by address@hidden).
cheers,
Gerd
- [PATCH v2 0/6] ui: rework -show-cursor option, Gerd Hoffmann, 2020/02/06
- [PATCH v2 6/6] ui: deprecate legacy -show-cursor option, Gerd Hoffmann, 2020/02/06
- [PATCH v2 4/6] ui/cocoa: implement show-cursor option, Gerd Hoffmann, 2020/02/06
- [PATCH v2 1/6] ui: add show-cursor option, Gerd Hoffmann, 2020/02/06
- [PATCH v2 5/6] ui: wire up legacy -show-cursor option, Gerd Hoffmann, 2020/02/06
- [PATCH v2 2/6] ui/gtk: implement show-cursor option, Gerd Hoffmann, 2020/02/06
- [PATCH v2 3/6] ui/sdl: implement show-cursor option, Gerd Hoffmann, 2020/02/06
- Re: [PATCH v2 0/6] ui: rework -show-cursor option, Peter Maydell, 2020/02/06
- Re: [PATCH v2 0/6] ui: rework -show-cursor option,
Gerd Hoffmann <=