Merge pull request #373 from freedomofpress/refresh-polish

polish refresh
freedomofpress · May 20, 2019 · b8baa47 · b8baa47
2 parents 684b5c7 + 205531c
commit b8baa47
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 22 deletions.
diff --git a/securedrop_client/gui/main.py b/securedrop_client/gui/main.py
@@ -145,7 +145,7 @@ def show_sync(self, updated_on):
         Display a message indicating the data-sync state.
         """
         if updated_on:
-            self.update_activity_status(_('Last refresh: {}').format(updated_on.humanize()))
+            self.update_activity_status(_('Last Refresh: {}').format(updated_on.humanize()))
         else:
             self.update_activity_status(_('Waiting to refresh...'), 5000)
 

diff --git a/securedrop_client/gui/widgets.py b/securedrop_client/gui/widgets.py
@@ -62,7 +62,7 @@ def __init__(self):
         self.setLayout(layout)
 
         # Remove margins and spacing
-        layout.setContentsMargins(0, 0, 0, 0)
+        layout.setContentsMargins(10, 0, 0, 0)
         layout.setSpacing(0)
 
         # Refresh button
@@ -189,14 +189,14 @@ def __init__(self):
             disabled='refresh_offline.svg',
             active='refresh_active.svg',
             selected='refresh.svg',
-            svg_size=QSize(22, 22))
+            svg_size=QSize(18, 18))
 
         # Set css id
         self.setObjectName('refresh_button')
 
         # Set styles
         self.setStyleSheet(self.CSS)
-        self.setFixedSize(QSize(42, 42))
+        self.setFixedSize(QSize(22, 22))
 
         # Click event handler
         self.clicked.connect(self._on_clicked)

diff --git a/securedrop_client/resources/images/refresh.svg b/securedrop_client/resources/images/refresh.svg
diff --git a/tests/gui/test_main.py b/tests/gui/test_main.py
@@ -214,7 +214,7 @@ def test_show_sync(mocker):
     updated_on = mocker.MagicMock()
     w.show_sync(updated_on)
     w.update_activity_status.assert_called_once_with(
-        'Last refresh: {}'.format(updated_on.humanize()))
+        'Last Refresh: {}'.format(updated_on.humanize()))
 
 
 def test_show_sync_no_sync(mocker):