diff --git a/frame/window/components/brightnessadjwidget.cpp b/frame/window/components/brightnessadjwidget.cpp index 3ae544459..1b339a977 100644 --- a/frame/window/components/brightnessadjwidget.cpp +++ b/frame/window/components/brightnessadjwidget.cpp @@ -25,18 +25,24 @@ #include +const int ItemSpacing = 5; + BrightnessAdjWidget::BrightnessAdjWidget(QWidget *parent) : QWidget(parent) , m_mainLayout(new QVBoxLayout(this)) , m_brightnessModel(new BrightnessModel(this)) { - m_mainLayout->setSpacing(5); + m_mainLayout->setMargin(0); + m_mainLayout->setSpacing(ItemSpacing); + loadBrightnessItem(); } void BrightnessAdjWidget::loadBrightnessItem() { QList monitors = m_brightnessModel->monitors(); + int itemHeight = monitors.count() > 1 ? 56 : 30; + for (BrightMonitor *monitor : monitors) { SliderContainer *sliderContainer = new SliderContainer(this); if (monitors.count() > 1) @@ -48,7 +54,7 @@ void BrightnessAdjWidget::loadBrightnessItem() sliderContainer->setIcon(SliderContainer::IconPosition::RightIcon, rightPixmap, QSize(), 12); sliderContainer->setFixedWidth(310); - sliderContainer->setFixedHeight(monitors.count() > 1 ? 56 : 30); + sliderContainer->setFixedHeight(itemHeight); sliderContainer->updateSliderValue(monitor->brightness()); SliderProxyStyle *proxy = new SliderProxyStyle(SliderProxyStyle::Normal); @@ -58,5 +64,8 @@ void BrightnessAdjWidget::loadBrightnessItem() connect(monitor, &BrightMonitor::brightnessChanged, sliderContainer, &SliderContainer::updateSliderValue); connect(sliderContainer, &SliderContainer::sliderValueChanged, monitor, &BrightMonitor::setBrightness); } + + QMargins margins = this->contentsMargins(); + setFixedHeight(margins.top() + margins.bottom() + monitors.count() * itemHeight + monitors.count() * ItemSpacing); } diff --git a/frame/window/components/devitemdelegate.cpp b/frame/window/components/devitemdelegate.cpp index 876cde1cf..73e50d222 100644 --- a/frame/window/components/devitemdelegate.cpp +++ b/frame/window/components/devitemdelegate.cpp @@ -27,7 +27,7 @@ #include #define RADIUS_VALUE 10 -#define ITEM_SPACE 10 +#define ITEM_SPACE 20 #define ICON_WIDTH 16 #define ICON_HEIGHT 16 #define TEXT_RECT_HEIGHT 20 diff --git a/frame/window/components/displaysettingwidget.cpp b/frame/window/components/displaysettingwidget.cpp index 510108faa..6ac92c601 100644 --- a/frame/window/components/displaysettingwidget.cpp +++ b/frame/window/components/displaysettingwidget.cpp @@ -27,11 +27,13 @@ #include +const int ItemSpacing = 10; + DisplaySettingWidget::DisplaySettingWidget(QWidget *parent) : QWidget(parent) , m_brightnessAdjWidget(new BrightnessAdjWidget(this)) , m_collaborationWidget(new DevCollaborationWidget(this)) - , m_settingBtn(new QPushButton(this)) + , m_settingBtn(new QPushButton(tr("Display setting"), this)) { initUI(); @@ -46,8 +48,10 @@ DisplaySettingWidget::DisplaySettingWidget(QWidget *parent) void DisplaySettingWidget::initUI() { + setContentsMargins(0, 10, 0, 30); QVBoxLayout *mainLayout = new QVBoxLayout(); mainLayout->setMargin(0); + mainLayout->setSpacing(ItemSpacing); mainLayout->addWidget(m_brightnessAdjWidget); mainLayout->addWidget(m_collaborationWidget); @@ -55,4 +59,8 @@ void DisplaySettingWidget::initUI() mainLayout->addStretch(); setLayout(mainLayout); + + QMargins margins = this->contentsMargins(); + setFixedHeight(margins.top() + margins.bottom() + m_brightnessAdjWidget->height() + + m_collaborationWidget->height() + m_settingBtn->height() + ItemSpacing * 2); } diff --git a/frame/window/quicksettingcontainer.cpp b/frame/window/quicksettingcontainer.cpp index bf318e80f..83549ea80 100644 --- a/frame/window/quicksettingcontainer.cpp +++ b/frame/window/quicksettingcontainer.cpp @@ -327,6 +327,8 @@ void QuickSettingContainer::initUi() resizeView(); setFixedWidth(ITEMWIDTH * 4 + (ITEMSPACE * 5)); }, Qt::QueuedConnection); + + m_displaySettingWidget->setVisible(false); } void QuickSettingContainer::initConnection()