<table><tr><td style="">rjvbb 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/D22365">View Revision</a></tr></table><br /><div><div><p>As I thought this needs some hacking on OS X < 10.10 but a priori all one loses is the user notifications.</p>

<p>I'm missing a test case in this diff, i.e. a description how you can test these new notifications because I wouldn't know off the top of my head which applications post this kind of notifications (and notifications via DBus work on my system because I have the necessary services running).</p>

<p>Another question: Qt officially supports/allows all QPA plugins that work on a given platform, which means that it's not impossible to use the XCB "backend" on Mac. It looks like the new code doesn't consider this possibility, and sadly my KWindowSystem changes for Mac have still not made it into a release (they'd allow a runtime <tt style="background: #ebebeb; font-size: 13px;">isCocoa</tt> check).<br />
Still, I would prefer to see something of the sort</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">if (d->portalDBusServiceExists) {
  plugin = new NotifyByPortal(this);
} else {
#ifdef Q_OS_MACOS
  plugin = new NotifyByMacOSNotificationCenter(this);
#else
  plugin = new NotifyByPopup(this);
#endif
}</pre></div>

<p>FWIW: you should use the <tt style="background: #ebebeb; font-size: 13px;">Q_OS_MACOS</tt> token to test for the Mac desktop OS.</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22365">https://phabricator.kde.org/D22365</a></div></div><br /><div><strong>To: </strong>Inoki, rjvbb<br /><strong>Cc: </strong>nicolasfella, broulik, kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns<br /></div>