<?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>494261</bug_id>
          
          <creation_ts>2024-10-07 20:27:48 +0000</creation_ts>
          <short_desc>[ProcessesView] Clicking on the already-selected item deselects it but the selection highlight doesn&apos;t go away</short_desc>
          <delta_ts>2024-11-22 17:16:17 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>plasma-systemmonitor</product>
          <component>general</component>
          <version>6.1.5</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></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="popov895">popov895</reporter>
          <assigned_to name="KSysGuard Developers">ksysguard-bugs</assigned_to>
          <cc>ahiemstra</cc>
    
    <cc>kdedev</cc>
    
    <cc>nate</cc>
    
    <cc>plasma-bugs-null</cc>
    
    <cc>popov895</cc>
          
          <cf_commitlink>https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/13911268973ddde2304229a6565ce8429ff48ccd</cf_commitlink>
          <cf_versionfixedin>6.2.4</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2363506</commentid>
    <comment_count>0</comment_count>
    <who name="popov895">popov895</who>
    <bug_when>2024-10-07 20:27:48 +0000</bug_when>
    <thetext>SUMMARY

Clicking on the selected item in the Processes View disables the “End process” button. Same on Plasma 6.2.

STEPS TO REPRODUCE

1. Select any item in the Processes View: the &quot;End Process&quot; button becomes enabled
2. Click the selected item again: the &quot;End Process&quot; button becomes disabled

SOFTWARE/OS VERSIONS

Operating System: openSUSE Tumbleweed 20241006
KDE Plasma Version: 6.1.5
KDE Frameworks Version: 6.6.0
Qt Version: 6.7.3</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2363508</commentid>
    <comment_count>1</comment_count>
    <who name="popov895">popov895</who>
    <bug_when>2024-10-07 20:28:06 +0000</bug_when>
    <thetext>*** Bug 494257 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374083</commentid>
    <comment_count>2</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2024-11-13 19:29:08 +0000</bug_when>
    <thetext>Can reproduce.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374150</commentid>
    <comment_count>3</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2024-11-13 22:23:04 +0000</bug_when>
    <thetext>Seems like the item actually gets deselected, but the selection highlight doesn&apos;t show it. So the behavior is correct (or at least, as intended) but the UI isn&apos;t accurate.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374153</commentid>
    <comment_count>4</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2024-11-13 22:27:24 +0000</bug_when>
    <thetext>(in BaseCellDelegate.qml)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2376011</commentid>
    <comment_count>5</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2024-11-21 11:22:56 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/315</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2376287</commentid>
    <comment_count>6</comment_count>
    <who name="Akseli Lahtinen">akselmo</who>
    <bug_when>2024-11-22 10:01:21 +0000</bug_when>
    <thetext>Git commit b0c9c7b7a8c9becbc427ac4756016446f7ed6c88 by Akseli Lahtinen.
Committed on 22/11/2024 at 10:01.
Pushed by akselmo into branch &apos;master&apos;.

BaseCellDelegate: Use correct colorSet when selected

When item is set selected, it will change its colorSet to Selection as
well, but it will not change back to normal on deselect. See https://invent.kde.org/frameworks/qqc2-desktop-style/-/blob/master/org.kde.desktop/TreeViewDelegate.qml?ref_type=heads#L41

This makes sure we use the correct colorset here.

M  +1    -1    src/table/BaseCellDelegate.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/b0c9c7b7a8c9becbc427ac4756016446f7ed6c88</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2376289</commentid>
    <comment_count>7</comment_count>
    <who name="Akseli Lahtinen">akselmo</who>
    <bug_when>2024-11-22 10:08:50 +0000</bug_when>
    <thetext>Git commit 13911268973ddde2304229a6565ce8429ff48ccd by Akseli Lahtinen.
Committed on 22/11/2024 at 10:06.
Pushed by akselmo into branch &apos;Plasma/6.2&apos;.

BaseCellDelegate: Use correct colorSet when selected

When item is set selected, it will change its colorSet to Selection as
well, but it will not change back to normal on deselect. See https://invent.kde.org/frameworks/qqc2-desktop-style/-/blob/master/org.kde.desktop/TreeViewDelegate.qml?ref_type=heads#L41

This makes sure we use the correct colorset here.


(cherry picked from commit b0c9c7b7a8c9becbc427ac4756016446f7ed6c88)

a1441383 BaseCellDelegate: Use correct colorSet when selected

Co-authored-by: Akseli Lahtinen &lt;akselmo@akselmo.dev&gt;

M  +1    -1    src/table/BaseCellDelegate.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/13911268973ddde2304229a6565ce8429ff48ccd</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>