Bug 350001 - Konsole mistakenly believes it is non-visible when not focused
Summary: Konsole mistakenly believes it is non-visible when not focused
Status: RESOLVED FIXED
Alias: None
Product: konsole
Classification: Applications
Component: general (show other bugs)
Version: 15.04.0
Platform: Arch Linux Linux
: NOR minor
Target Milestone: ---
Assignee: Konsole Developer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-07 14:53 UTC by Nicolas F.
Modified: 2018-05-13 11:29 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas F. 2015-07-07 14:53:24 UTC
Konsole's notifications allow configuring a notification when a bell is emitted in a non-visible session. However, Konsole mistakenly sees itself as non-visible whenever it is not focused, which means that a user may accidentally scroll inside a Konsole window and thus produce a lot of "Bell in non-visible session" notifications.

Reproducible: Always

Steps to Reproduce:
1. Open a konsole window, enable notifications on bells in non-visible windows
2. Open a separate window next to it, make sure it does not cover the konsole window, and focus it
3. Move your mouse over to the konsole window (without focusing it) and scroll downwards. Alternatively, use
sleep 3; echo "^G"
in the konsole window and focus on the other window until the echo actually executes.

Actual Results:  
Konsole notifies the user about a bell caused by s

Expected Results:  
Konsole determines that the user sees the window, and does not emit a notification.
Comment 1 Kurt Hindenburg 2017-02-12 15:10:04 UTC
Yes, perhaps it should swap "non-visible" with "non-focused"
Comment 2 Kurt Hindenburg 2017-12-23 17:40:40 UTC
Git commit 3e7981a2917dd3f5a175f3e5be04a5000841a4f1 by Kurt Hindenburg.
Committed on 23/12/2017 at 17:39.
Pushed by hindenburg into branch 'master'.

Use "focused" instead of "visible" and "hidden" in notifications

Patch by ahmadsamir
Differential Revision: https://phabricator.kde.org/D9471

M  +21   -21   desktop/konsole.notifyrc

https://commits.kde.org/konsole/3e7981a2917dd3f5a175f3e5be04a5000841a4f1