Apply suggestions from code review
Co-authored-by: Benjamin Bræstrup Sayoc <benj5378@outlook.com>
This commit is contained in:
@@ -43,9 +43,9 @@ QSize ActionLabel::sizeHint() const
|
||||
QStyleOptionToolButton opt;
|
||||
initStyleOption(&opt);
|
||||
|
||||
QString s = text().isEmpty() ? QString::fromLatin1("XXXX") : text();
|
||||
QString s = text().isEmpty() ? QStringLiteral("XXXX") : text();
|
||||
QSize textSize = fontMetrics().size(Qt::TextShowMnemonic, s);
|
||||
const int padding = 10;
|
||||
constexpr int padding = 10;
|
||||
|
||||
int width = textSize.width();
|
||||
if (!icon().isNull()) {
|
||||
|
||||
@@ -12,7 +12,7 @@ namespace QSint
|
||||
{
|
||||
|
||||
|
||||
const QString ActionPanelScheme::minimumStyle = QString::fromLatin1(
|
||||
constexpr QString ActionPanelScheme::minimumStyle = QStringLiteral(
|
||||
"QSint--ActionGroup QFrame[class='header'] {"
|
||||
"border: none;"
|
||||
"}"
|
||||
@@ -49,7 +49,7 @@ QString ActionPanelScheme::systemStyle(const QPalette& p)
|
||||
replacements["headerBackground"] = headerBackground.name();
|
||||
replacements["groupBackground"] = groupBackground.name();;
|
||||
|
||||
QString style = QString::fromLatin1(
|
||||
QString style = QStringLiteral(
|
||||
"QSint--ActionGroup QFrame[class='header'] {"
|
||||
"background-color: {headerBackground};"
|
||||
"}"
|
||||
|
||||
Reference in New Issue
Block a user