new: calling favorite pages via shortcut

Review Request #101043 - Created April 6, 2011 and submitted

Information
Thomas Murach
rekonq
Master
Reviewers
rekonq
This patch adds the following functionality: Pressing Ctrl + 1 up to Ctrl + 9 will load the favorite page with number 1 ... 9 in the current tab. I find this very useful when using Opera.
This is my first patch. So please tell me if there is something to improve in my kind of coding / formatting / ...
The code itself is pretty much copied from the function SwitchToTab() in MainView. 
Numbers higher than 9 do not make sense as shortcuts (in my opinion) and are therefore omitted, as well as zero
Yes. Couldn't find any problems.
For example pressing Ctrl + _Number_X_ doesn't crash if you have less than _Number_X_ favorite pages
Furkan Üzümcü
Thomas Murach
Pierre Rossi
Thomas Murach
Felix Rohrbach
Thomas Murach
Pierre Rossi
Andrea Diamantini
Thomas Murach
Review request changed

Change Summary:

Making QSignalMapper non-member. Finally the code looks fine to me :)
If this gets accepted, could someone with write-permissions apply the patch for me? Thanks a lot.

Diff:

Revision 5 (+26 -1)

Show changes

Andrea Diamantini
Going merging...
Loading...