https://bugs.kde.org/show_bug.cgi?id=519761

--- Comment #9 from Sebastian Sauer <[email protected]> ---
For Aurora themes: Seems there never was a "Small" entry unlike e.g. for
Breeze.

See
https://invent.kde.org/plasma/kdecoration/-/blob/master/src/decorationdefines.h?ref_type=heads#L90

```
enum class BorderSize {
    /**
     * Border sizes of all non title bar sides should be set to 0.
     **/
    None,
    /**
     * Border sizes of the sides should be set to 0. Title bar and
     * the border on opposite side of the title bar should follow the
     * Normal settings.
     **/
    NoSides,
    /**
     * Borders should be smaller than Normal, e.g. a factor of 0.5.
     **/
    Tiny,
    /**
     * The default border size with borders on each side. This should
     * be the base for calculating other border sizes.
     **/
    Normal,
    /**
     * Increased border sizes, considered a factor of 1.5.
     **/
    Large,
    /**
     * Increased border sizes, considered a factor of 2.0.
     **/
    VeryLarge,
    /**
     * Increased border sizes, considered a factor of 2.5.
     **/
    Huge,
    /**
     * Increased border sizes, considered a factor of 3.0.
     **/
    VeryHuge,
    /**
     * Increased border sizes, considered a factor of 5.0.
     **/
    Oversized,
};
```

and it was like that in 2014 already when it was introduced in git commit
https://invent.kde.org/plasma/kdecoration/-/commit/b47ed6557d38ada81330dc99ef4edc9cda39bd54

The buttonFactor calculation also equals between Aurora v1 and v2. But in v1
the overall buttonHeight() seems to be slightly different:
https://invent.kde.org/plasma/aurorae/-/blob/master/v1/lib/auroraetheme.cpp?ref_type=heads#L171

The ideal solution would be of course to have more steps between "Tiny" and
"Normal". Unfortunately this map to the int values 0 and 1 and we probably
cannot just
introduce easily a new value as long as we also need to be compatible to v1.

imho:
* The missing "small" step is not a regression. But a good wishlist item.
* That "Normal" vs "Tiny" are maybe a bit bigger in v2 compared to v1 could be
seen as a bug/regression.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to