Bug 465948 - Option to disable window border introduced in 5.27
Summary: Option to disable window border introduced in 5.27
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: window decoration (show other bugs)
Version: 5.27.0
Platform: Arch Linux Linux
: NOR wishlist
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 466196 467500 (view as bug list)
Depends on: 455248
Blocks:
  Show dependency treegraph
 
Reported: 2023-02-17 16:48 UTC by ricevacuum
Modified: 2024-07-30 20:48 UTC (History)
20 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0
Sentry Crash Report:


Attachments
window borders 5.26.5 vs 5.27.0 (219.79 KB, image/png)
2023-02-17 16:48 UTC, ricevacuum
Details
Image of new border "eating" into dark window contents (173.47 KB, image/png)
2023-02-17 18:39 UTC, Synthetic451
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ricevacuum 2023-02-17 16:48:25 UTC
Created attachment 156382 [details]
window borders 5.26.5 vs 5.27.0

Borders look ugly on my colorscheme now, see attached picture.

I propose a toggle setting to bring back old (5.26.5) behaviour. 

I just don't get it why we need visible borders on "no borders" setting.
Comment 1 subhuman2222 2023-02-17 18:30:19 UTC Comment hidden (spam)
Comment 2 subhuman2222 2023-02-17 18:30:50 UTC Comment hidden (spam)
Comment 3 Synthetic451 2023-02-17 18:39:48 UTC
Created attachment 156385 [details]
Image of new border "eating" into dark window contents

Agreed. And it seems like a lot of other people are upset by this as well: https://www.reddit.com/r/kde/comments/113y9py/how_to_get_rid_of_the_white_border_around_window/

There's 3 problems with the new border:

1. It is really distracting with windows with dark contents or if dark theme is enabled.
2. People are confused about why it doesn't respect the "No borders" setting.
3. The border appears on the secondary monitor when maximizing a window on the primary.

The most glaring issue for me personally is that the color choices make it seem like it "eats" into dark window contents even when it's not. I'm attaching a screenshot showing the behavior.

Both the Mullvad and Konsole windows used to look flush with the window edge, and now the dark parts of the window contents look like they have an extra 1px padding. I know its part of the window frame, but the color choices make it so that it looks like it's actually chewing into the window contents.

It's especially obvious at the meeting point between light and dark window contents (the corner area where the Konsole toolbars meet the terminal contents and the corner area where the green Mullvad toolbar meets the map).

This is genuinely a very distracting change and I would very much like an option to disable this.
Comment 4 Christian (Fuchs) 2023-02-17 19:13:09 UTC
Agreed on offering an option or a customizable colour (including "none"), since it unfortunately doesn't look that good on some colour combinations, including mine  (dark blue)
Comment 5 Krystian 2023-02-17 20:31:58 UTC Comment hidden (spam)
Comment 6 Marc 2023-02-17 23:35:03 UTC Comment hidden (spam)
Comment 8 Nate Graham 2023-02-21 22:50:46 UTC
*** Bug 466196 has been marked as a duplicate of this bug. ***
Comment 9 Sanras 2023-02-22 16:28:49 UTC Comment hidden (spam)
Comment 10 Blazer Silving 2023-02-22 18:37:50 UTC
To me, the reason this border is so distracting is because it does not match the global/custom color schema. For instance, with an accent value of #007e7e, the border renders at #14a3a3. I'm guessing this was intentional due to this being intended as an accessibility feature. 

Absolutely should be an opt-in setting for those that desire a bright border, not the default. The "No Borders" window decoration setting is misleading in this state, I personally do not desire borders for any windows, period. The code has a routine to check for the No Borders and
Comment 11 Blazer Silving 2023-02-22 18:40:58 UTC
Sorry, hit enter by accident and submitted. 

