lazzexus/Loupe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

image_page: Use AdwSpinner

Browse source 
Depends on
https://gitlab.gnome.org/GNOME/libadwaita/-/merge_requests/1201/

AdwSpinner automatically handles pausing the animation
when unmapped.

Part-of: <https://gitlab.gnome.org/GNOME/loupe/-/merge_requests/420>
This commit is contained in:
Christopher Davis 2024-07-06 13:52:59 -04:00 • committed by Sophie Herold
parent 4a1a4d270c
commit cd68778890
2 changed files with 2 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/widgets/image_page.rs
View file

@ -51,7 +51,7 @@ mod imp {
#[template_child]
pub(super) image_stack_page: TemplateChild<gtk::Widget>,
#[template_child]
pub(super) spinner: TemplateChild<gtk::Spinner>,
pub(super) spinner: TemplateChild<gtk::Widget>,
#[template_child]
pub(super) scrolled_window: TemplateChild<gtk::ScrolledWindow>,
#[template_child]
@ -145,10 +145,6 @@ mod imp {
glib::spawn_future_local(async move { obj.show_error().await });
}
));
// Do not waste CPU on spinner if it is not visible
self.spinner.connect_map(|s| s.start());
self.spinner.connect_unmap(|s| s.stop());
}
}

8
src/widgets/image_page.ui
View file

@ -47,13 +47,7 @@
<property name="hexpand">true</property>
<property name="vexpand">true</property>
<child>
<object class="GtkSpinner" id="spinner">
<property name="height-request">24</property>
<property name="width-request">24</property>
<property name="halign">center</property>
<property name="valign">center</property>
<property name="spinning">True</property>
</object>
<object class="AdwSpinner" id="spinner"/>
</child>
</object>
</child>