Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

[CEF 2171] Red scrollbars #10107

Closed
marcelgerber opened this issue Dec 7, 2014 · 9 comments
Closed

[CEF 2171] Red scrollbars #10107

marcelgerber opened this issue Dec 7, 2014 · 9 comments

Comments

@marcelgerber
Copy link
Contributor

So, with the CEF 2171 Preview build, I'm seeing red ("bleeding") scrollbars without a thumb in Developer tools and the test window (not in Brackets itself!):
image

Windows 8.1 64bit

@marcelgerber
Copy link
Contributor Author

I should have mentioned that the CEF 3.2171.1902 Test App for Windows 32, downloaded from http://cefbuilds.com, works perfectly fine.
So this is definitely a Brackets issue.

@JeffryBooher
Copy link
Contributor

interesting indeed as we do not style these elements directly.

@marcelgerber
Copy link
Contributor Author

@JeffryBooher The issue is that cef_100_percent.pak is not included in installer builds.
I could easily fix the issue by moving that file into my Brackets folder (next to the Brackets.exe).
Don't know though if we also need cef_200_percent.pak.

E: 100_percent is for devices with a pixel ratio of 1, 200_percent for ones with a pixel ratio of 2.

@JeffryBooher
Copy link
Contributor

Good catch!

@peterflynn peterflynn added this to the Release 1.1 milestone Dec 8, 2014
@mackenza
Copy link
Contributor

mackenza commented Dec 8, 2014

I am seeing this as well. I thought it was a devtools "thing" indicating there was an error... like "bam! here are some red scrollbars, fix your damn code" type of thing ;)

But I guess not... heh

@ingorichter
Copy link
Contributor

@JeffryBooher did you add cef_200_percent.pak to the gyp file?

@JeffryBooher
Copy link
Contributor

@ingorichter the files are all copied out of the resources folder so it isn't copying them out by name. The grunt script for the installer, howerver, will need to do this. I have not yet looked at that.

@JeffryBooher
Copy link
Contributor

@ingorichter changes pushed. can you roll test builds?

@ingorichter
Copy link
Contributor

Thanks @JeffryBooher! I have kicked off a new build on the build machine. I've tested it locally on my machine and now the once missing files are there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants