----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107530/#review23130 -----------------------------------------------------------
Will you use the buttons also for the shown dialogs? libs/widgets/KoTripleButton.cpp <http://git.reviewboard.kde.org/r/107530/#comment17669> The public should not be indented libs/widgets/KoTripleButton.cpp <http://git.reviewboard.kde.org/r/107530/#comment17670> There should be no spaces after ( and before ), also no space after Private - Thorsten Zachmann On Dec. 7, 2012, 7:37 a.m., Jean-Nicolas Artaud wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107530/ > ----------------------------------------------------------- > > (Updated Dec. 7, 2012, 7:37 a.m.) > > > Review request for Calligra and C. Boemann. > > > Description > ------- > > Implement triple button widget in the calligra widget libs. > This also can be used for double buttons (like in gwenview for instance). > > > Diffs > ----- > > libs/widgets/CMakeLists.txt 10e4e09 > libs/widgets/KoTripleButton.h PRE-CREATION > libs/widgets/KoTripleButton.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/107530/diff/ > > > Testing > ------- > > Use the triple button on the property docker. > > > Screenshots > ----------- > > Strocke properties docker with the triple buttons > http://git.reviewboard.kde.org/r/107530/s/865/ > MS Word uses it every where > http://git.reviewboard.kde.org/r/107530/s/869/ > Normal Mode, Normal Size > http://git.reviewboard.kde.org/r/107530/s/875/ > Flat Mode, Normal Size > http://git.reviewboard.kde.org/r/107530/s/876/ > > > Thanks, > > Jean-Nicolas Artaud > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
