diff options
author | Lucian Branescu Mihaila <lucian.branescu@gmail.com> | 2010-07-19 16:28:10 (GMT) |
---|---|---|
committer | Lucian Branescu Mihaila <lucian.branescu@gmail.com> | 2010-07-19 16:28:10 (GMT) |
commit | c0fb71b99b70bc3c0e52f4c52d61cb895668ff2b (patch) | |
tree | 5792d2160d4d28a0d5419f32e13ce9a4ebb5839c /webtoolbar.py | |
parent | 90605dd362635ea334f9c25d6c9f5d8dea3e8614 (diff) |
Dumb down clever trick in webtoolbar.py. Flesh out pdfview with a better test harness.
Diffstat (limited to 'webtoolbar.py')
-rw-r--r-- | webtoolbar.py | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/webtoolbar.py b/webtoolbar.py index bbb3a10..ffa8131 100644 --- a/webtoolbar.py +++ b/webtoolbar.py @@ -408,17 +408,19 @@ class PrimaryToolbar(ToolbarBox): for menu_item in palette.menu.get_children(): palette.menu.remove(menu_item) - def populate(history_list, palette): - for history_item in history_list: - menu_item = MenuItem(history_item.get_title(), text_maxlen=60) - menu_item.connect('activate', self.__history_item_activated_cb, - history_item) + self._populate_history_palette( + history.get_back_list_with_limit(limit), back_palette) + self._populate_history_palette( + history.get_forward_list_with_limit(limit), forward_palette) + + def _populate_history_palette(self, history_list, palette): + for history_item in history_list: + menu_item = MenuItem(history_item.get_title(), text_maxlen=60) + menu_item.connect('activate', self.__history_item_activated_cb, + history_item) - palette.menu.append(menu_item) - menu_item.show() - - populate(history.get_back_list_with_limit(limit), back_palette) - populate(history.get_forward_list_with_limit(limit), forward_palette) + palette.menu.append(menu_item) + menu_item.show() def __history_item_activated_cb(self, menu_item, history_item): browser = self._tabbed_view.props.current_browser |