[gnome-games/wip/exalm/unified-window: 60/60] preferences-subpage-gamepad: Simplify layout
- From: Alexander Mikhaylenko <alexm src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-games/wip/exalm/unified-window: 60/60] preferences-subpage-gamepad: Simplify layout
- Date: Fri, 10 Apr 2020 14:43:34 +0000 (UTC)
commit f5bf47c074c76b2e408d8d94bce813b7e6529e90
Author: Alexander Mikhaylenko <alexm gnome org>
Date: Fri Apr 10 19:40:43 2020 +0500
preferences-subpage-gamepad: Simplify layout
data/ui/gamepad-mapper.ui | 1 +
data/ui/gamepad-tester.ui | 1 +
data/ui/preferences-subpage-gamepad.ui | 174 +++++++++++++++++---------------
src/ui/preferences-subpage-gamepad.vala | 46 ++++-----
4 files changed, 115 insertions(+), 107 deletions(-)
---
diff --git a/data/ui/gamepad-mapper.ui b/data/ui/gamepad-mapper.ui
index 5fd11b08..28c298ce 100644
--- a/data/ui/gamepad-mapper.ui
+++ b/data/ui/gamepad-mapper.ui
@@ -3,6 +3,7 @@
<requires lib="gtk+" version="3.24"/>
<template class="GamesGamepadMapper" parent="GtkBin">
<property name="visible">True</property>
+ <property name="vexpand">True</property>
<child>
<object class="GamesGamepadView" id="gamepad_view">
<property name="visible">True</property>
diff --git a/data/ui/gamepad-tester.ui b/data/ui/gamepad-tester.ui
index 3eb96981..c1fd209e 100644
--- a/data/ui/gamepad-tester.ui
+++ b/data/ui/gamepad-tester.ui
@@ -3,6 +3,7 @@
<requires lib="gtk+" version="3.24"/>
<template class="GamesGamepadTester" parent="GtkBin">
<property name="visible">True</property>
+ <property name="vexpand">True</property>
<child>
<object class="GamesGamepadView" id="gamepad_view">
<property name="visible">True</property>
diff --git a/data/ui/preferences-subpage-gamepad.ui b/data/ui/preferences-subpage-gamepad.ui
index ebe4d1de..87ca8889 100644
--- a/data/ui/preferences-subpage-gamepad.ui
+++ b/data/ui/preferences-subpage-gamepad.ui
@@ -1,109 +1,125 @@
<?xml version="1.0" encoding="UTF-8"?>
<interface>
<requires lib="gtk+" version="3.24"/>
- <template class="GamesPreferencesSubpageGamepad" parent="GtkBox">
+ <template class="GamesPreferencesSubpageGamepad" parent="GtkBin">
<property name="visible">True</property>
- <property name="orientation">vertical</property>
<child>
- <object class="HdyHeaderBar" id="header_bar">
+ <object class="GtkStack" id="stack">
<property name="visible">True</property>
- <style>
- <class name="titlebar"/>
- </style>
+ <property name="transition-type">crossfade</property>
<child>
- <object class="GtkButton" id="back_button">
+ <object class="GtkBox" id="tester_box">
<property name="visible">True</property>
- <signal name="clicked" handler="on_back_clicked"/>
- <style>
- <class name="image-button"/>
- </style>
- <child internal-child="accessible">
- <object class="AtkObject" id="a11y-back">
- <property name="accessible-name" translatable="yes">Back</property>
- </object>
- </child>
+ <property name="orientation">vertical</property>
<child>
- <object class="GtkImage" id="back_image">
+ <object class="HdyHeaderBar" id="tester_header_bar">
<property name="visible">True</property>
- <property name="icon-name">go-previous-symbolic</property>
- <property name="icon-size">1</property>
- </object>
- </child>
- </object>
- <packing>
- <property name="pack-type">start</property>
- </packing>
- </child>
- <child>
- <object class="GtkButton" id="cancel_button">
- <property name="visible">True</property>
- <property name="label" translatable="yes">Cancel</property>
- <signal name="clicked" handler="on_cancel_clicked"/>
- </object>
- <packing>
- <property name="pack-type">end</property>
- </packing>
- </child>
- </object>
- </child>
- <child>
- <object class="GtkStack" id="gamepad_view_stack">
- <property name="visible">True</property>
- <property name="vexpand">True</property>
- </object>
- </child>
- <child>
- <object class="GtkStack" id="action_bar_stack">
- <property name="visible">True</property>
- <child>
- <object class="GtkActionBar" id="tester_action_bar">
- <property name="visible">True</property>
- <child>
- <object class="GtkButton" id="reset_button">
- <property name="visible">True</property>
- <property name="label" translatable="yes" context="Keyboard configuration factory
reset">Reset</property>
- <signal name="clicked" handler="on_reset_clicked"/>
+ <property name="show-close-button">True</property>
<style>
- <class name="destructive-action"/>
+ <class name="titlebar"/>
</style>
+ <child>
+ <object class="GtkButton">
+ <property name="visible">True</property>
+ <signal name="clicked" handler="on_back_clicked"/>
+ <style>
+ <class name="image-button"/>
+ </style>
+ <child internal-child="accessible">
+ <object class="AtkObject" id="a11y-back">
+ <property name="accessible-name" translatable="yes">Back</property>
+ </object>
+ </child>
+ <child>
+ <object class="GtkImage">
+ <property name="visible">True</property>
+ <property name="icon-name">go-previous-symbolic</property>
+ <property name="icon-size">1</property>
+ </object>
+ </child>
+ </object>
+ <packing>
+ <property name="pack-type">start</property>
+ </packing>
+ </child>
</object>
- <packing>
- <property name="pack-type">end</property>
- </packing>
</child>
<child>
- <object class="GtkButton" id="configure_button">
+ <object class="GtkActionBar">
<property name="visible">True</property>
- <property name="label" translatable="yes">Configure</property>
- <signal name="clicked" handler="on_configure_clicked"/>
- <style>
- <class name="suggested-action"/>
- </style>
+ <child>
+ <object class="GtkButton" id="reset_button">
+ <property name="visible">True</property>
+ <property name="label" translatable="yes" context="Keyboard configuration factory
reset">Reset</property>
+ <signal name="clicked" handler="on_reset_clicked"/>
+ <style>
+ <class name="destructive-action"/>
+ </style>
+ </object>
+ <packing>
+ <property name="pack-type">end</property>
+ </packing>
+ </child>
+ <child>
+ <object class="GtkButton">
+ <property name="visible">True</property>
+ <property name="label" translatable="yes">Configure</property>
+ <signal name="clicked" handler="on_configure_clicked"/>
+ <style>
+ <class name="suggested-action"/>
+ </style>
+ </object>
+ <packing>
+ <property name="pack-type">start</property>
+ </packing>
+ </child>
</object>
- <packing>
- <property name="pack-type">start</property>
- </packing>
</child>
</object>
</child>
<child>
- <object class="GtkActionBar" id="mapper_action_bar">
+ <object class="GtkBox" id="mapper_box">
<property name="visible">True</property>
+ <property name="orientation">vertical</property>
<child>
- <object class="GtkButton" id="skip_button">
+ <object class="HdyHeaderBar" id="mapper_header_bar">
<property name="visible">True</property>
- <property name="label" translatable="yes">Skip</property>
- <signal name="clicked" handler="on_skip_clicked"/>
+ <style>
+ <class name="titlebar"/>
+ <class name="selection-mode"/>
+ </style>
+ <child>
+ <object class="GtkButton">
+ <property name="visible">True</property>
+ <property name="label" translatable="yes">Cancel</property>
+ <signal name="clicked" handler="on_cancel_clicked"/>
+ </object>
+ <packing>
+ <property name="pack-type">end</property>
+ </packing>
+ </child>
</object>
- <packing>
- <property name="pack-type">start</property>
- </packing>
</child>
- <child type="center">
- <object class="GtkLabel">
+ <child>
+ <object class="GtkActionBar">
<property name="visible">True</property>
- <property name="ellipsize">end</property>
- <property name="label" bind-source="GamesPreferencesSubpageGamepad"
bind-property="info-message" bind-flags="bidirectional"/>
+ <child>
+ <object class="GtkButton">
+ <property name="visible">True</property>
+ <property name="label" translatable="yes">Skip</property>
+ <signal name="clicked" handler="on_skip_clicked"/>
+ </object>
+ <packing>
+ <property name="pack-type">start</property>
+ </packing>
+ </child>
+ <child type="center">
+ <object class="GtkLabel">
+ <property name="visible">True</property>
+ <property name="ellipsize">end</property>
+ <property name="label" bind-source="GamesPreferencesSubpageGamepad"
bind-property="info-message" bind-flags="bidirectional"/>
+ </object>
+ </child>
</object>
</child>
</object>
diff --git a/src/ui/preferences-subpage-gamepad.vala b/src/ui/preferences-subpage-gamepad.vala
index c31a71b6..b6df01f4 100644
--- a/src/ui/preferences-subpage-gamepad.vala
+++ b/src/ui/preferences-subpage-gamepad.vala
@@ -1,7 +1,7 @@
// This file is part of GNOME Games. License: GPL-3.0+.
[GtkTemplate (ui = "/org/gnome/Games/ui/preferences-subpage-gamepad.ui")]
-private class Games.PreferencesSubpageGamepad : Gtk.Box, PreferencesSubpage {
+private class Games.PreferencesSubpageGamepad : Gtk.Bin, PreferencesSubpage {
private const GamepadInput[] STANDARD_GAMEPAD_INPUTS = {
{ EventCode.EV_KEY, EventCode.BTN_EAST },
{ EventCode.EV_KEY, EventCode.BTN_SOUTH },
@@ -37,24 +37,13 @@ private class Games.PreferencesSubpageGamepad : Gtk.Box, PreferencesSubpage {
private State state {
set {
_state = value;
- back_button.visible = (state == State.TEST);
- cancel_button.visible = (state == State.CONFIGURE);
- header_bar.show_close_button = (state == State.TEST);
allow_back = (state == State.TEST);
- if (state == State.CONFIGURE)
- header_bar.get_style_context ().add_class ("selection-mode");
- else
- header_bar.get_style_context ().remove_class ("selection-mode");
-
switch (value) {
case State.TEST:
reset_button.set_sensitive (device.has_user_mapping ());
- /* translators: testing a gamepad, %s is its name */
- header_bar.title = _("Testing %s").printf (device.get_name ());
- gamepad_view_stack.visible_child = tester;
- action_bar_stack.visible_child = tester_action_bar;
+ stack.visible_child = tester_box;
tester.start ();
mapper.stop ();
@@ -62,10 +51,7 @@ private class Games.PreferencesSubpageGamepad : Gtk.Box, PreferencesSubpage {
break;
case State.CONFIGURE:
- /* translators: configuring a gamepad, %s is its name */
- header_bar.title = _("Configuring %s").printf (device.get_name ());
- gamepad_view_stack.visible_child = mapper;
- action_bar_stack.visible_child = mapper_action_bar;
+ stack.visible_child = mapper_box;
tester.stop ();
mapper.start ();
@@ -81,21 +67,17 @@ private class Games.PreferencesSubpageGamepad : Gtk.Box, PreferencesSubpage {
public string info_message { get; set; }
[GtkChild]
- private Hdy.HeaderBar header_bar;
+ private Gtk.Stack stack;
[GtkChild]
- private Gtk.Stack gamepad_view_stack;
+ private Gtk.Box tester_box;
[GtkChild]
- private Gtk.Stack action_bar_stack;
+ private Gtk.Box mapper_box;
[GtkChild]
- private Gtk.ActionBar tester_action_bar;
+ private Hdy.HeaderBar tester_header_bar;
[GtkChild]
- private Gtk.ActionBar mapper_action_bar;
+ private Hdy.HeaderBar mapper_header_bar;
[GtkChild]
private Gtk.Button reset_button;
- [GtkChild]
- private Gtk.Button back_button;
- [GtkChild]
- private Gtk.Button cancel_button;
private GamepadMapper mapper;
private GamepadTester tester;
@@ -106,11 +88,19 @@ private class Games.PreferencesSubpageGamepad : Gtk.Box, PreferencesSubpage {
construct {
_device = value;
mapper = new GamepadMapper (value, GamepadViewConfiguration.get_default (),
STANDARD_GAMEPAD_INPUTS);
- gamepad_view_stack.add (mapper);
tester = new GamepadTester (value, GamepadViewConfiguration.get_default ());
- gamepad_view_stack.add (tester);
+
+ tester_box.add (tester);
+ tester_box.reorder_child (tester, 1);
+ mapper_box.add (mapper);
+ mapper_box.reorder_child (mapper, 1);
mapper.bind_property ("info-message", this, "info-message", BindingFlags.SYNC_CREATE);
+
+ /* translators: testing a gamepad, %s is its name */
+ tester_header_bar.title = _("Testing %s").printf (device.get_name ());
+ /* translators: configuring a gamepad, %s is its name */
+ mapper_header_bar.title = _("Configuring %s").printf (device.get_name ());
}
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
