<?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>403356</bug_id>
          
          <creation_ts>2019-01-18 13:38:44 +0000</creation_ts>
          <short_desc>move the lock of the filter bar on the left close the close button</short_desc>
          <delta_ts>2019-05-26 03:32:38 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>dolphin</product>
          <component>bars: filter</component>
          <version>unspecified</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>usability</keywords>
          <priority>NOR</priority>
          <bug_severity>wishlist</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>403379</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="pier andre">pier_andreit</reporter>
          <assigned_to name="Dolphin Bug Assignee">dolphin-bugs-null</assigned_to>
          <cc>nate</cc>
          
          <cf_commitlink>https://commits.kde.org/dolphin/c22df71df28315b8b1964671d3452fad5b6ae0f5</cf_commitlink>
          <cf_versionfixedin>19.08.0</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1832925</commentid>
    <comment_count>0</comment_count>
    <who name="pier andre">pier_andreit</who>
    <bug_when>2019-01-18 13:38:44 +0000</bug_when>
    <thetext>moving the lock button on the left of the bar could increase the usability becouse usually the bar is very long and one start to write on the left and to use the lock has to move to the end of the window on the right</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1833025</commentid>
    <comment_count>1</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2019-01-18 20:51:42 +0000</bug_when>
    <thetext>Not a terrible idea.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1855778</commentid>
    <comment_count>2</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2019-05-13 01:55:32 +0000</bug_when>
    <thetext>Here&apos;s a patch: https://phabricator.kde.org/D21177</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1858806</commentid>
    <comment_count>3</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2019-05-26 03:32:38 +0000</bug_when>
    <thetext>Git commit c22df71df28315b8b1964671d3452fad5b6ae0f5 by Nate Graham.
Committed on 26/05/2019 at 03:32.
Pushed by ngraham into branch &apos;master&apos;.

When filter bar is focused, switch to view when tab key is pressed

Summary:
Various people have requested that the tab key move keyboard focus from the filter bar&apos;s
text edit field to the main view. It already does this when the Enter and return keys
are pressed, but apparently this is not very intuitive and people expect Tab to work too.

This patch makes that behavior possible by moving the Lock button to the left of the
text edit field, and re-arranging the code so that the filter bar is initialized first.
This works because Qt assigns tab ordering by default according to the order of widget
construction. So if we simply construct the main view right after the filter bar, then
the tab ordering is set up this way automatically.
Related: bug 403379
FIXED-IN: 19.08.0

Test Plan:
1. Activate Filter bar
2. Hit Tab key
3. View has become focused
4. Play around in Dolphin for a while and notice no regressions or crashes

Reviewers: #dolphin, elvisangelaccio

Reviewed By: #dolphin, elvisangelaccio

Subscribers: kfm-devel

Tags: #dolphin

Differential Revision: https://phabricator.kde.org/D21177

M  +14   -12   src/dolphinviewcontainer.cpp
M  +1    -1    src/filterbar/filterbar.cpp

https://commits.kde.org/dolphin/c22df71df28315b8b1964671d3452fad5b6ae0f5</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>