fix:[#875] move output behind button

Author: jardon

src/view/action_page.rs

@@ -14,6 +14,7 @@ use crate::utils;
 use crate::view;
 
 const ACTION_CANCEL: &str = "action-page.cancel";
+const ACTION_VIEW_OUTPUT: &str = "action-page.view-output";
 const ACTION_VIEW_ARTIFACT: &str = "action-page.view-artifact";
 const ACTION_RETRY: &str = "action-page.retry";
 
@@ -30,6 +31,8 @@ mod imp {
         pub(super) show_view_artifact: Cell<bool>,
         #[template_child]
         pub(super) status_page: TemplateChild<adw::StatusPage>,
+        #[template_child]
+        pub(super) output_box: TemplateChild<adw::Clamp>,
     }
 
     #[glib::object_subclass]
@@ -41,6 +44,9 @@ mod imp {
         fn class_init(klass: &mut Self::Class) {
             klass.bind_template();
             klass.install_action(ACTION_CANCEL, None, |widget, _, _| widget.cancel());
+            klass.install_action(ACTION_VIEW_OUTPUT, None, |widget, _, _| {
+                widget.view_output()
+            });
             klass.install_action(ACTION_VIEW_ARTIFACT, None, |widget, _, _| {
                 widget.view_artifact();
             });
@@ -146,6 +152,7 @@ impl ActionPage {
         use model::ActionType::*;
 
         let imp = self.imp();
+        imp.output_box.set_visible(false);
 
         match action.state() {
             model::ActionState::Ongoing => {
@@ -225,6 +232,7 @@ impl ActionPage {
         self.set_description(action);
 
         self.action_set_enabled(ACTION_CANCEL, action.state() == Ongoing);
+        self.action_set_enabled(ACTION_VIEW_OUTPUT, action.state() == Finished);
         self.action_set_enabled(
             ACTION_VIEW_ARTIFACT,
             self.show_view_artifact()
@@ -277,6 +285,14 @@ impl ActionPage {
         }
     }
 
+    fn view_output(&self) {
+        let imp = self.imp();
+        imp.status_page.set_icon_name(None);
+        imp.status_page.set_description(None);
+        imp.output_box.set_visible(true);
+        self.action_set_enabled(ACTION_VIEW_OUTPUT, false);
+    }
+
     fn view_artifact(&self) {
         match self.action().as_ref().and_then(model::Action::artifact) {
             Some(artifact) => {

src/view/action_page.ui

@@ -5,6 +5,7 @@
     <property name="mode">horizontal</property>
     <widgets>
       <widget name="abort_button"/>
+      <widget name="view_output_button"/>
       <widget name="view_artifact_button"/>
     </widgets>
   </object>
@@ -40,10 +41,12 @@
                 <child>
                   <object class="AdwClamp">
                     <property name="margin-bottom">9</property>
+                    <property name="orientation">vertical</property>
 
                     <child>
-                      <object class="AdwClamp">
+                      <object class="AdwClamp" id="output_box">
                         <property name="orientation">vertical</property>
+                        <property name="height-request">220</property>
 
                         <property name="child">
                           <object class="AdwBin">
@@ -107,6 +110,20 @@
                   </object>
                 </child>
 
+                <child>
+                  <object class="GtkButton" id="view_output_button">
+                    <style>
+                      <class name="pill"/>
+                    </style>
+                    <property name="action-name">action-page.view-output</property>
+                    <property name="halign">center</property>
+                    <property name="label" translatable="yes">_View Output</property>
+                    <property name="use-underline">True</property>
+                    <property name="visible" bind-source="view_output_button" bind-property="sensitive" bind-flags="sync-create"/>
+                    <property name="width-request">200</property>
+                  </object>
+                </child>
+
                 <child>
                   <object class="GtkButton" id="view_artifact_button">
                     <style>