Review Request 128411: Integrate tableWidget into CSVWizard

Łukasz Wojniłowicz lukasz.wojnilowicz at gmail.com
Sat Jul 9 19:19:16 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128411/
-----------------------------------------------------------

Review request for KMymoney.


Repository: kmymoney


Description
-------

Result: [CSVWizard](http://i.imgur.com/ZkvSFce.png)
1) integrated tableWidget into CSVWizard,
2) removed separate window holding tableWidget,
3) modified some .ui files, so tableWidget gets the most space while resizing CSVWizard,
4) removed hardcoded values for HiDPI support; that functionality can be easier achieved with policies in QtDesigner
5) CSVWizar doesn't stay forever after import without setup,
6) tableWidget gets resized to fit content and not exceed screen area,
7) simplified completion page.

I think csvdialog.ui could be deleted as it isn't needed anymore.


Diffs
-----

  kmymoney/plugins/csvimport/bankingwizardpage.ui 95e5276 
  kmymoney/plugins/csvimport/csvdialog.h d226665 
  kmymoney/plugins/csvimport/csvdialog.cpp 4fff108 
  kmymoney/plugins/csvimport/csvimporterplugin.cpp b58ed84 
  kmymoney/plugins/csvimport/csvwizard.h 0bf1e59 
  kmymoney/plugins/csvimport/csvwizard.cpp 39bf9f2 
  kmymoney/plugins/csvimport/csvwizard.ui 5c42f46 
  kmymoney/plugins/csvimport/investmentdlg.cpp 514bff3 
  kmymoney/plugins/csvimport/investprocessing.h 15cba5e 
  kmymoney/plugins/csvimport/investprocessing.cpp e0d2b8f 

Diff: https://git.reviewboard.kde.org/r/128411/diff/


Testing
-------

Banking and investment statement CSV imports; with and without setup.


Thanks,

Łukasz Wojniłowicz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20160709/087bfcd9/attachment.html>


More information about the KMyMoney-devel mailing list