KDE QFileDialog helper: support name filters without parenthesis.
ClosedPublic

Authored by dfaure on Jun 22 2017, 10:54 PM.

Details

Summary

The QFileDialog documentation says these are equivalent:

dialog.setNameFilter("All C++ files (*.cpp *.cc *.C *.cxx *.c++)");
dialog.setNameFilter("*.cpp *.cc *.C *.cxx *.c++");

so we should support both syntaxes.

Test Plan

See autotest addition; also tested together with my kdialog port to QFileDialog.

Diff Detail

Repository
R135 Integration for Qt applications in Plasma
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
dfaure created this revision.Jun 22 2017, 10:54 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJun 22 2017, 10:54 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
apol accepted this revision.Jun 23 2017, 12:42 AM
This revision is now accepted and ready to land.Jun 23 2017, 12:42 AM
This revision was automatically updated to reflect the committed changes.