[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 ninj...@gmail.com changed: What|Removed |Added CC||philipp.k...@gmail.com --- Comment #8 from ninj...@gmail.com --- *** Bug 452767 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 ninj...@gmail.com changed: What|Removed |Added Version Fixed In||v22.04.0 CC||ninj...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 Kurt Hindenburg changed: What|Removed |Added Status|ASSIGNED|RESOLVED Latest Commit||https://invent.kde.org/util ||ities/konsole/commit/0fe44c ||52310a6c276388cf30c81b6fc45 ||a39da32 Resolution|--- |FIXED --- Comment #7 from Kurt Hindenburg --- Git commit 0fe44c52310a6c276388cf30c81b6fc45a39da32 by Kurt Hindenburg, on behalf of Luis Javier Merino Morán. Committed on 11/01/2022 at 15:57. Pushed by hindenburg into branch 'master'. Pass widget again to KNotification on bell 67508680 refactored TerminalBell out from TerminalDisplay, and in the process stopped passing the TerminalDisplay widget to KNotification, which resulted in issues when notifications where configured to mark the task bar entry or run a command with %w/%t substitutions (for window id and window title). Many thanks to Nicole Kleinhoff and Jan Palus who identified the commit that introduced the regression. Related: bug 445520, bug 443117 M +3-2src/terminalDisplay/TerminalBell.cpp M +2-1src/terminalDisplay/TerminalBell.h M +1-1src/terminalDisplay/TerminalDisplay.cpp https://invent.kde.org/utilities/konsole/commit/0fe44c52310a6c276388cf30c81b6fc45a39da32 -- You are receiving this mail because: You are watching all bug changes.
[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 Bug Janitor Service changed: What|Removed |Added Status|CONFIRMED |ASSIGNED --- Comment #6 from Bug Janitor Service --- A possibly relevant merge request was started @ https://invent.kde.org/utilities/konsole/-/merge_requests/573 -- You are receiving this mail because: You are watching all bug changes.
[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 Andriy Gapon changed: What|Removed |Added CC||a...@uabsd.com --- Comment #5 from Andriy Gapon --- I can confirm the same problem with, e.g., '%w' using Konsole 21.08.3. -- You are receiving this mail because: You are watching all bug changes.
[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 --- Comment #4 from Kurt Hindenburg --- OK thanks that makes sense as to why my testing was confusing me -- You are receiving this mail because: You are watching all bug changes.
[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 --- Comment #3 from Jan Palus --- Not sure if I follow, do you mean where it's replaced (https://invent.kde.org/frameworks/knotifications/-/commit/e25f1f552a9ce6b684e573f26c82cfc76bfcbe18) or how do I use it (for mimicking urgency hint in sway) or perhaps something else? -- You are receiving this mail because: You are watching all bug changes.
[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 --- Comment #2 from Kurt Hindenburg --- Can you explain where you're getting %t? -- You are receiving this mail because: You are watching all bug changes.
[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 Kurt Hindenburg changed: What|Removed |Added Ever confirmed|0 |1 Status|REPORTED|CONFIRMED --- Comment #1 from Kurt Hindenburg --- Jonah Brüchert do you have to look at this? -- You are receiving this mail because: You are watching all bug changes.
[konsole] [Bug 443411] Regression: %w and %t placeholders not available in bell notifications
https://bugs.kde.org/show_bug.cgi?id=443411 Jan Palus changed: What|Removed |Added CC||jbb.m...@gmx.de, ||tum...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
