<?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>455753</bug_id>
          
          <creation_ts>2022-06-21 21:21:50 +0000</creation_ts>
          <short_desc>Present Windows search bar retains previous search on subsequent openings</short_desc>
          <delta_ts>2022-06-23 17:17:06 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Plasma</classification>
          <product>kwin</product>
          <component>effects-present-windows</component>
          <version>5.25.1</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>minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Mark S">wolfendeathx</reporter>
          <assigned_to name="KWin default assignee">kwin-bugs-null</assigned_to>
          <cc>kde</cc>
    
    <cc>notify</cc>
          
          <cf_commitlink>https://invent.kde.org/plasma/kwin/commit/dcc77bfa8fbca8032047fbbb5ab65382293488ac</cf_commitlink>
          <cf_versionfixedin>5.25.2</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2133558</commentid>
    <comment_count>0</comment_count>
    <who name="Mark S">wolfendeathx</who>
    <bug_when>2022-06-21 21:21:50 +0000</bug_when>
    <thetext>SUMMARY

When using the Present Windows effect, the search bar retains the previous search/filter query. Prior to 5.25.0 the search query was not retained.

STEPS TO REPRODUCE

1. Use the Present Windows effect (Ctrl+F9 or Ctrl+F10)
2. Type in the search field
3. Close out of the Present Windows effect (via the Escape key or by selecting a window)
4. Reopen the Present Windows effect

OBSERVED RESULT

The previous search query is retained

EXPECTED RESULT

The previous search query is not retained

SYSTEM INFO

Operating System: Arch Linux
KDE Plasma Version: 5.25.1
KDE Frameworks Version: 5.95.0
Qt Version: 5.15.5
Kernel Version: 5.18.5-arch1-1 (64-bit)
Graphics Platform: Wayland</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2133762</commentid>
    <comment_count>1</comment_count>
    <who name="Niklas Stephanblome">notify</who>
    <bug_when>2022-06-22 15:06:00 +0000</bug_when>
    <thetext>I already fixed this as part of my merge request that will very soon be merged: https://invent.kde.org/plasma/kwin/-/merge_requests/2562</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2133867</commentid>
    <comment_count>2</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2022-06-22 18:54:53 +0000</bug_when>
    <thetext>*** Bug 455786 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2134142</commentid>
    <comment_count>3</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2022-06-23 17:14:27 +0000</bug_when>
    <thetext>Git commit 6838b1132fad8b32c26409fd46b2e49803684dd6 by Nate Graham, on behalf of Niklas Stephanblom.
Committed on 23/06/2022 at 17:13.
Pushed by ngraham into branch &apos;master&apos;.

Windowview: Fix broken keyboard navigation while filtering

After the 5.25 update, one could not see which window was highlighted until one
manually unfocused the SearchField and then pressed any key to refresh the WindowHeap.
Also, the searchbar would (most of the time) &quot;absorb&quot; the arrow keys so one had to
also unfocus it to really be able to select windows with they keyboard. 

With this change, there is always one window highlighted while filtering using the
search box. Also, one can select another window with the arrow keys without manually
unfocusing the searchbox. This heavily improves the keyboard functionality in this
effect that got lost with the 5.25 update of presentwindows to windowview and
resolves complaints about the keyboard navigation being buggy. 
Related: bug 455633, bug 455764, bug 455099, bug 455586
FIXED-IN: 5.25.2

M  +6    -2    src/effects/windowview/qml/main.qml
M  +3    -0    src/effects/windowview/windowvieweffect.cpp

https://invent.kde.org/plasma/kwin/commit/6838b1132fad8b32c26409fd46b2e49803684dd6</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2134148</commentid>
    <comment_count>4</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2022-06-23 17:17:06 +0000</bug_when>
    <thetext>Git commit dcc77bfa8fbca8032047fbbb5ab65382293488ac by Nate Graham, on behalf of Niklas Stephanblom.
Committed on 23/06/2022 at 17:16.
Pushed by ngraham into branch &apos;Plasma/5.25&apos;.

Windowview: Fix broken keyboard navigation while filtering

After the 5.25 update, one could not see which window was highlighted until one
manually unfocused the SearchField and then pressed any key to refresh the WindowHeap.
Also, the searchbar would (most of the time) &quot;absorb&quot; the arrow keys so one had to
also unfocus it to really be able to select windows with they keyboard. 

With this change, there is always one window highlighted while filtering using the
search box. Also, one can select another window with the arrow keys without manually
unfocusing the searchbox. This heavily improves the keyboard functionality in this
effect that got lost with the 5.25 update of presentwindows to windowview and
resolves complaints about the keyboard navigation being buggy. 
Related: bug 455633, bug 455764, bug 455099, bug 455586
FIXED-IN: 5.25.2


(cherry picked from commit 6838b1132fad8b32c26409fd46b2e49803684dd6)

M  +6    -2    src/effects/windowview/qml/main.qml
M  +3    -0    src/effects/windowview/windowvieweffect.cpp

https://invent.kde.org/plasma/kwin/commit/dcc77bfa8fbca8032047fbbb5ab65382293488ac</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>