| Summary: | Add more useful monitor notifications | ||
|---|---|---|---|
| Product: | [Applications] konsole | Reporter: | Radics Baltazár <baltazar.radics> |
| Component: | general | Assignee: | Konsole Developer <konsole-devel> |
| Status: | RESOLVED FIXED | ||
| Severity: | wishlist | CC: | nate, ninjalj |
| Priority: | NOR | Keywords: | usability |
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/utilities/konsole/commit/2b2dbdfd4edd695e8604cba2e8b96481250ba85e | Version Fixed In: | v22.04.0 |
| Sentry Crash Report: | |||
|
Description
Radics Baltazár
2021-04-12 08:57:16 UTC
Good idea! 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. A possibly relevant merge request was started @ https://invent.kde.org/utilities/konsole/-/merge_requests/572 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 |