broulik added inline comments.

INLINE COMMENTS

> filechooser.cpp:167
>  
> +    if (options.contains(QStringLiteral("directory"))) {
> +        directory = options.value(QStringLiteral("directory")).toBool();

This check isn't needed, but is probably consistent with the rest and should be 
cleaned up separately afterwards

REPOSITORY
  R838 Flatpak Support: KDE Portal for XDG Desktop

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

To: jgrulich, apol, #plasma
Cc: broulik, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to