Bug 435644 - Add more useful monitor notifications
Summary: Add more useful monitor notifications
Status: RESOLVED FIXED
Alias: None
Product: konsole
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: Konsole Developer
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2021-04-12 08:57 UTC by Radics Baltazár
Modified: 2022-04-19 16:51 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: v22.04.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Radics Baltazár 2021-04-12 08:57:16 UTC
There are various options to monitor a terminal, all of which cause a notification `Bell in session 'New Profile'` to be shown when triggered.

This is not very useful, since (at least for me) all instances of konsole run the same profile, so it's basically not telling anything.  For me it would be the most useful if it rather showed the tab/window's title, but I suppose it would make sense to make it customizable with a format string like the tab/window title.
Comment 1 Nate Graham 2021-04-13 14:54:22 UTC
Good idea!
Comment 3 Radics Baltazár 2021-08-20 19:32:51 UTC
I'm on konsole 21.08.0, the notification only prints the raw title string, without formatting it first, so I get `Bell in '%w' (Session 'new')`, which is quite useless.
Comment 4 Bug Janitor Service 2022-01-07 23:47:02 UTC
A possibly relevant merge request was started @ https://invent.kde.org/utilities/konsole/-/merge_requests/572
Comment 5 Kurt Hindenburg 2022-01-10 23:10:31 UTC
Git commit 2b2dbdfd4edd695e8604cba2e8b96481250ba85e by Kurt Hindenburg, on behalf of Luis Javier Merino Morán.
Committed on 10/01/2022 at 22:59.
Pushed by hindenburg into branch 'master'.

Make %w and %# work for bell notifications

Since commit c3982cfb, bell notifications include the display title, in
addition to the name title, to help identifying which tab the
notification came from.  Unfortunately, %w (which can be quite useful in
identifying tabs) and %# weren't working for this case, neither for the
DBUS query:

qdbus $KONSOLE_DBUS_SERVICE $KONSOLE_DBUS_SESSION org.kde.konsole.Session.title 1

M  +10   -1    src/session/Session.cpp

https://invent.kde.org/utilities/konsole/commit/2b2dbdfd4edd695e8604cba2e8b96481250ba85e