<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>495264</bug_id>
          
          <creation_ts>2024-10-23 19:29:40 +0000</creation_ts>
          <short_desc>Missing notification about modifier key status changes</short_desc>
          <delta_ts>2024-10-28 14:42:52 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Plasma</classification>
          <product>plasmashell</product>
          <component>Notifications</component>
          <version>6.2.1</version>
          <rep_platform>Fedora RPMs</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.0</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Coacher">itumaykin+kde</reporter>
          <assigned_to name="Plasma Bugs List">plasma-bugs-null</assigned_to>
          <cc>jayabbhi</cc>
    
    <cc>kde</cc>
    
    <cc>nate</cc>
    
    <cc>nicolas.fella</cc>
          
          <cf_commitlink>https://invent.kde.org/plasma/kwin/-/commit/1485605dbc1d7a792119dc70d131c6d47e32095f</cf_commitlink>
          <cf_versionfixedin>6.2.3</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2368670</commentid>
    <comment_count>0</comment_count>
    <who name="Coacher">itumaykin+kde</who>
    <bug_when>2024-10-23 19:29:40 +0000</bug_when>
    <thetext>STEPS TO REPRODUCE
1. Enable notifications for the modifier keys in systemsettings (Accessibility -&gt; Modifier Keys)
2. Press modifier key, e.g. CapsLock

OBSERVED RESULT
No notification

EXPECTED RESULT
Notification is shown telling about the CapsLock status change


SOFTWARE/OS VERSIONS
$ kinfo
Operating System: Fedora Linux 40
KDE Plasma Version: 6.2.1
KDE Frameworks Version: 6.7.0
Qt Version: 6.7.2
Kernel Version: 6.11.4-201.fc40.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 8 × 11th Gen Intel® Core™ i7-1165G7 @ 2.80GHz
Memory: 15.3 GiB of RAM
Graphics Processor: Mesa Intel® Xe Graphics

ADDITIONAL INFORMATION
This is a regression from Plasma-6.1 where modifier key notifications worked as expected. Other notifications also work as expected.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2368717</commentid>
    <comment_count>1</comment_count>
    <who name="Nicolas Fella">nicolas.fella</who>
    <bug_when>2024-10-23 22:12:08 +0000</bug_when>
    <thetext>This works as expected for me, which is not surprising given I literally implemented support for this on Wayland for 6.2 (https://invent.kde.org/plasma/kwin/-/merge_requests/6068).

Previously it would only work if XWayland was configured to allow key eavesdropping.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2368775</commentid>
    <comment_count>2</comment_count>
    <who name="Coacher">itumaykin+kde</who>
    <bug_when>2024-10-24 08:00:06 +0000</bug_when>
    <thetext>(In reply to Nicolas Fella from comment #1)
&gt; This works as expected for me, which is not surprising given I literally
&gt; implemented support for this on Wayland for 6.2
&gt; (https://invent.kde.org/plasma/kwin/-/merge_requests/6068).
&gt; 
&gt; Previously it would only work if XWayland was configured to allow key
&gt; eavesdropping.

Hmm, it was working for me before, but stopped working in Plasma 6.2.

I&apos;ve tried disabling key X11 from reading keystrokes in system settings, it didn&apos;t help.

Could it be related to https://bugs.kde.org/show_bug.cgi?id=458699#c14 ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2369624</commentid>
    <comment_count>3</comment_count>
    <who name="Nicolas Fella">nicolas.fella</who>
    <bug_when>2024-10-27 16:22:20 +0000</bug_when>
    <thetext>*** Bug 495430 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2369625</commentid>
    <comment_count>4</comment_count>
    <who name="Nicolas Fella">nicolas.fella</who>
    <bug_when>2024-10-27 16:24:50 +0000</bug_when>
    <thetext>okay, I see what&apos;s wrong. I made a mistake while handling the notification categories. It works when the &quot;A modifier key has been locked&quot; notification is enabled, but not with &quot;A lock key has been activated&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2369626</commentid>
    <comment_count>5</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2024-10-27 16:27:42 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/6704</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2369649</commentid>
    <comment_count>6</comment_count>
    <who name="Abbhinav Jayaraman">jayabbhi</who>
    <bug_when>2024-10-27 18:55:54 +0000</bug_when>
    <thetext>(In reply to Nicolas Fella from comment #4)
&gt; okay, I see what&apos;s wrong. I made a mistake while handling the notification
&gt; categories. It works when the &quot;A modifier key has been locked&quot; notification
&gt; is enabled, but not with &quot;A lock key has been activated&quot;

I am confused by this comment, but I just installed Fedora 40 KDE Spin, and it seems like for any notification to work you need to have the &quot;A modifier has been unlocked&quot; 

Not sure if it is relevant, or if that&apos;s what you meant.
Operating System: Fedora Linux 40
KDE Plasma Version: 6.2.2
KDE Frameworks Version: 6.7.0
Qt Version: 6.7.2
Kernel Version: 6.11.4-201.fc40.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 16 × AMD Ryzen 7 5800U with Radeon Graphics
Memory: 15.0 GiB of RAM
Graphics Processor: AMD Radeon Graphics
Manufacturer: Acer
Product Name: Swift SFX14-41G
System Version: V1.10</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2369831</commentid>
    <comment_count>7</comment_count>
    <who name="Nicolas Fella">nicolas.fella</who>
    <bug_when>2024-10-28 10:09:45 +0000</bug_when>
    <thetext>&gt; it seems like for any notification to work you need to have the &quot;A modifier has been unlocked&quot; 

That&apos;s what I meant</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2369834</commentid>
    <comment_count>8</comment_count>
    <who name="Nicolas Fella">nicolas.fella</who>
    <bug_when>2024-10-28 10:17:50 +0000</bug_when>
    <thetext>Git commit b786c7e930ed15fa3a372c8a883d872ec4050e77 by Nicolas Fella.
Committed on 28/10/2024 at 10:08.
Pushed by nicolasfella into branch &apos;master&apos;.

plugins/keynotification: Fix event ID for notification

M  +1    -1    src/plugins/keynotification/keynotification.cpp

https://invent.kde.org/plasma/kwin/-/commit/b786c7e930ed15fa3a372c8a883d872ec4050e77</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2369840</commentid>
    <comment_count>9</comment_count>
    <who name="Nicolas Fella">nicolas.fella</who>
    <bug_when>2024-10-28 10:55:34 +0000</bug_when>
    <thetext>Git commit 1485605dbc1d7a792119dc70d131c6d47e32095f by Nicolas Fella.
Committed on 28/10/2024 at 10:20.
Pushed by vladz into branch &apos;Plasma/6.2&apos;.

plugins/keynotification: Fix event ID for notification


(cherry picked from commit b786c7e930ed15fa3a372c8a883d872ec4050e77)

Co-authored-by: Nicolas Fella &lt;nicolas.fella@gmx.de&gt;

M  +1    -1    src/plugins/keynotification/keynotification.cpp

https://invent.kde.org/plasma/kwin/-/commit/1485605dbc1d7a792119dc70d131c6d47e32095f</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>