Merge ~gusnan/sakura:fix_primary_clipboard into sakura:master

Proposed by Andreas Rönnquist
Status: Merged
Approved by: David Gómez
Approved revision: e4e8b211ed26ff06a064f96bed8670fb1c43aac5
Merged at revision: e4e8b211ed26ff06a064f96bed8670fb1c43aac5
Proposed branch: ~gusnan/sakura:fix_primary_clipboard
Merge into: sakura:master
Diff against target: 29 lines (+0/-11)
1 file modified
src/sakura.c (+0/-11)
Reviewer Review Type Date Requested Status
David Gómez Pending
Review via email: mp+411052@code.launchpad.net

Description of the change

This should fix

LP: #1948479
https://bugs.launchpad.net/sakura/+bug/1948479

(see the test in that bug).

However, this undos some earlier commits:

I am not the author of the patch - I am just forwarding
(I have modified the git commit author).

Here what he says:

----------------------------

The bug is introduced in the commit 3f11e123;
the title says it all: "Make copy on select to clipboard the default
and only behavior".

Commit d8a6e2c0 "add config support for button behavior"
introduces the option 'copy_on_select' that "automatically copy
selected text", this option defaults to 'false'.

Commit 4981d2c6 "Solve problem with duplicates when pasting
with middle button" changes the default to 'true' (!)
and unifies PRIMARY and CLIPBOARD selections.

Commit 3f11e123 "Make copy on select to clipboard
the default and only behavior" removes the option
'copy_on_select' but leaves everything else as is.

So my patch sort of reverse 3f11e123; I have never
experienced the bug solved by 4981d2c6 and I do not
see it now, so I do not know if the modifications of
this 3 patches are still needed.

----------------------------

And also to me this patch fixes the problem.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/sakura.c b/src/sakura.c
2index 553b728..d960b3f 100644
3--- a/src/sakura.c
4+++ b/src/sakura.c
5@@ -891,9 +891,6 @@ sakura_term_buttonreleased_cb (GtkWidget *widget, GdkEventButton *button_event,
6 if (button_event->type != GDK_BUTTON_RELEASE)
7 return FALSE;
8
9- if (button_event->button == 1)
10- sakura_copy();
11-
12 return FALSE;
13 }
14
15@@ -923,14 +920,6 @@ sakura_term_buttonpressed_cb (GtkWidget *widget, GdkEventButton *button_event, g
16 return TRUE;
17 }
18
19- /* Paste when paste button is pressed */
20- if (button_event->button == sakura.paste_button) {
21- sakura_paste();
22-
23- /* Do not propagate. vte has his own copy-on-select and we'll end with duplicates pastes */
24- return TRUE;
25- }
26-
27 /* Show the popup menu when menu button is pressed */
28 if (button_event->button == sakura.menu_button) {
29 GtkMenu *menu;

Subscribers

People subscribed via source and target branches