<div dir="ltr"><div><div><div><div>Hi, Scott!<br><br></div>I'm not sure this very change looks ok (probably, it might be fixed somehow without reverting the patch, I don't know).<br><br></div>Before the change the combo box had a fixed width. That is whatever option was selected it kept the same look, and therefore looked more highlevel option that the rest of the list<br><br></div>After your patch the width of the dropdown box is no longer fixed and it changes its size when the user clicks on it (to select another type of smoothing). In my opinion such jumping-under-cursor controls look quite ugly. I feel that we should still show somehow that the type of smoothing is more highlevel option.<br><br></div><div><div><div><div><br></div></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 5, 2015 at 6:52 AM, Scott Petrovic <span dir="ltr"><<a href="mailto:scottpetrovic@gmail.com" target="_blank">scottpetrovic@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Git commit fb929a857ceb75bfcf96543b935749d4e3ac0877 by Scott Petrovic.<br>
Committed on 05/02/2015 at 03:52.<br>
Pushed by scottpetrovic into branch 'calligra/2.9'.<br>
<br>
add label to smoothing property so it is more consistent with other options<br>
<br>
M +7 -4 krita/plugins/tools/defaulttools/kis_tool_brush.cc<br>
M +2 -0 krita/plugins/tools/defaulttools/kis_tool_brush.h<br>
<br>
<a href="http://commits.kde.org/calligra/fb929a857ceb75bfcf96543b935749d4e3ac0877" target="_blank">http://commits.kde.org/calligra/fb929a857ceb75bfcf96543b935749d4e3ac0877</a><br>
<br>
diff --git a/krita/plugins/tools/defaulttools/kis_tool_brush.cc b/krita/plugins/tools/defaulttools/kis_tool_brush.cc<br>
index 40dfd14..8db9ba0 100644<br>
--- a/krita/plugins/tools/defaulttools/kis_tool_brush.cc<br>
+++ b/krita/plugins/tools/defaulttools/kis_tool_brush.cc<br>
@@ -318,14 +318,17 @@ QWidget * KisToolBrush::createOptionWidget()<br>
optionsWidget->layout()->addWidget(specialSpacer);<br>
<br>
// Line smoothing configuration<br>
+ m_smoothingLabel = new QLabel(optionsWidget);<br>
+ m_smoothingLabel->setText(i18n("Smoothing:"));<br>
+<br>
m_cmbSmoothingType = new QComboBox(optionsWidget);<br>
m_cmbSmoothingType->addItems(QStringList()<br>
- << i18n("No Smoothing")<br>
- << i18n("Basic Smoothing")<br>
- << i18n("Weighted Smoothing")<br>
+ << i18n("None")<br>
+ << i18n("Basic")<br>
+ << i18n("Weighted")<br>
<< i18n("Stabilizer"));<br>
connect(m_cmbSmoothingType, SIGNAL(currentIndexChanged(int)), this, SLOT(slotSetSmoothingType(int)));<br>
- addOptionWidgetOption(m_cmbSmoothingType);<br>
+ addOptionWidgetOption(m_cmbSmoothingType, m_smoothingLabel);<br>
<br>
m_sliderSmoothnessDistance = new KisDoubleSliderSpinBox(optionsWidget);<br>
m_sliderSmoothnessDistance->setRange(3.0, MAXIMUM_SMOOTHNESS_DISTANCE, 1);<br>
diff --git a/krita/plugins/tools/defaulttools/kis_tool_brush.h b/krita/plugins/tools/defaulttools/kis_tool_brush.h<br>
index 161b248..08419a1 100644<br>
--- a/krita/plugins/tools/defaulttools/kis_tool_brush.h<br>
+++ b/krita/plugins/tools/defaulttools/kis_tool_brush.h<br>
@@ -34,6 +34,7 @@<br>
<br>
class QCheckBox;<br>
class QComboBox;<br>
+class QLabel;<br>
class QGridLayout;<br>
<br>
class KoCanvasBase;<br>
@@ -115,6 +116,7 @@ private:<br>
void addSmoothingAction(int enumId, const QString &id, const QString &name, KActionCollection *globalCollection);<br>
<br>
private:<br>
+ QLabel *m_smoothingLabel;<br>
QComboBox *m_cmbSmoothingType;<br>
<br>
QCheckBox *m_chkAssistant;<br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">Dmitry Kazakov</div>
</div>