davidedmundson added inline comments.

INLINE COMMENTS

> filechooser.cpp:287
>  
>      QFileDialog *fileDialog = new QFileDialog();
>      fileDialog->setWindowTitle(title);

Given this is the plasma portal, it might be a good case to use the KFileWidget 
directly rather than have a proxy layer in the way which converts filter 
strings back again.

KFileWidget::setFilter takes an expression directly - and if it doesn't match 
perfectly we can change KIO to come up with something that does.

REPOSITORY
  R838 Flatpak Support: KDE Portal for XDG Desktop

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

To: jgrulich, broulik
Cc: davidedmundson, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to