<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/122660/">https://git.reviewboard.kde.org/r/122660/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On February 21st, 2015, 1:44 p.m. UTC, <b>Hugo Pereira Da Costa</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">"Preinstalled colours schemes to ensure consistent colouring"
that is a regression with respect to exiting code. Its a no go. Please include.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">"Updated the behaviour of the resize grip - fixed bug for fullscreen"
Can you post a bug report related to the bug ? There was indeed an issue with size grip and fullscreen window, but this was fixed, as far as I know. See https://bugs.kde.org/show_bug.cgi?id=343988 and commit therein</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">"Redesigned buttons and tweaked the titlebar slightly."
Can you post screenshots ? Were these changes discussed in the Visual Design Group forum ? (https://forum.kde.org/viewforum.php?f=285)</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">"Font weight will now affect the boldness of icons in buttons"
Same question.
No other element in the widget style has this behavior (icons, frame sizes, etc.), so I'd like to know the rationale behind it and get this discussed.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I have not tested the changes yet, but will do asap</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">"that is a regression with respect to exiting code. Its a no go. Please include."
Sorry, I should have said "tested with preinstalled colour schemes"; I was just checking to ensure colours were sane for themes; make sure the buttons didn't turn neon-pink when using obsidian or anything like that.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">"There was indeed an issue with size grip and fullscreen window ..."
I re-wrote the grip behaviour with the input of Jens (another VDG member); I wasn't specifically aiming to do it with the change to the grip display code, but I was aware that it would be fixed in my build. The new behaviour should hide the grip on maximised windows and inactive windows; on maximised windows the grip is pointless, and on inactive windows the grip is not obvious, but still covers scrollbars or status items...</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">"Can you post screenshots ?"
Of course; they're included now.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">"Were these changes discussed in the Visual Design Group forum ?"
Not on the forum, no. I did refer to Jens during the sprint, and other members of the VDG have responded positivly to my screenshot on G+. This is also an initial version, and assuming it flies I'll be updating based on feedback accordingly. The design is also evolutionary anyway.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">"No other element in the widget style has this behavior (icons, frame sizes, etc.), so I'd like to know the rationale behind it and get this discussed."
I was going to add a config option but if I had of done this, but I chose to use the font width to avoid overburdening the settings dialog with tweaker stuff - it's less than a 0.7px difference.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I'd be perfectly fine removing the feature; when first starting I did not know there was a float variant of setWidth - so I could not get the line to a happy medium; now that I have it at a 'good' width it was kept simply because it could 'follow along' with the font easily. The edit is <1 line.</p></pre>
<br />
<p>- Ken</p>
<br />
<p>On February 21st, 2015, 2:53 p.m. UTC, Ken Vermette wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma and Hugo Pereira Da Costa.</div>
<div>By Ken Vermette.</div>
<p style="color: grey;"><i>Updated Feb. 21, 2015, 2:53 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
breeze
</div>
<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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Overhaul of the Breeze Window decoration drawing code;
- Redesigned buttons and tweaked the titlebar slightly.
- Updated the behaviour of the resize grip - fixed bug for fullscreen
- Font weight will now affect the boldness of icons in buttons</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Note; Buttons are not animated yet in this variant.</p></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;"><ul style="padding: 0;text-rendering: inherit;margin: 0 0 0 1em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Tried out preinstalled colours schemes to ensure consistent colouring</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Viewed various button sizes</li>
</ul></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>kdecoration/breezebutton.cpp <span style="color: grey">(5ac0cfe)</span></li>
<li>kdecoration/breezedecoration.h <span style="color: grey">(9eb6c65)</span></li>
<li>kdecoration/breezedecoration.cpp <span style="color: grey">(b474a8b)</span></li>
<li>kdecoration/breezebutton.h <span style="color: grey">(c43959a)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/122660/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/02/21/66f8d3d8-5852-4b79-b211-339cbc7bf712__newdecos.png">Updated Decos</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/02/21/20580b13-3bb1-4020-9ddf-8d1253184a45__snapshot1.png">Full Windows</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>