On Monday 08 of July 2019, Jan-Marek Glogowski wrote:
Hi
I'm trying to gather information how various apps handle the primary
selection, so I can fix tdf#104717 [1]. There is already a patch for that
in Gerrit[2].
...
IMHO we shouldn't handle the primary selection lifetime different from the
clipboard lifetime. When selecting stuff, the primary should stay alive,
even if I close the document or module, just like the normal clipboard. I
don't see a point of clearing the selection at all, except on application
shutdown, when the application won't be able to serve it anymore.
How do other applications you know handle the primary selection?
Variously. I don't think there's an official document on how the X11
selections should work from the UI point of view. At least I'm not aware of
that, and I'm saying that as somebody who used to practically maintain KDE's
Klipper for about a decade and a number of times sent patches for Qt's
QClipboard class.
I think the idea is sort of that the PRIMARY selection is _the_ selection in
the desktop. Just one in the entire desktop, which means that if you open two
documents and select text in both of them, there should be still just one
visual representation, the one that is the X11 selection (i.e. making another
selection removes the previous one, including the UI representation, you can
see this e.g. if you try it with 2 xterm's). That's however rather silly and
confusing from the usability point of view, as if the X11 clipboard handling
wasn't confusing enough as it is, so often apps do not reset the previous UI
representation or do other things slightly differently (and let's not forget
that there are many apps that are buggy). So from this strict point of view,
LO does everything properly. But from the practical point of view, I think
that's a bad way of doing it. IMO people generally don't want their text
editor UI selection to disappear just because they selected some text in some
other app or dialog.
As for tdf#104717, my understanding of what happens is:
- after the first Ctrl+V, Calc pastes the new cell content
- as a part of that, it also selects the cell
- since the cell is selected, Calc also sets PRIMARY
- Klipper detects a change in PRIMARY, since it's set to synchronize it with
CLIPBOARD, it sets CLIPBOARD to the contents of PRIMARY; Klipper does not
support the advanced types Calc uses for representing cells, this only keeps
it as plain text
- => next Ctrl+V pastes just the plain text
I see several ways of handling this:
1) Do not set PRIMARY for cell selections. I was originally going to say that
pasting those cells with MMB doesn't work anyway, but to my surprise it
actually does. Still, does somebody actually use that? And even if somebody
does that inside LO, will they use it to MMB the cell contents into another
app? I'm not aware of anything else using the mouse-select+MMB-paste
mechanism for anything else than text. Based on this assumption I made
#gerrit72369, and not setting PRIMARY at all would further remove
interferences with tools like Klipper.
2) Make Klipper handle all data types. Well, not really, I consider this
unfeasible, but for completeness. It'd fix the content loss during
synchronizing, but it'd lead to horrible performance problems of needless
copying data all over the place.
3) Remove Klipper synchronization. I myself don't use it, but some people
apparently do, although I can't say how popular it is. The point of the
option is to remove the confusion of two different clipboards, making them
appear as one. It makes sense to some people, my take is that most people
don't even know mouse selecting should do something with clipboards, so for
those it doesn't matter, and those that do this exit should also actually
learn how to use it. So it can be shrugged off as NOTOURBUG (and I expect
whoever maintains Klipper now will probably shrug it off too).
4) Your patch. If I understand it correctly, it still keeps makes the new
cells selected in the UI, it just avoids setting PRIMARY. I think that would
kind of work, but it seems hackish.
5) Don't select the pasted cells, not even in the UI. If I Ctrl+V in a text
editor, the pasted text doesn't become selected either. This should
(hopefully?) be simple, consistent and avoid the problem too.
I think the best one is 5), closely followed by 1).
--
Luboš Luňák
l.lunak@collabora.com
Context
Privacy Policy |
Impressum (Legal Info) |
Copyright information: Unless otherwise specified, all text and images
on this website are licensed under the
Creative Commons Attribution-Share Alike 3.0 License.
This does not include the source code of LibreOffice, which is
licensed under the Mozilla Public License (
MPLv2).
"LibreOffice" and "The Document Foundation" are
registered trademarks of their corresponding registered owners or are
in actual use as trademarks in one or more countries. Their respective
logos and icons are also subject to international copyright laws. Use
thereof is explained in our
trademark policy.