Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request #10645 from shuffle2/qt-hack
qt/win: remove hack for tab bar scroll buttons
  • Loading branch information
Tilka committed May 9, 2022
2 parents b339c6c + 5d45938 commit 866a8da
Showing 1 changed file with 0 additions and 16 deletions.
16 changes: 0 additions & 16 deletions Source/Core/DolphinQt/Settings.cpp
Expand Up @@ -155,22 +155,6 @@ void Settings::SetCurrentUserStyle(const QString& stylesheet_name)
.arg(border_color.rgba(), 0, 16);
stylesheet_contents.append(QStringLiteral("%1").arg(tooltip_stylesheet));
}
#ifdef _WIN32
// MSVC has a bug causing QTabBar scroll buttons to be partially transparent when they inherit any
// stylesheet (see https://bugreports.qt.io/browse/QTBUG-74187) which is triggered when setting
// qApp's stylesheet below. Setting the scroll buttons' color directly fixes the problem.

// Create a temporary QToolButton that's a child of a QTabBar in case that has different styling
// than a plain QToolButton.
const auto tab_bar = std::make_unique<QTabBar>();
auto* const tool_button = new QToolButton(tab_bar.get());

const QRgb background_color = tool_button->palette().color(QPalette::Button).rgba();

const std::string style_var =
fmt::format("QTabBar QToolButton {{ background-color: #{:08x}; }}", background_color);
stylesheet_contents.append(QString::fromStdString(style_var));
#endif

qApp->setStyleSheet(stylesheet_contents);

Expand Down

0 comments on commit 866a8da

Please sign in to comment.