[Konsole-devel] [Bug 305162] Konsole notification "Bell in session" should have button to bring up that session

Kasper Laudrup laudrup at stacktrace.dk
Thu Jan 3 13:17:20 UTC 2013


https://bugs.kde.org/show_bug.cgi?id=305162

Kasper Laudrup <laudrup at stacktrace.dk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |laudrup at stacktrace.dk

--- Comment #5 from Kasper Laudrup <laudrup at stacktrace.dk> ---
(In reply to comment #4)
> 
> Thanks for the patch. It really helps. I will try to make the feature and
> patch into KDE SC 4.11

I'm happy to hear that I can be helpful. I enjoy making these minor improvement
patches and since the response from you Konsole maintainers has been so
positive, I'll hopefully continue to do so.

Btw. my patch introduces a warning I forgot to fix about an unused variable in
the notificationClicked() function in TerminalDisplay.cpp. Simply removing the
name of the variable or using the Q_UNUSED macro gets rid of that warning.

I can resubmit the patch with that change if you wish?

-- 
You are receiving this mail because:
You are the assignee for the bug.



More information about the konsole-devel mailing list