From 477997cf61edb593f0286f79dd1ec3af0c3cac53 Mon Sep 17 00:00:00 2001 From: ronso0 Date: Fri, 10 Apr 2020 16:37:25 +0200 Subject: [PATCH 1/2] WidgetGroup: add default index0 widget to stacked layout Fixes https://bugs.launchpad.net/mixxx/+bug/1627859 Previously, the first widget in a stacked layout was always visible after skin load regardless of the 'visible' CO value. Adding a zero-sized dummy widget at index 0 here avoids having to do that in skin xml files. --- src/widget/wwidgetgroup.cpp | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/widget/wwidgetgroup.cpp b/src/widget/wwidgetgroup.cpp index 3fa1d2cae4fe..f83b8f5b0b70 100644 --- a/src/widget/wwidgetgroup.cpp +++ b/src/widget/wwidgetgroup.cpp @@ -128,6 +128,12 @@ void WWidgetGroup::setup(const QDomNode& node, const SkinContext& context) { auto pStackedLayout = new QStackedLayout(); pStackedLayout->setStackingMode(QStackedLayout::StackAll); pLayout = pStackedLayout; + // Adding a zero-size dummy widget as index 0 here before + // any child is added in the xml template works around + // https://bugs.launchpad.net/mixxx/+bug/1627859 + QWidget *dummyWidget = new QWidget(); + dummyWidget->setFixedSize(0, 0); + pLayout->addWidget(dummyWidget); } // Set common layout parameters. From 15dcec9f793b47996cd715b8bfa2831818e2c3df Mon Sep 17 00:00:00 2001 From: ronso0 Date: Fri, 10 Apr 2020 17:44:18 +0200 Subject: [PATCH 2/2] remove stacked layout workarounds from skins --- res/skins/LateNight/skin_settings.xml | 21 --------- .../Tango (64 Samplers)/skin_settings.xml | 43 ------------------- res/skins/Tango/aux_unit.xml | 4 -- res/skins/Tango/deck_row_overview_right.xml | 2 +- res/skins/Tango/fx_unit_left_mini.xml | 4 -- res/skins/Tango/fx_unit_right_mini.xml | 4 -- res/skins/Tango/mic_aux_rack.xml | 36 ++++------------ res/skins/Tango/mic_unit.xml | 4 -- res/skins/Tango/mixer_channel_left.xml | 4 -- res/skins/Tango/mixer_channel_right.xml | 4 -- res/skins/Tango/mixer_headphone.xml | 13 +----- res/skins/Tango/preview_deck.xml | 3 -- res/skins/Tango/sampler.xml | 4 -- res/skins/Tango/skin_settings.xml | 30 ------------- res/skins/Tango/vinyl_controls_left.xml | 11 ++--- res/skins/Tango/vinyl_controls_right.xml | 11 ++--- 16 files changed, 17 insertions(+), 181 deletions(-) diff --git a/res/skins/LateNight/skin_settings.xml b/res/skins/LateNight/skin_settings.xml index e35cc66b05bc..9cd9686d7266 100644 --- a/res/skins/LateNight/skin_settings.xml +++ b/res/skins/LateNight/skin_settings.xml @@ -85,8 +85,6 @@ Description: stacked 180min,17f - - 0f,0f SubmenuCover stacked @@ -131,8 +129,6 @@ Description: stacked 190f,17f - - 0f,0f SubmenuCover vertical @@ -179,9 +175,6 @@ Description: stacked 180min,17f - - 0f,0f - SubmenuCover vertical @@ -205,9 +198,6 @@ Description: stacked 180min,17f - - 0f,0f - SubmenuCover vertical @@ -231,9 +221,6 @@ Description: stacked 180min,17f - - 0f,0f - SubmenuCover vertical @@ -277,9 +264,6 @@ Description: stacked 180min,17f - - 0f,0f - SubmenuCover stacked @@ -302,9 +286,6 @@ Description: stacked 180min,17f - - 0f,0f - SubmenuCover stacked @@ -348,8 +329,6 @@ Description: stacked 180min,17f - - 0f,0f SubmenuCover stacked diff --git a/res/skins/Tango (64 Samplers)/skin_settings.xml b/res/skins/Tango (64 Samplers)/skin_settings.xml index 8d4a80e19ee4..091fdea29a80 100644 --- a/res/skins/Tango (64 Samplers)/skin_settings.xml +++ b/res/skins/Tango (64 Samplers)/skin_settings.xml @@ -92,9 +92,6 @@ Description: me,me stacked - - 0f,0f - SubmenuCover vertical @@ -105,7 +102,6 @@ Description: visible - 182,13 182,20 @@ -183,9 +179,6 @@ Description: stacked 16f,15f - - 0f,0f - SubmenuCover @@ -197,7 +190,6 @@ Description: visible - - - stacked 182,13 182,20 me,me - - 0f,0f - SubmenuCover @@ -578,12 +554,10 @@ Description: visible - - @@ -604,9 +578,6 @@ Description: 182,20 me,me - - 0f,0f - SubmenuCover @@ -619,7 +590,6 @@ Description: visible -