<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>Kai Uwe Broulik wrote on 20190718::07:13:16 re: "<a href="https://phabricator.kde.org/D22365" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D22365</a>: KNotification macOS native support by NSNotificationCenter"</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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);">Thanks a lot!</pre></div></blockquote>

<p>Indeed. I've looked a few times in doing this myself but was never convinced that the amount of work was justified and some aspects seemed just impossible (like the kind of control over notification buttons that is expected in a KNotification, IIRC).<br />
(IOW I'll reserve my opinion whether or not I'll be happy with the native behaviour over what I've gotten used to ;))</p>

<p>Summer holidays are never a good time for me to get involved in coding/review efforts but I'll see if I can test this next week. Provided the NSNotification code still builds on my ageing 10.9 install...</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>notifybymacosnotificationcenter.mm:162<br />
 +<br />
 +    CFStringRef cfTitle = notification->title().toCFString(),<br />
 +                cfText = notification->text().toCFString();</p></blockquote>

<p>Why not just <tt style="background: #ebebeb; font-size: 13px;">toNSString()</tt>, given you convert it below anyway? Or is that ObjC awful ownership model quirk?</p></blockquote>

<p>CFString and NSString are supposed to be "toll-free bridged" so you can cast freely between them. I agree, if you need an NSString just use toNSString(); if anything the extra detour might lead to ARC quirks.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>+    close(notification);<br />
 +    notify(notification, config);</p></blockquote>

<p>Is there no way to transparently update a notification?</p></blockquote>

<p>TBH I'd be surprised if there were.</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/D22365">https://phabricator.kde.org/D22365</a></div></div><br /><div><strong>To: </strong>Inoki, rjvbb<br /><strong>Cc: </strong>broulik, kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns<br /></div>