[meld/ui-next] meldwindow: Fix focus issues for headerbar buttons



commit 3ecf779dd8e15b4ce9a6c624be410672db51f2d0
Author: Kai Willadsen <kai willadsen gmail com>
Date:   Sat Mar 2 11:32:04 2019 +1000

    meldwindow: Fix focus issues for headerbar buttons
    
    It never really makes sense for these to be focused on click, so we're
    removing it from all headerbar buttons here. Our focus handling story
    for what pane an action affects is a little fragmented due for
    historical reasons, and this helps mitigate that.

 meld/resources/ui/appwindow.ui | 5 +++++
 1 file changed, 5 insertions(+)
---
diff --git a/meld/resources/ui/appwindow.ui b/meld/resources/ui/appwindow.ui
index c926ac08..801dc429 100644
--- a/meld/resources/ui/appwindow.ui
+++ b/meld/resources/ui/appwindow.ui
@@ -28,6 +28,7 @@
                 <property name="action-name">win.new-tab</property>
                 <property name="visible">true</property>
                 <property name="tooltip-text">Start a new comparison</property>
+                <property name="focus_on_click">False</property>
                 <child>
                   <object class="GtkImage">
                     <property name="icon-name">tab-new-symbolic</property>
@@ -71,6 +72,7 @@
                 <property name="visible">True</property>
                 <property name="action-name">view.previous-change</property>
                 <property name="tooltip-text">Go to the previous change</property>
+                <property name="focus_on_click">False</property>
                 <style>
                   <class name="image-button"/>
                 </style>
@@ -90,6 +92,7 @@
                 <property name="visible">True</property>
                 <property name="action-name">view.next-change</property>
                 <property name="tooltip-text">Go to the next change</property>
+                <property name="focus_on_click">False</property>
                 <style>
                   <class name="image-button"/>
                 </style>
@@ -119,6 +122,7 @@
                 <property name="visible">True</property>
                 <property name="action-name">view.file-previous-conflict</property>
                 <property name="tooltip-text">Go to the previous conflict</property>
+                <property name="focus_on_click">False</property>
                 <style>
                   <class name="image-button"/>
                 </style>
@@ -138,6 +142,7 @@
                 <property name="visible">True</property>
                 <property name="action-name">view.file-next-conflict</property>
                 <property name="tooltip-text">Go to the next conflict</property>
+                <property name="focus_on_click">False</property>
                 <style>
                   <class name="image-button"/>
                 </style>


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]