<table><tr><td style="">vkrause 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/D20158">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/D20158#452099" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D20158#452099</a>, <a href="https://phabricator.kde.org/p/broulik/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@broulik</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p><a href="https://phabricator.kde.org/p/vkrause/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@vkrause</a> I just noticed xdg portal notification spec (the ones used for sandboxed flatpak apps) has four urgency levels: low, normal, *high*, critical.<br />
 Do you think it would be useful to add <tt style="background: #ebebeb; font-size: 13px;">HighUrgency</tt>, too? For Android, maybe? For freedesktop Notifications we could just map High to Normal then (like Gnome does it).<br />
 It's not released yet, we can still meddle the enums for another few weeks :)</p></div>
</blockquote>

<p>Yep, Urgent/High/Normal/Low would indeed match Android perfectly. Adding more enum values also should be possible later without breaking compatibility, unless we assume them to be in the right order anywhere. If we need them ordered, maybe explicitly assign them values to leave some space between them?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R289 KNotifications</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20158">https://phabricator.kde.org/D20158</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, Frameworks, vkrause, ogoffart, mck182<br /><strong>Cc: </strong>mlaurent, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>