Bug 431295

Summary:	KWin rules kcm does not fokus search field when adding a new property
Product:	[Applications] systemsettings	Reporter:	Till Schäfer <till2.schaefer>
Component:	kcm_kwinrules	Assignee:	KWin default assignee <kwin-bugs-null>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	elman, isma.af, justin.zobel, kishore96, nate, plasma-bugs-null
Priority:	NOR
Version First Reported In:	5.20.5
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/frameworks/kirigami/commit/21de2a2912839a7076623910025ff88c5cc31c2e	Version Fixed/Implemented In:	5.81
Sentry Crash Report:
Bug Depends on:	433991
Bug Blocks:

Description Till Schäfer 2021-01-08 09:18:36 UTC

When pressing/clicking "Add Properties..." via the novel kwin rules kcm, it is not possible to immediately start typing to search the properties. Thus, I propose to focus the search field as soon the property list pops up. 


Operating System: Gentoo Linux
KDE Plasma Version: 5.20.5
KDE Frameworks Version: 5.77.0
Qt Version: 5.15.2
Kernel Version: 5.10.5-gentoo
OS Type: 64-bit
Processors: 8 × Intel® Core™ i7-4810MQ CPU @ 2.80GHz
Memory: 15.5 GiB of RAM
Graphics Processor: Mesa DRI Intel® HD Graphics 4600

Comment 1 Justin Zobel 2021-01-15 00:15:25 UTC

Thanks for the suggestion, seems like a sane idea to me. Set to wishlist.

Comment 2 Ismael Asensio 2021-03-05 08:52:31 UTC

*** Bug 433985 has been marked as a duplicate of this bug. ***

Comment 3 Ismael Asensio 2021-03-05 08:53:06 UTC

*** Bug 433330 has been marked as a duplicate of this bug. ***

Comment 4 David Edmundson 2021-03-28 10:18:53 UTC

Git commit 21de2a2912839a7076623910025ff88c5cc31c2e by David Edmundson.
Committed on 28/03/2021 at 10:18.
Pushed by davidedmundson into branch 'master'.

Fix focus handling in OverlaySheet to be managed as one FocusScope

Focus handling inside OverlaySheet had some issues, namely:
- if the contentItem was a flickable, we would be calling
forceActiveFocus on the unused invisible container for non-flickables
(contentItemParent)

 -ScrollArea was a non-focus scope that has focus set by default,
this meant it could take focus arbitrarily

This moves the FocusScope to the toplevel visible item

M  +32   -26   src/controls/templates/OverlaySheet.qml

https://invent.kde.org/frameworks/kirigami/commit/21de2a2912839a7076623910025ff88c5cc31c2e