Add missing 2 * faremWidth() also to WEffectParameterNameBase's m_wid…

…thHint
mixxxdj · Feb 4, 2023 · 9ca1deb · 9ca1deb
1 parent 29c3bbc
commit 9ca1deb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/widget/weffectparameternamebase.cpp b/src/widget/weffectparameternamebase.cpp
@@ -79,9 +79,12 @@ void WEffectParameterNameBase::parameterUpdated() {
         m_text = kNoEffectString;
         setBaseTooltip(tr("No effect loaded."));
     }
+    // frameWidth() is the maximum of the sum of margin, border and padding
+    // width of the left and the right side.
     m_widthHint = math_max(
-            valueWidth,
-            metrics.size(0, m_text).width());
+                          valueWidth,
+                          metrics.size(0, m_text).width()) +
+            2 * frameWidth();
     setText(m_text);
     m_parameterUpdated = true;
 }

diff --git a/src/widget/wlabel.cpp b/src/widget/wlabel.cpp
@@ -95,7 +95,7 @@ void WLabel::setText(const QString& text) {
     m_longText = text;
     if (m_elideMode != Qt::ElideNone) {
         QFontMetrics metrics(font());
-        // Measure the text for label width
+        // Measure the text for the optimum label width
         // frameWidth() is the maximum of the sum of margin, border and padding
         // width of the left and the right side.
         m_widthHint = metrics.size(0, m_longText).width() + 2 * frameWidth();