Review Request 127591: Porting KDialog to QDialog

Garvit Khatri garvitdelhi at gmail.com
Wed Apr 6 18:56:27 UTC 2016



> On April 6, 2016, 5:45 p.m., Stefan Gerlach wrote:
> > the layout is broken for me in FileInfoDialog, SortDialog and RandomValuesDialog after this patch. Can you check it?

Hey, my mistake I didn't checked the patch thouroughly I have updated anu with some mistaked she has pushed the fixes and it should be working now. Applogies, will be carefull next time before shipping it :)


- Garvit


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


On April 6, 2016, 4:40 p.m., Anu Mittal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127591/
> -----------------------------------------------------------
> 
> (Updated April 6, 2016, 4:40 p.m.)
> 
> 
> Review request for KDE Edu and Alexander Semke.
> 
> 
> Repository: labplot
> 
> 
> Description
> -------
> 
> Removing obsolete functions and porting to Qt5 (KDialog to QDialog)
> 
> 
> Diffs
> -----
> 
>   src/kdefrontend/datasources/FileInfoDialog.h 111d1ba 
>   src/kdefrontend/datasources/FileInfoDialog.cpp 69246d3 
>   src/kdefrontend/datasources/ImportFileDialog.h 2d0f31c 
>   src/kdefrontend/datasources/ImportFileDialog.cpp 395906b 
>   src/kdefrontend/datasources/ImportFileWidget.cpp 29ada09 
>   src/kdefrontend/spreadsheet/RandomValuesDialog.h 3f0f1f6 
>   src/kdefrontend/spreadsheet/RandomValuesDialog.cpp 712ea60 
>   src/kdefrontend/spreadsheet/SortDialog.h 9b519d3 
>   src/kdefrontend/spreadsheet/SortDialog.cpp d97b2c2 
>   src/kdefrontend/worksheet/ExportWorksheetDialog.cpp e24b87b 
> 
> Diff: https://git.reviewboard.kde.org/r/127591/diff/
> 
> 
> Testing
> -------
> 
> Successfully built.
> 
> 
> Thanks,
> 
> Anu Mittal
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20160406/19c487e9/attachment.html>


More information about the kde-edu mailing list