<table><tr><td style="">filipf 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/D17906">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/D17906#385682" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17906#385682</a>, <a href="https://phabricator.kde.org/p/mart/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mart</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I'm very much against this, I think it looks very bad :/</p></div>
</blockquote>

<p>I think the idea to have a light style for big titles is nice and could theoretically work, but in our case unfortunately fails in practice :/</p>

<p>The thing is, even if we didn't have a problem with fonts (btw defaults are not a guaranteed experience either, e.g. Fedora ships Plasma with "Sans") potentially failing, visual hierarchy is still something which doesn't work the best with the old code.</p>

<p>Attaching some screenshots from our main competitors for comparison's sake:</p>

<p><strong>Notifications (<a href="https://phabricator.kde.org/D17905" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17905</a>):</strong></p>

<p><a href="https://phabricator.kde.org/F6522734" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6522734: photo_2018-12-31_12-59-40.jpg</a></p>

<p><strong>App stores:</strong></p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Mac App Store</li>
</ul>

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

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

<ul class="remarkup-list">
<li class="remarkup-list-item">Windows Store</li>
</ul>

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

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

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

<ul class="remarkup-list">
<li class="remarkup-list-item">GNOME Software:</li>
</ul>

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

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

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

<p>With the exception of the case of the Windows Store (which does nonetheless manage to achieve some visual hierarchy with different font sizes and colors), others mostly actually have an opposite approach and go the bold/semi-bold route for the more prominent titles, which makes sense - you want to highlight the information contained in them. But the point of this patch is to just make sure the most important information is not made less important or even illegible, which is guaranteed if the labels are "regular" style.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>BRANCH</strong><div><div>no-light-headings (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17906">https://phabricator.kde.org/D17906</a></div></div><br /><div><strong>To: </strong>filipf, ngraham, Plasma, VDG, Frameworks<br /><strong>Cc: </strong>abetts, mart, plasma-devel, dkardarakos, apol, davidedmundson, hein<br /></div>