Bug 396759

Summary: CSV: ordering of buttons when finishing wizard
Product: [Applications] kmymoney Reporter: Martin Steigerwald <Martin>
Component: importerAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: 5.0.1   
Target Milestone: ---   
Platform: Debian unstable   
OS: Linux   
Latest Commit: Version Fixed In: 5.0.2
Sentry Crash Report:
Attachments: screenshot showing the button order when finishing the importer (german text)

Description Martin Steigerwald 2018-07-22 16:49:34 UTC
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.
Comment 1 Thomas Baumgart 2018-07-29 16:58:05 UTC
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