D7765: make krunner accessible
Frederik Gladhorn
noreply at phabricator.kde.org
Mon Sep 11 21:00:55 UTC 2017
gladhorn added a comment.
Lovely! Just a bit of nit-picking.
INLINE COMMENTS
> view.cpp:65
>
> + //used only by on screen readers
> + setTitle(i18n("KRunner"));
I'd remove the "on" :)
> RunCommand.qml:120
> }
> - Keys.forwardTo: [listView, results]
> + //Keys.forwardTo: [listView, results]
> }
Left over comment?
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D7765
To: mart, #plasma, davidedmundson, broulik
Cc: gladhorn, 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/4c62a028/attachment.html>
More information about the Plasma-devel
mailing list