| Summary: | CSV: ordering of buttons when finishing wizard | ||
|---|---|---|---|
| Product: | [Applications] kmymoney | Reporter: | Martin Steigerwald <Martin> |
| Component: | importer | Assignee: | KMyMoney Devel Mailing List <kmymoney-devel> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version First Reported In: | 5.0.1 | ||
| Target Milestone: | --- | ||
| Platform: | Debian unstable | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/kmymoney/033b747aa5184b29dd846d1573ce87a0cd6bffab | Version Fixed/Implemented In: | 5.0.2 |
| Sentry Crash Report: | |||
| Attachments: | screenshot showing the button order when finishing the importer (german text) | ||
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 |
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.