----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105118/#review14296 -----------------------------------------------------------
Ship it! Polarity determines whether north is visible top or bottom, so I don't know why the original code was there. I think you can even strip it down further to a simple qreal const direction = polarity < 0 ? -1 : 1; i.e. ignore the projection -- in non-spherical case polarity is 1. Note that your patch also fixes bug 216156. You could use zoomIn()/zoomOut() though instead of the hard-coded +/- 40. - Dennis Nienhüser On May 31, 2012, 12:52 p.m., Greg T wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105118/ > ----------------------------------------------------------- > > (Updated May 31, 2012, 12:52 p.m.) > > > Review request for Marble and Plasma. > > > Description > ------- > > I had to delete a bunch of code to fix the issue. But what was the purpose of > this code? Am I missing some cornercase? > > > This addresses bug 264755. > http://bugs.kde.org/show_bug.cgi?id=264755 > > > Diffs > ----- > > wallpapers/marble/marble.cpp 988a8b6 > > Diff: http://git.reviewboard.kde.org/r/105118/diff/ > > > Testing > ------- > > bug fixed. > > > Thanks, > > Greg T > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel