<?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>489099</bug_id>
          
          <creation_ts>2024-06-24 11:53:11 +0000</creation_ts>
          <short_desc>Keyboard navigation with up arrow key is broken since reorg of switch</short_desc>
          <delta_ts>2025-02-10 20:40:15 +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>Power and Battery  widget</component>
          <version>6.1.0</version>
          <rep_platform>Other</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>regression</keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.0</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Christian (Fuchs)">kde</reporter>
          <assigned_to name="Plasma Bugs List">plasma-bugs-null</assigned_to>
          <cc>kde</cc>
    
    <cc>natalie_clarius</cc>
    
    <cc>nate</cc>
          
          <cf_commitlink>https://invent.kde.org/plasma/powerdevil/-/commit/241b7b7d6d0ee95b2c8d95016e198f8d8770e6c5</cf_commitlink>
          <cf_versionfixedin>6.3.1</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2331175</commentid>
    <comment_count>0</comment_count>
    <who name="Christian (Fuchs)">kde</who>
    <bug_when>2024-06-24 11:53:11 +0000</bug_when>
    <thetext>SUMMARY
Since re-ordering the inhibit action from the header into the plasmoid (why?) keyboard navigation is now broken.
The down arrow works as expected, the up arrow depends on what item the focus is currently at, but it&apos;s wrong.

STEPS TO REPRODUCE
1.  Update to plasma &gt;= 6.1
2.  Open the battery plasmoid
3.  Try to navigate by keyboard

OBSERVED RESULT
Up arrow doesn&apos;t work as it&apos;s still bound to the old layout

EXPECTED RESULT
Keys work as expected</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2331664</commentid>
    <comment_count>1</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2024-06-25 15:55:41 +0000</bug_when>
    <thetext>Can confirm. I&apos;ll see if I can fix it sometime soon.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2395795</commentid>
    <comment_count>2</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2025-02-07 19:37:36 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/plasma/powerdevil/-/merge_requests/507</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2396566</commentid>
    <comment_count>3</comment_count>
    <who name="Ismael Asensio">isma.af</who>
    <bug_when>2025-02-10 20:32:00 +0000</bug_when>
    <thetext>Git commit f7b5559a4cf88eff7d7c0f509b7e56aea5a5d2d1 by Ismael Asensio.
Committed on 10/02/2025 at 20:28.
Pushed by iasensio into branch &apos;master&apos;.

applets/battery: Fix navigation keyboard cycle for the Up key

When on the first item of the battery list, trying to force focus
to the header item fails, probably because it is wrapped within a
Column.

Just setting it to `null` works in this case, since it automatically
completes the `KeyNavigation.down` chain, but in reverse. This is
the same trick already being used in the footer.
FIXED-IN: 6.3.1

M  +1    -1    applets/batterymonitor/package/contents/ui/PopupDialog.qml

https://invent.kde.org/plasma/powerdevil/-/commit/f7b5559a4cf88eff7d7c0f509b7e56aea5a5d2d1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2396572</commentid>
    <comment_count>4</comment_count>
    <who name="Ismael Asensio">isma.af</who>
    <bug_when>2025-02-10 20:40:15 +0000</bug_when>
    <thetext>Git commit 241b7b7d6d0ee95b2c8d95016e198f8d8770e6c5 by Ismael Asensio.
Committed on 10/02/2025 at 20:34.
Pushed by iasensio into branch &apos;Plasma/6.3&apos;.

applets/battery: Fix navigation keyboard cycle for the Up key

When on the first item of the battery list, trying to force focus
to the header item fails, probably because it is wrapped within a
Column.

Just setting it to `null` works in this case, since it automatically
completes the `KeyNavigation.down` chain, but in reverse. This is
the same trick already being used in the footer.
FIXED-IN: 6.3.1


(cherry picked from commit f7b5559a4cf88eff7d7c0f509b7e56aea5a5d2d1)

Co-authored-by: Ismael Asensio &lt;isma.af@gmail.com&gt;

M  +1    -1    applets/batterymonitor/package/contents/ui/PopupDialog.qml

https://invent.kde.org/plasma/powerdevil/-/commit/241b7b7d6d0ee95b2c8d95016e198f8d8770e6c5</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>