37057 - X selection is not exported to the X cut buffer

Status: RESOLVED INVALID

(Core :: DOM: Selection, defect, P5)

Description

[Chris Siebenmann]

Mozilla (recent nightly builds/CVS pull) automatically exposes the Mozilla
selection as the X selection (so if you highlight some text in Mozilla, you
can immediately paste it elsewhere) but doesn't export this current selection
into the (older) X cut buffer.

 Possibly there are internationalization issues that preclude exporting
things to the X cut buffer (I believe it only deals in ASCII, although
it will likely handle UTF-8 Unicode).

Comment 1

[rubydoo123]

assiging to akkana for review

Comment 2

[rubydoo123]

honest -- assigning it to akkana

Comment 3

[Stuart Parmenter]

we don't currently put data in to SECONDARY, but it would be pretty easy to
add... I think I have a bug on this, but I can't find it.  I don't really plan
on supporting CUT_BUFFER[0-7] unless there is real a real demand for it, and
even then I won't get a chance to get to it any time soon.

Comment 4

[Akkana Peck]

If you already have a bug on this, I'll send this one your way too, and let you
dup it.  Might be a good candidate for HELPWANTED ... in fact, I'll add that
now.

Comment 5

[Peter Trudelle]

Mass-moving all M20-M30 XPToolkit bugs to Future

Comment 6

[Andrew Taylor]

I've created a patch that exports the contents of the X selection to the X cut
buffer.  Right now it updates from the global clipboard (on edit/copy menu
item), not the selection clipboard (on text highlight), I'm not sure which
behavior makes more sense.

I'm not sure this is really desirable behavior (X cut buffers were obsoleted for
a reason :) so it is #ifdefed out by default.  Perhaps it can be changed to a
preference setting in the future.

Comment 7

[Andrew Taylor]

Attachment: Exports selection to X cut buffer

Comment 8

[Akkana Peck]

Adding keyword patch.  Might be worth adding the code even if it's ifdefed out
by default, in case we decide we want it.

Comment 9

[Blake Ross]

*SPAM*: Changing the QA contact of all open/resolved Selection bugs from 
[e-mail] to [e-mail].  After the many great years of service 
Eli has given to Mozilla, it's time for him to move on; he has accepted a 
position at Eazel.  We'll be sad to see him go, and I'll do my best to fill his 
spot...

Comment 10

[David A. Madore]

I wrote a discussion on the subject of this bug in the article
news://news.mozilla.org/8o9928$[e-mail]

There I explain why it is a good idea to export the selected text to the (0)
cutbuffer even though cutbuffers are supposedly "obsolete".

Comment 11

[Boris Zbarsky [:bzbarsky]]

The other reason to export the selection to cut buffer 0 is that that allows the
following sequence of actions to actually do something useful:

1) select text in mozilla
2) quit mozilla
3) attempt to paste the text you had selected into some other application

Currently, that has the weird result of pasting something that was selected in a
different app before he selection was made in Mozilla.

Comment 12

[Keyser Sose]

Adding Review keyword..this has been sitting around for a while now.

Comment 13

[Mike Shaver (:shaver emeritus)]

I like the idea of having our X selection behaviour be closer to xterm's (and
4.x's?) behaviour.  I'll take a look at the patch soon.

Comment 14

[Keyser Sose]

*** Bug 75224 has been marked as a duplicate of this bug. ***

Comment 15

[Christopher Blizzard (:blizzard)]

+  // Make sure we have a transferable:
+  if (!transferable) {
+    g_print("Clipboard has no transferable!\n");
+    return;
+  }

Please remove that or at least make it an NS_WARNING()

Fix that and you have an sr=blizzard.

Comment 16

[Asa Dotzler [:asa]]

a= [e-mail] for checkin to the trunk.
(on behalf of drivers)

Comment 17

[Stuart Parmenter]

Checked in patch r=pavlov, sr=blizzard, a=asa.

So i'm not sure what to do with this bug.. i've checked in the patch, but since 
the ifdef is off, we still don't export to the X cut buffer.  Should we leave 
this open for discussion about turning it on or should we file a new bug?

Comment 18

[Jason Kersey]

-> 0.9.4 per Pavlov

Comment 19

[Stuart Parmenter]

.

Comment 20

[Blake Ross]

changing selection qa to tpreston.

Comment 21

[Mike Fedyk]

I haven't tested this yet (too many tabs open), but it would be nice to be able
to copy by select, close mozilla, and then be able to paste it into another app.

Comment 22

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, >=3 years untouched DOM/Storage bug's priority.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.

Comment 23

[Sean Feng [:sefeng]]

I don't think this bug is still valid. We are using the Primary Selection buffer on Linux, which is the modern replacement of X cut buffer, which allows copy pasting between different applications.

So I am going to close this bug as INVALID.