Created attachment 114068 [details] screenshot showing the button order when finishing the importer (german text) Usually when I run the CSV import wizard, then I like to import an CSV file. I found that I clicked on "Finish" (german "Abschließen") instead of "Import CSV file" (german "CSV-Datei importieren") due to the expectation that the most usual action would be next to the cancel button.
Git commit 033b747aa5184b29dd846d1573ce87a0cd6bffab by Thomas Baumgart. Committed on 29/07/2018 at 16:58. Pushed by tbaumgart into branch 'master'. Improve CSV importer Summary: - Make the CSV importer to run as a modal dialog of the application as other importers do. This avoids a whole bunch of problems. - Combine the Finish and Import button into a single "Import CSV" button on the last page of the wizard. - In case 'Skip setup' is marked on the first page, the wizard stops on the last page and does not automatically start importing. Test Plan: Tested single file import and web-connect import Reviewers: wojnilowicz Reviewed By: wojnilowicz Differential Revision: https://phabricator.kde.org/D14459 M +1 -1 kmymoney/kmymoney.cpp M +18 -34 kmymoney/plugins/csv/import/csvimporter.cpp M +1 -11 kmymoney/plugins/csv/import/csvimporter.h M +37 -37 kmymoney/plugins/csv/import/csvwizard.cpp M +6 -7 kmymoney/plugins/csv/import/csvwizard.h https://commits.kde.org/kmymoney/033b747aa5184b29dd846d1573ce87a0cd6bffab