Review Request 117937: Remove hardcoded margins from Krita filter dialog, use normal buttonbox for ok/cancel

Friedrich W. H. Kossebau kossebau at kde.org
Thu May 1 23:36:05 BST 2014


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

(Updated May 1, 2014, 10:36 p.m.)


Review request for Calligra and Boudewijn Rempt.


Changes
-------

fix wrong before screenshot


Repository: calligra


Description
-------

The Ok & Cancel buttons in the Krita filter dialog are not following the standards, also will not adapt to different Cancel/Ok order on other platforms (e.g. Gnome OS). Also are some margins hardcoded to certain random pixel sizes, which is a problem on HiDPI screens and with styles where those margins do not match the built-in ones.

These two patches remove some hard-coded margins, use a QButtonBox for the Ok and Cancel buttons and also some ";" to labels where they are missing.


Diffs
-----

  krita/plugins/filters/blur/wdg_motion_blur.ui 5377ccf 
  krita/plugins/filters/blur/wdg_lens_blur.ui e7daea4 
  krita/plugins/filters/blur/wdg_gaussian_blur.ui 728f608 
  krita/plugins/filters/colorsfilters/kis_hsv_adjustment_filter.cpp 85f476c 
  krita/ui/dialogs/kis_dlg_filter.cpp 4fd86ca 
  krita/ui/forms/wdgfilterdialog.ui 7a1e3c6 
  krita/ui/forms/wdgfilterselector.ui c9d87c7 

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


Testing
-------


File Attachments (updated)
----------------

After, by example of levels filter
  https://git.reviewboard.kde.org/media/uploaded/files/2014/05/01/cbb4ea9c-90a8-4b0d-90e0-febdca546f98__filterdialogafter.png
Before, by example of levels filter
  https://git.reviewboard.kde.org/media/uploaded/files/2014/05/01/9dd88e38-8a3f-4ab5-aced-ecad20d887db__filterdialogbefore.png


Thanks,

Friedrich W. H. Kossebau

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20140501/d6e1e43b/attachment.htm>


More information about the calligra-devel mailing list