Skip to content

Commit

Permalink
[diagrams] Diagram size widget should allow non-integer values
Browse files Browse the repository at this point in the history
There's no reason we should limit diagram sizes to round number of
mm, so adapt widget to follow same behavior as other symbol size widgets.

(cherry picked from commit b2b5450)
  • Loading branch information
nyalldawson committed Nov 23, 2019
1 parent 5ee9b20 commit ad260ff
Showing 1 changed file with 17 additions and 16 deletions.
33 changes: 17 additions & 16 deletions src/ui/qgsdiagrampropertiesbase.ui
Expand Up @@ -1073,11 +1073,14 @@
</sizepolicy>
</property>
<property name="decimals">
<number>5</number>
<number>6</number>
</property>
<property name="maximum">
<double>9999999.990000000223517</double>
</property>
<property name="singleStep">
<double>0.200000000000000</double>
</property>
</widget>
</item>
<item row="0" column="1">
Expand Down Expand Up @@ -1140,16 +1143,6 @@
</property>
</widget>
</item>
<item row="3" column="1" colspan="2">
<widget class="QgsSpinBox" name="mSizeSpinBox">
<property name="maximum">
<number>10000000</number>
</property>
<property name="value">
<number>50</number>
</property>
</widget>
</item>
<item row="3" column="3">
<widget class="QLabel" name="mScaleDependencyLabel">
<property name="text">
Expand Down Expand Up @@ -1267,6 +1260,19 @@
</property>
</widget>
</item>
<item row="3" column="1" colspan="2">
<widget class="QgsDoubleSpinBox" name="mSizeSpinBox">
<property name="decimals">
<number>6</number>
</property>
<property name="maximum">
<double>9999999.000000000000000</double>
</property>
<property name="singleStep">
<double>0.200000000000000</double>
</property>
</widget>
</item>
</layout>
</item>
</layout>
Expand Down Expand Up @@ -2149,11 +2155,6 @@
<header>qgscollapsiblegroupbox.h</header>
<container>1</container>
</customwidget>
<customwidget>
<class>QgsSpinBox</class>
<extends>QSpinBox</extends>
<header>qgsspinbox.h</header>
</customwidget>
<customwidget>
<class>QgsDoubleSpinBox</class>
<extends>QDoubleSpinBox</extends>
Expand Down

0 comments on commit ad260ff

Please sign in to comment.