D7766: make krunner accessible

Kai Uwe Broulik noreply at phabricator.kde.org
Mon Sep 11 17:10:22 UTC 2017


broulik added inline comments.

INLINE COMMENTS

> ResultDelegate.qml:50
> +    Accessible.name: displayLabel.text
> +    Accessible.description: i18n("%1, in category %2", subtextLabel.text.length > 0 ? subtextLabel.text : " ", ListView.section)
>      property bool __pressed: false

So we end up with " , in category foo" if subtextLabel is empty?

> ResultDelegate.qml:246
> +                        Accessible.name: modelData.text
> +                        Accessible.description: ""
>                          checkable: checked

Not needed?

> ResultDelegate.qml:249
>                          checked: resultDelegate.activeAction === index
> +                        focus: resultDelegate.activeAction === index
>  

I don't understand this focus stuff at all but if it works..

> ResultsView.qml:140
> +                if (currentIndex == 0) {
> +                    listView.nextItemInFocusChain(false).forceActiveFocus();
> +                    return;

Can we perhaps bind the "isCurrentItem" to activeFocus instead of doing this in addition to incrementing/decrementing the index?

REPOSITORY
  R112 Milou

REVISION DETAIL
  https://phabricator.kde.org/D7766

To: mart, #plasma, davidedmundson, broulik
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170911/08ee573b/attachment.html>


More information about the Plasma-devel mailing list