Funkwhale/Clementine-audio-player-Mac-Linux-Windows
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Avoiding usage of pixelMetric because it breaks on non-Oxygen styling

Browse Source 
Partial revertion for commit "3f79fa5c651f23e48faf1af6ca56dbff9e5660ab" because of
https://github.com/clementine-player/Clementine/issues/4770
This commit is contained in:
v-fox 2015-12-13 17:40:17 +05:00
parent 170c64cd8b
commit a9f9b0ef72
2 changed files with 10 additions and 43 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

49
src/moodbar/moodbarproxystyle.cpp
View File

@ -247,13 +247,6 @@ void MoodbarProxyStyle::EnsureMoodbarRendered(const QStyleOptionSlider* opt) {
} }
} }
int MoodbarProxyStyle::GetExtraSpace(const QStyleOptionComplex* opt) const {
int space_available = slider_->style()->pixelMetric(
QStyle::PM_SliderSpaceAvailable, opt, slider_);
int w = slider_->width();
return w - space_available;
}
QRect MoodbarProxyStyle::subControlRect(ComplexControl cc, QRect MoodbarProxyStyle::subControlRect(ComplexControl cc,
const QStyleOptionComplex* opt, const QStyleOptionComplex* opt,
SubControl sc, SubControl sc,
@ -270,28 +263,17 @@ QRect MoodbarProxyStyle::subControlRect(ComplexControl cc,
case MoodbarOn: case MoodbarOn:
case FadingToOn: case FadingToOn:
switch (sc) { switch (sc) {
case SC_SliderGroove: { case SC_SliderGroove:
int margin_leftright = GetExtraSpace(opt) / 2; return opt->rect.adjusted(kMarginSize, kMarginSize, -kMarginSize,
return opt->rect.adjusted(margin_leftright, kMarginSize, -kMarginSize);
-margin_leftright, -kMarginSize);
}
case SC_SliderHandle: { case SC_SliderHandle: {
const QStyleOptionSlider* slider_opt = const QStyleOptionSlider* slider_opt =
qstyleoption_cast<const QStyleOptionSlider*>(opt); qstyleoption_cast<const QStyleOptionSlider*>(opt);
int space_available = slider_->style()->pixelMetric( const int x = (slider_opt->sliderValue - slider_opt->minimum) *
QStyle::PM_SliderSpaceAvailable, opt, slider_); (opt->rect.width() - kArrowWidth) /
int w = slider_->width(); (slider_opt->maximum - slider_opt->minimum);
int margin = (w - space_available) / 2;
int x = 0;
if (slider_opt->maximum != slider_opt->minimum) {
x = (slider_opt->sliderValue - slider_opt->minimum) *
(space_available - kArrowWidth) /
(slider_opt->maximum - slider_opt->minimum);
}
x += margin;
return QRect(QPoint(opt->rect.left() + x, opt->rect.top()), return QRect(QPoint(opt->rect.left() + x, opt->rect.top()),
QSize(kArrowWidth, kArrowHeight)); QSize(kArrowWidth, kArrowHeight));
@ -330,14 +312,9 @@ QPixmap MoodbarProxyStyle::MoodbarPixmap(const ColorVector& colors,
const QSize& size, const QSize& size,
const QPalette& palette, const QPalette& palette,
const QStyleOptionSlider* opt) { const QStyleOptionSlider* opt) {
int margin_leftright = GetExtraSpace(opt); QRect rect(QPoint(0, 0), size);
const QRect rect(QPoint(0, 0), size);
QRect border_rect(rect); QRect border_rect(rect);
// I would expect we need to adjust by margin_lr/2, so the extra space is border_rect.adjust(kMarginSize, kMarginSize, -kMarginSize, -kMarginSize);
// distributed on both side, but if we do so, the margin is too small, and I'm
// not sure why...
border_rect.adjust(margin_leftright, kMarginSize, -margin_leftright,
-kMarginSize);
QRect inner_rect(border_rect); QRect inner_rect(border_rect);
inner_rect.adjust(kBorderSize, kBorderSize, -kBorderSize, -kBorderSize); inner_rect.adjust(kBorderSize, kBorderSize, -kBorderSize, -kBorderSize);
@ -356,14 +333,8 @@ QPixmap MoodbarProxyStyle::MoodbarPixmap(const ColorVector& colors,
// Draw the outer bit // Draw the outer bit
p.setPen(QPen(palette.brush(QPalette::Active, QPalette::Background), p.setPen(QPen(palette.brush(QPalette::Active, QPalette::Background),
kMarginSize, Qt::SolidLine, Qt::FlatCap, Qt::MiterJoin)); kMarginSize, Qt::SolidLine, Qt::FlatCap, Qt::MiterJoin));
// First: a rectangle around the slider
p.drawRect(rect.adjusted(1, 1, -2, -2)); p.drawRect(rect.adjusted(1, 1, -2, -2));
// Then, thicker border on left and right, because of the margins.
p.setPen(QPen(palette.brush(QPalette::Active, QPalette::Background),
margin_leftright * 2 - kBorderSize, Qt::SolidLine, Qt::FlatCap,
Qt::MiterJoin));
p.drawLine(rect.topLeft(), rect.bottomLeft());
p.drawLine(rect.topRight(), rect.bottomRight());
p.end(); p.end();

4
src/moodbar/moodbarproxystyle.h
View File

@ -79,10 +79,6 @@ class MoodbarProxyStyle : public QProxyStyle {
void ChangeStyle(QAction* action); void ChangeStyle(QAction* action);
private: private:
// The slider "groove" is smaller than the actual slider: this convenient
// function returns the difference between groove width and slider width.
int GetExtraSpace(const QStyleOptionComplex* opt) const;
Application* app_; Application* app_;
QSlider* slider_; QSlider* slider_;