Allows color form elements to wrap, sets minimum width to thickness s… (

#2068) * Allows color form elements to wrap, sets minimum width to thickness slider. * Uses named constant variable for slider width
flameshot-org · Nov 15, 2021 · f25d58b · f25d58b
1 parent 58e37b0
commit f25d58b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/src/widgets/panel/sidepanelwidget.cpp b/src/widgets/panel/sidepanelwidget.cpp
@@ -31,9 +31,11 @@ SidePanelWidget::SidePanelWidget(QPixmap* p, QWidget* parent)
     }
 
     QFormLayout* colorForm = new QFormLayout();
+    colorForm->setRowWrapPolicy(QFormLayout::WrapLongRows);
     m_toolSizeSlider = new QSlider(Qt::Horizontal);
     m_toolSizeSlider->setRange(1, maxToolSize);
     m_toolSizeSlider->setValue(m_toolSize);
+    m_toolSizeSlider->setMinimumWidth(minSliderWidth);
     m_colorLabel = new QLabel();
     m_colorLabel->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Fixed);
     colorForm->addRow(tr("Active tool size:"), m_toolSizeSlider);

diff --git a/src/widgets/panel/sidepanelwidget.h b/src/widgets/panel/sidepanelwidget.h
@@ -15,6 +15,7 @@ class QColorPickingEventFilter;
 class QSlider;
 
 constexpr int maxToolSize = 50;
+constexpr int minSliderWidth = 100;
 
 class SidePanelWidget : public QWidget
 {