Bug 418111

Summary: minor visual regressions due to scrollbar border
Product: [Plasma] Breeze Reporter: maxmustermann1884
Component: QStyleAssignee: Unassigned bugs <unassigned-bugs-null>
Status: RESOLVED NOT A BUG    
Severity: minor CC: kde, nate
Priority: NOR    
Version First Reported In: 5.18.0   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: minor visual regressions

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?