| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
These also apply to light mode, should they? And also * applies to every element. But more importantly, this may override the user's OS scrollbar settings, which they may have customised for accessibility. Forcing them smaller and darker might mean it's hard for some to control or see. |
Sorry, something went wrong.
|
I adjusted the scope and now only add it where needed
This has been reused according to the existing heatmap design standards, so they should not have much impact:) cpython/Lib/profiling/sampling/_heatmap_assets/heatmap.css Lines 815 to 827 in 1f36a51 |
Sorry, something went wrong.
|
@hugovk @pablogsal May I ask if there are any further changes I need? |
Sorry, something went wrong.
|
@hugovk I am good with this PR but I will give you a couple of days before merging in case you have further comments. |
Sorry, something went wrong.
|
I don't think we should change the scrollbars, it makes the colour contrast worse and can be harder to use for those who need it. I don't think the original grey looks bad either. |
Sorry, something went wrong.
Sorry, something went wrong.
There was a problem hiding this comment.
Testing on macOS, with Chrome 143, there's no difference. The scrollbars are already invisible: black on black with no outline. This PR doesn't change this.
Likewise on Safari 263, the scrollbars are black on black, but at least with a dark grey outline, so they're just about visible. This PR doesn't change this either.
On Firefox 149, this PR changes the scrollbars from light grey on black, to a darker grey on black. Both of these are better than Chrome and Safari, but I think it was better before.
If anything, we should be making the scrollbars [more] visible, not less.
Sorry, something went wrong.
| } | ||
|
|
||
| /* Scrollbar Styling */ | ||
| .line-content { |
There was a problem hiding this comment.
Nit: we already have a .line-content block just before this.
Sorry, something went wrong.
|
This PR is stale because it has been open for 90 days with no activity. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
@hugovk
For compatibility, both approaches must be retained here.
::-webkit-scrollbar offers the widest compatibility, but requires fully customizing the scrollbar.

https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/Selectors/::-webkit-scrollbar#browser_compatibility
scrollbar-color is the newer standard, allowing us to style scrollbars without full customization. However, it is not supported by browsers from several years ago.
https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/Properties/scrollbar-color#browser_compatibility
