Review Request 109897: Remove any hardcoding of spacing to 6 px and margins to 9/11 px in Krita's UI
Commit Hook
null at kde.org
Tue Apr 9 10:44:06 BST 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109897/
-----------------------------------------------------------
(Updated April 9, 2013, 9:44 a.m.)
Status
------
This change has been marked as submitted.
Review request for Calligra, Sven Langkamp and Boudewijn Rempt.
Description
-------
This seems a left-over from KDE3/Qt3 times, spacings and margins should be completely controlled by the style IMHO. Only exceptions should be setting the values to 0 to not have any margins/spacings, or if there is some pixel-perfect layouting done.
Diffs
-----
krita/plugins/extensions/colorrange/wdg_colorrange.ui 37e64c3
krita/plugins/extensions/colorspaceconversion/wdgconvertcolorspace.ui b31861b
krita/plugins/extensions/compose/wdg_compose.ui 91a1788
krita/plugins/extensions/dockers/defaultdockers/wdglayerbox.ui ecfd983
krita/plugins/extensions/dropshadow/wdg_dropshadow.ui f03818a
krita/plugins/extensions/histogram/wdghistogram.ui f80c3c1
krita/plugins/extensions/imagesize/wdg_canvassize.ui 056b545
krita/plugins/extensions/imagesize/wdg_imagesize.ui 2c0a2ab
krita/plugins/extensions/imagesize/wdg_layersize.ui 94e71a2
krita/plugins/extensions/imagesplit/wdg_imagesplit.ui 601f5ab
krita/plugins/extensions/modify_selection/wdg_border_selection.ui 73d983b
krita/plugins/extensions/modify_selection/wdg_feather_selection.ui af505ab
krita/plugins/extensions/modify_selection/wdg_grow_selection.ui 8b3b8c4
krita/plugins/extensions/modify_selection/wdg_shrink_selection.ui 234aab5
krita/plugins/extensions/offsetimage/wdg_offsetimage.ui 2ab32ff
krita/plugins/extensions/rotateimage/wdg_rotateimage.ui e721842
krita/plugins/extensions/separate_channels/wdg_separations.ui 5ed2c89
krita/plugins/extensions/shearimage/wdg_shearimage.ui 4bd9c4d
krita/plugins/filters/blur/wdg_gaussian_blur.ui dc1e594
krita/plugins/filters/blur/wdg_lens_blur.ui 7993830
krita/plugins/filters/blur/wdg_motion_blur.ui 729bc4f
krita/plugins/filters/blur/wdgblur.ui 4b7535c
krita/plugins/filters/colors/wdgcolortoalphabase.ui 2dcc3d4
krita/plugins/filters/colorsfilters/wdg_brightness_contrast.ui f462b7b
krita/plugins/filters/colorsfilters/wdg_perchannel.ui a8872b6
krita/plugins/filters/dodgeburn/DodgeBurnConfigurationBaseWidget.ui 975e1e7
krita/plugins/filters/fastcolortransfer/wdgfastcolortransfer.ui c97eb5b
krita/plugins/filters/levelfilter/wdg_level.ui cda6ffc
krita/plugins/filters/noisefilter/wdgnoiseoptions.ui cdf8b07
krita/plugins/filters/randompickfilter/wdgrandompickoptions.ui 5800237
krita/plugins/filters/unsharp/wdgunsharp.ui 152ef9f
krita/plugins/filters/wavefilter/wdgwaveoptions.ui c70d62e
krita/plugins/formats/pdf/pdfimportwidgetbase.ui 506fd49
krita/plugins/formats/png/kis_wdg_options_png.ui 85f6f57
krita/plugins/formats/raw/wdgrawimport.ui 6a78272
krita/plugins/formats/tiff/kis_wdg_options_tiff.ui f1f7389
krita/plugins/generators/solid/wdgcoloroptions.ui 936def0
krita/plugins/paintops/libpaintop/forms/wdgcustombrush.ui cd6eb34
krita/plugins/tools/defaulttools/kis_tool_measure.cc 223f317
krita/plugins/tools/defaulttools/wdgcolorpicker.ui 7d1c711
krita/plugins/tools/tool_crop/wdg_tool_crop.ui 08bc2ca
krita/plugins/tools/tool_text/wdgtextoptions.ui 5560fc0
krita/ui/dialogs/kis_dlg_adj_layer_props.cc 70a389c
krita/ui/forms/kis_matrix_widget.ui 63c1e97
krita/ui/forms/kis_previewwidgetbase.ui 2faa678
krita/ui/forms/wdg_resolution.ui 1108a6e
krita/ui/forms/wdgapplyprofile.ui 7dd5d71
krita/ui/forms/wdgautogradient.ui 2b157b7
krita/ui/forms/wdgbookmarkedconfigurationseditor.ui 0054dbf
krita/ui/forms/wdgcolorsettings.ui cee3691
krita/ui/forms/wdgcustompalette.ui 65f4ea3
krita/ui/forms/wdgdisplaysettings.ui 8ac25c7
krita/ui/forms/wdgdlgblacklistcleanup.ui c99ccc0
krita/ui/forms/wdgflipbookselector.ui dca0ab7
krita/ui/forms/wdggeneralsettings.ui f4e80c0
krita/ui/forms/wdggridsettings.ui eab3c57
krita/ui/forms/wdgimageproperties.ui 4f1e2a9
krita/ui/forms/wdglayerproperties.ui d952ab7
krita/ui/forms/wdgnewimage.ui a9f1a27
krita/ui/forms/wdgperformancesettings.ui cca6e5f
krita/ui/forms/wdgselectionoptions.ui 19fc539
krita/ui/forms/wdgshapeoptions.ui 65c0e9c
krita/ui/forms/wdgtabletsettings.ui dd0cde5
Diff: http://git.reviewboard.kde.org/r/109897/diff/
Testing
-------
Thanks,
Friedrich W. H. Kossebau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130409/9c790fe4/attachment.htm>
More information about the calligra-devel
mailing list