<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="https://git.reviewboard.kde.org/r/114283/">https://git.reviewboard.kde.org/r/114283/</a>

     </td>

    </tr>

   </table>

   <br />

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 66e2b915f6c8bc82b0fc08819a1e1c1601721cdf by David Edmundson to branch master.</pre>

 <br />

<p>- Commit Hook</p>

<br />

<p>On June 19th, 2014, 6 p.m. UTC, David Edmundson wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">

 <tr>

  <td>

<div>Review request for KDE Frameworks and Plasma.</div>

<div>By David Edmundson.</div>

<p style="color: grey;"><i>Updated June 19, 2014, 6 p.m.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

plasma-framework

</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;">Use QtQuickControls ScrollView internally

Flickable has a very odd API in which contentHeight can't be trusted. we need to update our ScrollBars/ScrollArea to make use of the new

originX and originY in order to fix a lot of bugs that we currently see with scrollbars.

Instead of doing that, by porting to use QtQuick Controls we get all

these bug fixes as well as _significantly_ reducing our code.

Having a separate logic vs UI, makes things (IMHO) easier to read.

I don't expect to merge this as-is; I haven't included the plasma

feature that creates a SectionScroller if the content is a listview, but

that should be viable. As far as I know all other features (keyboard

controls / hiding buttons on touch screens) are still kept.

Files removed are as indication as to long term changes from making this

change, in order to do that we need to port all users from ScrollBars ->

ScrollArea. (which is worth doing anyway IMHO)

</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;">Widget Explorer in plasma-shell looks exactly as before.</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>src/declarativeimports/plasmaextracomponents/qml/ScrollArea.qml <span style="color: grey">(860c0e7)</span></li>

 <li>src/declarativeimports/plasmaextracomponents/qml/styles/ScrollViewStyle.qml <span style="color: grey">(PRE-CREATION)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/114283/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>