Bug 431904 - Extra separator line
Summary: Extra separator line
Status: CLOSED INTENTIONAL
Alias: None
Product: Breeze
Classification: Plasma
Component: general (show other bugs)
Version: 5.20.90
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-01-21 22:05 UTC by Iyán Méndez Veiga
Modified: 2021-01-28 12:29 UTC (History)
7 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Screenshots (399.20 KB, application/gzip)
2021-01-21 22:05 UTC, Iyán Méndez Veiga
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Iyán Méndez Veiga 2021-01-21 22:05:14 UTC
Created attachment 135045 [details]
Screenshots

SUMMARY

New Breeze theme looks awesome but there is a minor detail that I find a bit annoying, specially with Breeze Dark.

It feels to me that there is an extra and not needed line between the menu bars and the rest of the application. I attach a few screenshots to make my point.
Comment 1 Janet Blackquill 2021-01-21 23:24:36 UTC
This is an intentional design aspect; we added a separator between the area with toolbars + menubars (which we call the "tools area") and the rest of the window content. Without this separator, it would look strange having the top area be a different colour from the rest of the window.
Comment 2 Nate Graham 2021-01-21 23:53:40 UTC
Yup, what she said. ^^
Comment 3 Iyán Méndez Veiga 2021-01-22 07:06:17 UTC
Okay, I see. Still, is this open to discussion before the release of final Plasma 5.21? At lest, I think a different color should be used. In breeze dark, that white separator is really distracting, and I still think it makes no sense in most applications.

Can you let me know, in what file is this separator defined so I can play a bit with it in Breeze dark and propose alternatives?

Thanks.
Comment 4 Iyán Méndez Veiga 2021-01-22 08:09:13 UTC
Also, I don't think this extra bar look like what was discussed in https://phabricator.kde.org/T10891

Note that in all those mockups, yes, there is a separator line but it is not doubled.
Comment 5 Nate Graham 2021-01-22 15:35:13 UTC
What you're calling a doubled line is an artifact of some apps' main views still being inside a frame. This is something we plan to fix in Plasma 5.22. But the separator line stays, sorry. :)

As for its color, there is a faction in KDE's visual design group that prefers dark-colored separators when using a dark theme. I happen to be in that group. I think it would look great. We haven't reached a consensus yet though.
Comment 6 Iyán Méndez Veiga 2021-01-22 16:25:12 UTC
Yeah, okay, you are right that those two lines are different things. Hope theme continues improving over Plasma 5.22. I guess I will get used to it after a few days.
Comment 7 Nicolas Fella 2021-01-23 09:35:14 UTC
*** Bug 431969 has been marked as a duplicate of this bug. ***
Comment 8 Rind 2021-01-23 12:44:19 UTC
> What you're calling a doubled line is an artifact of some apps' main views still being inside a frame. This is something we plan to fix in Plasma 5.22. But the separator line stays, sorry. :)

So this artifact will continue to be displayed for nearly four months until the 5.22 plasma launch? It's annoying to watch a maximized video with this white line on top of the screen.
Comment 9 Nate Graham 2021-01-23 22:50:00 UTC
(In reply to Rind from comment #8)
> > What you're calling a doubled line is an artifact of some apps' main views still being inside a frame. This is something we plan to fix in Plasma 5.22. But the separator line stays, sorry. :)
> 
> So this artifact will continue to be displayed for nearly four months until
> the 5.22 plasma launch? It's annoying to watch a maximized video with this
> white line on top of the screen.
Ouch, no, that's a bad bug that we need to fix. The line is not supposed to be visible in full-screen windows with nothing above it. Can you file a separate bug report for that? Thanks! I'll make sure it gets addressed.
Comment 10 Podagric 2021-01-28 12:29:41 UTC
I already did: 431969
was marked as a duplicate of this one.