Merge lp:~jamalta/gwibber/prev-tab into lp:gwibber/1.2

Proposed by Jamal Fanaian
Status: Rejected
Rejected by: Ken VanDine
Proposed branch: lp:~jamalta/gwibber/prev-tab
Merge into: lp:gwibber/1.2
Diff against target: None lines
To merge this branch: bzr merge lp:~jamalta/gwibber/prev-tab
Reviewer Review Type Date Requested Status
Jamal Fanaian (community) Disapprove
gwibber-committers Pending
Review via email: mp+8542@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Jamal Fanaian (jamalta) wrote :

This branch is no longer valid as gwibber 2.0 doesn't use tabs (not the same way at least).

review: Disapprove

Unmerged revisions

347. By Jamal Fanaian

Added keyboard shortcut to switch to previous tab (Ctrl+Shift+Tab)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'gwibber/client.py'
--- gwibber/client.py 2009-06-21 01:17:53 +0000
+++ gwibber/client.py 2009-07-08 20:51:14 +0000
@@ -287,7 +287,12 @@
287 if e.keyval == gtk.keysyms.F5:287 if e.keyval == gtk.keysyms.F5:
288 self.update()288 self.update()
289 return True289 return True
290 if e.keyval == gtk.keysyms.Tab and e.state & gtk.gdk.CONTROL_MASK:290 if e.keyval == gtk.keysyms.ISO_Left_Tab and e.state & gtk.gdk.CONTROL_MASK:
291 if self.tabs.get_current_page() == 0:
292 self.tabs.set_current_page(len(self.tabs) - 1)
293 else: self.tabs.prev_page()
294 return True
295 elif e.keyval == gtk.keysyms.Tab and e.state & gtk.gdk.CONTROL_MASK:
291 if len(self.tabs) == self.tabs.get_current_page() + 1:296 if len(self.tabs) == self.tabs.get_current_page() + 1:
292 self.tabs.set_current_page(0)297 self.tabs.set_current_page(0)
293 else: self.tabs.next_page()298 else: self.tabs.next_page()

Subscribers

People subscribed via source and target branches