> On April 8, 2011, 10:36 p.m., Pierre Rossi wrote: > > src/tabbar.cpp, line 188 > > <http://git.reviewboard.kde.org/r/101062/diff/3/?file=14052#file14052line188> > > > > while you're at it, it'd be nice to replace this BASE_WIDTH_DIVISOR > > deine by a "static const int BaseWidthDivisor"
should static const int baseWidthDivisor be a static member of the class? Or should it be a global variable? - Tirtha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101062/#review2505 ----------------------------------------------------------- On April 8, 2011, 7:40 p.m., Tirtha Chatterjee wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101062/ > ----------------------------------------------------------- > > (Updated April 8, 2011, 7:40 p.m.) > > > Review request for rekonq. > > > Summary > ------- > > Right now, the size of a tab preview keeps decreasing as tab size decreases. > This patch fixes the size of preview to the default base width of a tab (the > max width of the tab) which a tab has at the beginning. > > > Diffs > ----- > > src/tabbar.cpp 1ab357f > > Diff: http://git.reviewboard.kde.org/r/101062/diff > > > Testing > ------- > > Tested. Working properly. > > > Thanks, > > Tirtha > >
_______________________________________________ rekonq mailing list [email protected] https://mail.kde.org/mailman/listinfo/rekonq
