diff options
| author | Simon Schampijer <simon@laptop.org> | 2013-02-15 12:53:05 (GMT) |
|---|---|---|
| committer | Simon Schampijer <simon@laptop.org> | 2013-02-15 14:39:11 (GMT) |
| commit | 7b118678414a7221fe844031bbdd5941b2094d0f (patch) | |
| tree | 89b6ff7bad5efdf1d00923ada08ed1c3beeab16a | |
| parent | f6ba32cef3bbbc66fbf85e069fbb08d6c311fca6 (diff) | |
Journal sorting button: hide/show on click or touch
Using the 'toggle_palette' property of the Palette invoker
we get the behaviour for hiding/showing the Palette when
clicking or touching the button. We need to set the
'hide_tooltip_on_click' property of the ToolButton to False
in that case. This is the same behaviour as the Frame icons,
see the WirelessDeviceView for a similar case of a ToolButton
with that behaviour.
Signed-off-by: Simon Schampijer <simon@laptop.org>
Acked-by: Manuel Quiñones <manuq@laptop.org>
| -rw-r--r-- | src/jarabe/journal/journaltoolbox.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/src/jarabe/journal/journaltoolbox.py b/src/jarabe/journal/journaltoolbox.py index f794dd7..c7ee73a 100644 --- a/src/jarabe/journal/journaltoolbox.py +++ b/src/jarabe/journal/journaltoolbox.py @@ -107,8 +107,6 @@ class MainToolbox(ToolbarBox): tool_item.show() self._sorting_button = SortingButton() - self._sorting_button.connect('clicked', - self.__sorting_button_clicked_cb) self.toolbar.insert(self._sorting_button, -1) self._sorting_button.connect('sort-property-changed', self.__sort_changed_cb) @@ -226,9 +224,6 @@ class MainToolbox(ToolbarBox): def __sort_changed_cb(self, button): self._update_if_needed() - def __sorting_button_clicked_cb(self, button): - self._sorting_button.palette.popup(immediate=True, state=1) - def _update_if_needed(self): new_query = self._build_query() if self._query != new_query: @@ -559,6 +554,9 @@ class SortingButton(ToolButton): self.props.tooltip = _('Sort view') self.props.icon_name = 'view-lastedit' + self.props.hide_tooltip_on_click = False + self.palette_invoker.props.toggle_palette = True + menu_box = PaletteMenuBox() self.props.palette.set_content(menu_box) menu_box.show() |