----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101246/ -----------------------------------------------------------
(Updated May 1, 2011, 9:18 p.m.) Review request for rekonq. Changes ------- Fixing case index == 0 (which didn't lead to an error, but it's definitely cleaner this way). Getting rid of now unused property which was set with QVariant (both in tab switching and in load favorite functions) Summary ------- The principle of tab switching by shortcut (switchToTab()) is pretty much the same as loading favorites by shortcut (loadFavorite(const int index)), therefore a similar code should be used for both method invocations. As I was recommended to use QSignalMapper for loadFavorite(...), this class is now also used for switchToTab(...) as well. I couldn't figure out how to do this without having to instance another object of QSignalMapper class. Thus I have to use two seperate objects, which should be fine, I guess. Diffs (updated) ----- src/mainview.h d01bb5b src/mainview.cpp 19e4aae src/mainwindow.cpp 1ded983 Diff: http://git.reviewboard.kde.org/r/101246/diff Testing ------- Thanks, Thomas
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
