Add tooltips to playback controls (#352)

* Add tooltips to playback controls * Use "gettext" for playback control tooltip
xou816 · Oct 27, 2021 · 200498b · 200498b
1 parent 3d81a70
commit 200498b
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 3 deletions.
diff --git a/src/app/components/playback/playback_controls.rs b/src/app/components/playback/playback_controls.rs
@@ -1,3 +1,4 @@
+use gettextrs::gettext;
 use gtk::prelude::*;
 use gtk::subclass::prelude::*;
 use gtk::{glib, CompositeTemplate};
@@ -59,9 +60,17 @@ impl PlaybackControlsWidget {
             "media-playback-start-symbolic"
         };
 
-        imp::PlaybackControlsWidget::from_instance(self)
-            .play_pause
-            .set_icon_name(playback_icon);
+        let translated_tooltip = if is_playing {
+            gettext("Pause")
+        } else {
+            gettext("Play")
+        };
+        let tooltip_text = Some(translated_tooltip.as_str());
+
+        let playback_control = imp::PlaybackControlsWidget::from_instance(self);
+
+        playback_control.play_pause.set_icon_name(playback_icon);
+        playback_control.play_pause.set_tooltip_text(tooltip_text);
     }
 
     pub fn set_shuffled(&self, shuffled: bool) {

diff --git a/src/app/components/playback/playback_controls.ui b/src/app/components/playback/playback_controls.ui
@@ -14,6 +14,7 @@
         <property name="valign">center</property>
         <property name="has-frame">0</property>
         <property name="icon-name">media-playlist-shuffle-symbolic</property>
+        <property name="tooltip-text" translatable="yes">Shuffle</property>
       </object>
     </child>
     <child>
@@ -23,6 +24,7 @@
         <property name="valign">center</property>
         <property name="has-frame">0</property>
         <property name="icon-name">media-skip-backward-symbolic</property>
+        <property name="tooltip-text" translatable="yes">Previous</property>
       </object>
     </child>
     <child>
@@ -31,6 +33,7 @@
         <property name="halign">center</property>
         <property name="valign">center</property>
         <property name="icon-name">media-playback-start-symbolic</property>
+        <property name="tooltip-text" translatable="yes">Play/Pause</property>
         <style>
           <class name="circular"/>
           <class name="playback-button"/>
@@ -44,6 +47,7 @@
         <property name="valign">center</property>
         <property name="has-frame">0</property>
         <property name="icon-name">media-skip-forward-symbolic</property>
+        <property name="tooltip-text" translatable="yes">Next</property>
       </object>
     </child>
     <child>
@@ -53,6 +57,7 @@
         <property name="valign">center</property>
         <property name="has-frame">0</property>
         <property name="icon-name">media-playlist-consecutive-symbolic</property>
+        <property name="tooltip-text" translatable="yes">Repeat</property>
       </object>
     </child>
   </template>