From 0dd120ab4ecd6598a832f74da92805712f5ea35c Mon Sep 17 00:00:00 2001
From: Michael Jeronimo <michael.jeronimo@openrobotics.org>
Date: Fri, 9 Oct 2020 09:01:21 -0700
Subject: [PATCH] Update a few minor status bar-related items
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Add tooltips for ‘playback speed’ and ‘bag size’ in the status bar
* Update existing status bar tooltip strings for clarity and consistency
* Show the full timestamp in the status bar
* Adjust some status bar field widths to avoid clipping

Signed-off-by: Michael Jeronimo <michael.jeronimo@openrobotics.org>
---
 rqt_bag/resource/bag_widget.ui    | 18 ++++++++++++------
 rqt_bag/src/rqt_bag/bag_widget.py |  2 +-
 2 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/rqt_bag/resource/bag_widget.ui b/rqt_bag/resource/bag_widget.ui
index 9ab11cd..385154a 100644
--- a/rqt_bag/resource/bag_widget.ui
+++ b/rqt_bag/resource/bag_widget.ui
@@ -324,12 +324,12 @@
       <widget class="QLabel" name="stamp_label">
        <property name="maximumSize">
         <size>
-         <width>125</width>
+         <width>185</width>
          <height>16777215</height>
         </size>
        </property>
        <property name="toolTip">
-        <string>Time Stamp of the current playhead</string>
+        <string>Timestamp of the current playhead location</string>
        </property>
        <property name="frameShape">
         <enum>QFrame::Panel</enum>
@@ -349,12 +349,12 @@
       <widget class="QLabel" name="date_label">
        <property name="maximumSize">
         <size>
-         <width>180</width>
+         <width>185</width>
          <height>16777215</height>
         </size>
        </property>
        <property name="toolTip">
-        <string>Date and time of the playhead</string>
+        <string>Date and time of the current playhead location</string>
        </property>
        <property name="frameShape">
         <enum>QFrame::Panel</enum>
@@ -380,12 +380,12 @@
        </property>
        <property name="maximumSize">
         <size>
-         <width>110</width>
+         <width>90</width>
          <height>16777215</height>
         </size>
        </property>
        <property name="toolTip">
-        <string>Number of Seconds from the beginning of the current playhead</string>
+        <string>Number of seconds the current playhead location is from the beginning of the bag</string>
        </property>
        <property name="frameShape">
         <enum>QFrame::Panel</enum>
@@ -403,6 +403,9 @@
      </item>
      <item>
       <widget class="QLabel" name="playspeed_label">
+       <property name="toolTip">
+        <string>Playback speed</string>
+       </property>
        <property name="maximumSize">
         <size>
          <width>80</width>
@@ -425,6 +428,9 @@
      </item>
      <item>
       <widget class="QLabel" name="filesize_label">
+       <property name="toolTip">
+        <string>Bag size</string>
+       </property>
        <property name="maximumSize">
         <size>
          <width>80</width>
diff --git a/rqt_bag/src/rqt_bag/bag_widget.py b/rqt_bag/src/rqt_bag/bag_widget.py
index bee3bdb..4cf1f6e 100644
--- a/rqt_bag/src/rqt_bag/bag_widget.py
+++ b/rqt_bag/src/rqt_bag/bag_widget.py
@@ -339,7 +339,7 @@ def _update_status_bar(self):
             self.progress_bar.setValue(self._timeline.background_progress)
 
             # Raw timestamp
-            self.stamp_label.setText('%.3fs' % self._timeline._timeline_frame.playhead.to_sec())
+            self.stamp_label.setText('%.9fs' % self._timeline._timeline_frame.playhead.to_sec())
 
             # Human-readable time
             self.date_label.setText(