<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: VDG, Plasma.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />ngraham 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/D25413">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Right now, the Weather widget's compact representation's temperature text is not the same<br />
size as the text for the clock, which looks especially awkward when both are placed<br />
adjacent to one another. In addition, the text size has no upper bound, so it can become<br />
absolutely gigantic with a tall panel.</p>

<p>This patch fixes both issues by using the same size logic used in the digital clock,<br />
ensuring that the text sizes are identical.</p>

<p>I know that this uses a magic value, and that this is bad and I should feel bad.<br />
However it's the same one used in the digital clock. Once more than one widget is using<br />
this "height * 0.7" value, it becomes feasible to factor it out into something in<br />
plasma-framework maybe so we can use it to get consistent text size everywhere.</p>

<p>Hopefully that can be material for a subsequent patch though.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>No change for vertical panels.</p>

<p>Before, default horizontal panel: <a href="https://phabricator.kde.org/F7772047" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7772047: Before, small panel.png</a></p>

<p>After, default horizontal panel: <a href="https://phabricator.kde.org/F7772044" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7772044: After, small panel.png</a></p>

<p>Before: double-height horizontal panel: <a href="https://phabricator.kde.org/F7772046" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7772046: Before, double-height panel.png</a></p>

<p>After, double-height horizontal panel: <a href="https://phabricator.kde.org/F7772045" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7772045: After, double-height panel.png</a></p>

<p>Before, absurdly humongous horizontal panel: <a href="https://phabricator.kde.org/F7772048" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7772048: Before, humongously enormous panel.png</a></p>

<p>After, absurdely humongous horizontal panel: <a href="https://phabricator.kde.org/F7772049" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7772049: After, humongously enormous panel.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R114 Plasma Addons</div></div></div><br /><div><strong>BRANCH</strong><div><div>fix-weather-widget-temperature-text-size</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25413">https://phabricator.kde.org/D25413</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/weather/package/contents/ui/IconAndTextItem.qml</div></div></div><br /><div><strong>To: </strong>ngraham, VDG, Plasma<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>