Bug 418111 - minor visual regressions due to scrollbar border
Summary: minor visual regressions due to scrollbar border
Status: RESOLVED NOT A BUG
Alias: None
Product: Breeze
Classification: Plasma
Component: QStyle (show other bugs)
Version: 5.18.0
Platform: Arch Linux Linux
: NOR minor
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-02-23 22:51 UTC by maxmustermann1884
Modified: 2020-02-24 14:54 UTC (History)
2 users (show)

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


Attachments
minor visual regressions (958.23 KB, image/png)
2020-02-23 22:51 UTC, maxmustermann1884
Details

Note You need to log in before you can comment on or make changes to this bug.
Description maxmustermann1884 2020-02-23 22:51:36 UTC
Created attachment 126361 [details]
minor visual regressions

SUMMARY

some applications(dolphin, qmmp-> simple interface, clipboard configuration->actions) and widgets (notes, quick share) look slightly worse with the new scrollbar border.
Maybe it's possible to fix this by not drawing the border when not surrounded by a focus frame.
Or adding an option in breeze application style settings, wouldn't really fix it though :P




SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2020-02-24 05:02:02 UTC
Unfortunately each of these issues will have to be dealt with on an app-by-app basis. :(

Can you file a new bug for each issue?
Comment 2 Kai Uwe Broulik 2020-02-24 08:37:31 UTC
> each of these issues will have to be dealt with on an app-by-app basis. :(

A style should not needlessly break applications. Maybe this late scroll bar change wasn't a good idea, but I've been saying that a lot.
Comment 3 Nate Graham 2020-02-24 14:54:46 UTC
(In reply to Kai Uwe Broulik from comment #2)
> > each of these issues will have to be dealt with on an app-by-app basis. :(
> 
> A style should not needlessly break applications. Maybe this late scroll bar
> change wasn't a good idea, but I've been saying that a lot.
Well, it's complicated. Going by the image:

#1 looks like a a bug in PlasmaComponents; we decided not to add a line there, yet it's there anyway.
#2 seems to be a bug in Breeze
#3 is because the Breeze style doesn't use either frames or lines to separate views, so there's nothing for the scrollbar's line to "grab onto".
#4 is the same as #1

So I guess no apps are broken, it's more that the change itself needs more adjustments at the Breeze+Plasma+Frameworks levels.

Still, Max, can you file new bug reports for these?