[gnome-initial-setup/wjt/mark-accept-as-sensitive] assistant: mark "Accept" button as suggested action



commit afb42b26c77820ae8338ac14e127bcab2cbf6aa2
Author: Will Thompson <wjt endlessm com>
Date:   Wed Sep 25 19:15:24 2019 +0100

    assistant: mark "Accept" button as suggested action
    
    3857805 added this style class to the "Next" button, and removed code
    that would set this class depending on whether the widget is sensitive.
    At the time I overlooked that the suggested-action style class should
    also be present for the "Accept" button on the EULA page: I think it's
    correct to say that accepting the EULA is the suggested action.
    
    There is a third button which may appear in this position: "Skip". This
    is used (for example) on the Wi-Fi configuration page when no network is
    selected. It's correct not to mark this button as suggested: the
    suggested action is to configure a Wi-Fi network.

 gnome-initial-setup/gis-assistant.ui | 3 +++
 1 file changed, 3 insertions(+)
---
diff --git a/gnome-initial-setup/gis-assistant.ui b/gnome-initial-setup/gis-assistant.ui
index eac5768..936f829 100644
--- a/gnome-initial-setup/gis-assistant.ui
+++ b/gnome-initial-setup/gis-assistant.ui
@@ -60,6 +60,9 @@
       <object class="GtkButton" id="accept">
         <property name="use-underline">True</property>
         <property name="can-default">True</property>
+        <style>
+          <class name="suggested-action"/>
+        </style>
       </object>
       <packing>
         <property name="pack-type">end</property>


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