<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/107530/">http://git.reviewboard.kde.org/r/107530/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On December 20th, 2012, 11:34 p.m., <b>Jarosław Staniek</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">For a test I added three buttons switching views in Kexi.
- A mockup how that would look:
http://kexi-project.org/pics/2.7/groupbutton/triple1-mockup.png
Screenshots:
- actual look using your newest revision 8: http://kexi-project.org/pics/2.7/groupbutton/triple2-actual.png
- proposed final look: http://kexi-project.org/pics/2.7/groupbutton/triple7-final.png - I am unable to remove space between buttons; it somehow comes from the style. So it's not like the mockup but close and usable.
- the final look with Cleanlooks style: http://kexi-project.org/pics/2.7/groupbutton/triple6-cleanlooks.png it needs some work later
The widget looks OK on 'exotic' styles such as Windows.
If that's OK, I propose to commit the 'final' look - KoGroupButton.cpp code modified by me: http://paste.kde.org/629012/raw/ - already contains fixes I proposed.
I think we need to start somewhere and eat this dog food.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">i like your idea in the mockup and think it works for most styles i can think of without looking too weird
But how about calculating a correct panel rect to cover all the buttons. Then there will be no space on windows either. The manually drawn seperators will have to be tweaked too, but that should be doable.</pre>
<br />
<p>- C.</p>
<br />
<p>On December 18th, 2012, 2:46 p.m., Jean-Nicolas Artaud wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Calligra and C. Boemann.</div>
<div>By Jean-Nicolas Artaud.</div>
<p style="color: grey;"><i>Updated Dec. 18, 2012, 2:46 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Implement triple button widget in the calligra widget libs.
This also can be used for double buttons (like in gwenview for instance).</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Use the triple button on the property docker.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>libs/widgets/CMakeLists.txt <span style="color: grey">(10e4e09)</span></li>
<li>libs/widgets/KoGroupButton.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/widgets/KoGroupButton.cpp <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107530/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/107530/s/865/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/03/StrokePropertiesDocker_400x100.jpg" style="border: 1px black solid;" alt="Strocke properties docker with the triple buttons" /></a>
<a href="http://git.reviewboard.kde.org/r/107530/s/869/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/04/fenetre_400x100.jpg" style="border: 1px black solid;" alt="MS Word uses it every where" /></a>
<a href="http://git.reviewboard.kde.org/r/107530/s/875/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/06/StrokePropertiesDocker-normalmode_400x100.jpg" style="border: 1px black solid;" alt="Normal Mode, Normal Size" /></a>
<a href="http://git.reviewboard.kde.org/r/107530/s/876/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/06/StrokePropertiesDocker-flatmode_400x100.jpg" style="border: 1px black solid;" alt="Flat Mode, Normal Size" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>