cucumber-forge-desktop icon indicating copy to clipboard operation
cucumber-forge-desktop copied to clipboard

Fix OverlayScrollbar

Open jkuester opened this issue 4 years ago • 6 comments

Newer versions of Chrome/Electron appear to have broken the OverlayScrollbar functionality: https://github.com/electron/electron/issues/14901.

For now CFD has been reverted to the default scrollbars until we can find a solution to this issue. Possibly we just need to integrate a manual css/js solution from a dependency like https://github.com/KingSora/OverlayScrollbars.

jkuester avatar Jan 04 '21 15:01 jkuester

Do we want to just hide scrollbars?

TobiTenno avatar Jan 04 '21 21:01 TobiTenno

I mean, I don't think we want to lose them altogether since some people might want to use them for navigating the report. It would just be nice if they could "overlay" like they used to...

jkuester avatar Jan 04 '21 21:01 jkuester

yeah, i just want to avoid adding bloat for the sake of sidebars

TobiTenno avatar Jan 04 '21 22:01 TobiTenno

Agreed. Maybe there is still a way to do this "natively" in electron, but I have not found it yet.

jkuester avatar Jan 04 '21 23:01 jkuester

Theoretically we could do the scrollbars styling manually as we do the hiding elsewhere.

TobiTenno avatar Jan 05 '21 00:01 TobiTenno

Google Chrome supports overlay scrollbars as a flag that you can enable. Electron doesn't seem to have the same support. Are overlay scrollbars going to be added in? Without them apps look a bit ugly on the eye in Windows.

SeanHub avatar Oct 06 '22 23:10 SeanHub