<table><tr><td style="">rkflx added a comment.
</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/D9627" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D9627#185604" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D9627#185604</a>, <a href="https://phabricator.kde.org/p/rkflx/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@rkflx</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>One more thing: Is the 100px max working for you? Mine is still capped at 64px</p></div>
</blockquote>

<p>Figured that one out: The problem is clearly with <a href="https://phabricator.kde.org/D9549" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D9549</a>, because it only changed <tt style="background: #ebebeb; font-size: 13px;">breezesettingsdata.kcfg</tt>, i.e. the values which would be considered valid in the config file. As most users use the GUI to edit, this would've needed changes to <tt style="background: #ebebeb; font-size: 13px;">breezeconfigurationui.ui</tt> so the spinbox would've allowed bigger values.</p>

<p>Going for the combobox, this is now moot. However, I want to beg everyone (again!) for the sake of quality: Please, if you do changes or "accept" a Diff, try them in the actual application. Try different user-set values, too. Only focussing on the code in an editor leads to bugs, which are much more expensive to fix after the fact.</p>

<blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D9627#185783" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D9627#185783</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I still don't see my changes when I open the Breeze options in the Window Decoration KCM. Any idea what I'm doing wrong?</p></div>
</blockquote>

<p>First, determine whether you are actually trying to load those files, e.g. by removing them (now it should not work anymore). Next, check whether you aren't accidentally doing code edits in the build dir (<tt style="background: #ebebeb; font-size: 13px;">ui_*.h</tt>) instead of the <tt style="background: #ebebeb; font-size: 13px;">.ui</tt> files (been there, done that ;).</p>

<p>For me, installing locally as a user works fine, the trick is then to set <tt style="background: #ebebeb; font-size: 13px;">QT_PLUGIN_PATH</tt> so the new files take precedence over the system installation.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R31 Breeze</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9627" rel="noreferrer">https://phabricator.kde.org/D9627</a></div></div><br /><div><strong>To: </strong>ngraham, VDG, Breeze, hpereiradacosta, abetts<br /><strong>Cc: </strong>rkflx, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>