<table><tr><td style="">iasensio created this revision.<br />iasensio added reviewers: ngraham, KWin, Plasma.<br />Herald added a project: KWin.<br />Herald added a subscriber: kwin.<br />iasensio requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D29764">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Use <tt style="background: #ebebeb; font-size: 13px;">QSize</tt>/<tt style="background: #ebebeb; font-size: 13px;">QPoint</tt> to handle and store coordinate values (size and position)</p>

<p>Previously, the rules model stored the "coordinate" type properties as a <br />
<tt style="background: #ebebeb; font-size: 13px;">QString</tt> with format <tt style="background: #ebebeb; font-size: 13px;">x, y</tt>.</p>

<p>This fails when setting the properties to the config schema, as it requires<br />
a proper <tt style="background: #ebebeb; font-size: 13px;">QPoint</tt> or <tt style="background: #ebebeb; font-size: 13px;">QSize</tt> value, specially the latter which can't be <br />
convert from such a string.</p>

<p>BUG: 421055</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Add a new rule and set its position and size properties</li>
<li class="remarkup-list-item">Hitting apply stores the right values in <tt style="background: #ebebeb; font-size: 13px;">~\.config\kwinrulesrc</tt></li>
<li class="remarkup-list-item">Close the kcm and reopen, the values are loaded</li>
<li class="remarkup-list-item">Property detection still works for size and position</li>
</ul>

<p>Please note that there is a pre-existing bug of some position/sizes not being <br />
applied to the windows in some cases, when using <tt style="background: #ebebeb; font-size: 13px;">Apply Initially</tt>. <br />
Better try using the <tt style="background: #ebebeb; font-size: 13px;">Force</tt> policy.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>fixrulescoord</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29764">https://phabricator.kde.org/D29764</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcmkwin/kwinrules/package/contents/ui/RulesEditor.qml<br />
kcmkwin/kwinrules/package/contents/ui/ValueEditor.qml<br />
kcmkwin/kwinrules/ruleitem.cpp<br />
kcmkwin/kwinrules/ruleitem.h<br />
kcmkwin/kwinrules/rulesmodel.cpp</div></div></div><br /><div><strong>To: </strong>iasensio, ngraham, KWin, Plasma<br /><strong>Cc: </strong>kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>