Zren / material-decoration

Material-ish window decoration theme for KWin, with LIM, based on zzag's original design.
GNU General Public License v2.0
200 stars 17 forks source link

menu items too close to each other need some space to breath #52

Closed Aneeqasif closed 3 years ago

Aneeqasif commented 3 years ago

check out this screenshot

Zren commented 3 years ago

This is asking to change the default, but it's basically a duplicate of Issue #19.

Aneeqasif commented 3 years ago

This is asking to change the default, but it's basically a duplicate of Issue #19.

i am a noob user , its asking to edit a line can u please tell how to edit or which term should i add or remove in that line to have some space in between menu items

edit: so i multiplied it with 4, one thing that i just wanna ask , where can i change the close button hovered color , i wanna make it bright red as in pressed form. plz guid me a lil

Zren commented 3 years ago

It's here: https://github.com/Zren/material-decoration/blob/master/src/Button.cc#L323

You can create a RGB color with: = QColor(255, 255, 255); if you want a specific color.

Aneeqasif commented 3 years ago

It's here: https://github.com/Zren/material-decoration/blob/master/src/Button.cc#L323

You can create a RGB color with: = QColor(255, 255, 255); if you want a specific color.

` if (type() == KDecoration2::DecorationButtonType::Close) { auto *decoratedClient = deco->client().toStrongRef().data(); const QColor hoveredColor = QColor(255, 255, 255);

    QColor normalColor = QColor(hoveredColor);
    normalColor.setAlphaF(0);

    if (isPressed()) {
        const QColor pressedColor = decoratedClient->color(
            KDecoration2::ColorGroup::Warning,
            KDecoration2::ColorRole::Foreground
        ).lighter();
        return KColorUtils::mix(normalColor, pressedColor, m_transitionValue);
    }

    if (isHovered()) {
        return KColorUtils::mix(normalColor, hoveredColor, m_transitionValue);
    }
}`

like this?