The code checks for "no borders" and "no side-borders" to choose where to draw the outline, could that check be used to completely hide the border rather than adding "yet-another-checkbox"?
Comment 12 Christian (Fuchs) 2023-02-22 18:47:01 UTC
(In reply to breakingspell from comment #11)
> Sorry, hit enter by accident and submitted. 
> 
> The code checks for "no borders" and "no side-borders" to choose where to
> draw the outline, could that check be used to completely hide the border
> rather than adding "yet-another-checkbox"?

Imho no, since there are people who want a border but no outline (e.g. me), and most certainly also people who want no borders but the outline, since apparently it was added for accessibility.
Comment 13 Blazer Silving 2023-02-22 19:10:23 UTC
(In reply to Christian (Fuchs) from comment #12)
> (In reply to breakingspell from comment #11)
> > Sorry, hit enter by accident and submitted. 
> > 9
> > The code checks for "no borders" and "no side-borders" to choose where to
> > draw the outline, could that check be used to completely hide the border
> > rather than adding "yet-another-checkbox"?
> 
> Imho no, since there are people who want a border but no outline (e.g. me),
> and most certainly also people who want no borders but the outline, since
> apparently it was added for accessibility.

I see what you mean after switching them off and on for myself. Would have thought the "No Borders" label would be definitive for any decorations though, new users would likely think the same. 

An intensity slider is in the works per the merge request, at least.
Comment 14 Sanras 2023-02-23 17:04:02 UTC
(In reply to breakingspell from comment #13)
> (In reply to Christian (Fuchs) from comment #12)
> > (In reply to breakingspell from comment #11)
> > > Sorry, hit enter by accident and submitted. 
> > > 9
> > > The code checks for "no borders" and "no side-borders" to choose where to
> > > draw the outline, could that check be used to completely hide the border
> > > rather than adding "yet-another-checkbox"?
> > 
> > Imho no, since there are people who want a border but no outline (e.g. me),
> > and most certainly also people who want no borders but the outline, since
> > apparently it was added for accessibility.
> 
> I see what you mean after switching them off and on for myself. Would have
> thought the "No Borders" label would be definitive for any decorations
> though, new users would likely think the same. 
> 
> An intensity slider is in the works per the merge request, at least.

Good. I don't know what the hell they were thinking, just adding outlines to all windows without any configuration available of any sort.
Comment 15 Christian (Fuchs) 2023-02-23 17:11:46 UTC
Could we please try to stay friendly and within the KDE CoC?
I would have preferred an option, but it's easy enough to revert for now without breakage (just downgrade one single .so file), and attacking designers, devs or anyone really doesn't help matters at all.
Comment 16 Akseli Lahtinen 2023-03-03 20:52:00 UTC
Git commit 9b93fb968ed6a2817cee367aab5cfef7003b4073 by Akseli Lahtinen.
Committed on 03/03/2023 at 20:51.
Pushed by akselmo into branch 'master'.

Outline intensity setting

This MR adds a tiny outline intensity combobox to Breeze settings in the Shadow tab. 
I chose shadow tab since the outline is drawn in the shadow drawcall, 
so maybe this will help people realise it's not part of the borders.
FIXED-IN: 6.0

---

Here's examples with default breeze dark theme

Off

![Screenshot_20230302_181427](/uploads/22145f9d7b336f5437a4953933c4ab4d/Screenshot_20230302_181427.png)

Low

![Screenshot_20230302_181459](/uploads/807e3fb887e73a6fcd7c6cd70c713b26/Screenshot_20230302_181459.png)

Medium

![Screenshot_20230302_181437](/uploads/a14485ccbc0c2c6e7fc020972f96ab94/Screenshot_20230302_181437.png)

High

![Screenshot_20230302_181510](/uploads/30141d08ed9eb48eae4bba13c37db2fd/Screenshot_20230302_181510.png)

Maximum

![Screenshot_20230302_181452](/uploads/2eeb591d95a296adfa2daee4a985351e/Screenshot_20230302_181452.png)

Currently the effect only appears after pressing OK. This is same with all the Breeze window decoration settings afaik.

_Also, dont mind the settings window having same outline at the same time in the images: I'm not using the dev environment so it doesnt affect anything outside of the preview image. In dev env it changes everywhere._

M  +52   -30   kdecoration/breezedecoration.cpp
M  +10   -0    kdecoration/breezesettingsdata.kcfg
M  +12   -0    kdecoration/config/breezeconfigwidget.cpp
M  +47   -5    kdecoration/config/ui/breezeconfigurationui.ui

https://invent.kde.org/plasma/breeze/commit/9b93fb968ed6a2817cee367aab5cfef7003b4073
Comment 17 Nate Graham 2023-04-04 20:24:12 UTC
*** Bug 467500 has been marked as a duplicate of this bug. ***