-
-
Notifications
You must be signed in to change notification settings - Fork 110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Any colorscheme that looks like github code viewer? #298
Comments
hmmmm this is interesting. It may be because the bg is currently being derived (in a sort of manual fashion?) from the scale colors (and this may have been correct before but maybe the scale colors got changed/updated by github).
I have submitted a pr which should fix this. Once it is merged just update your plugin (to the latest main/master/HEAD), and then you can just use Note After updating the plugin, you may need to force an update of the cache with |
This should also correct any issues with the `Normal` bg being off or incorrect, as this color (i.e. the `Normal` bg, the global bg) ultimately derives from `canvas.default` (found in the palette). Fixes projekt0n#298
Thank you for the quick response @tmillr! looking forward the PR being merged. |
This should also correct any issues with the `Normal` bg being off or incorrect, as this color (i.e. the `Normal` bg, the global bg) ultimately derives from `canvas.default` (found in the palette). Fixes projekt0n#298
This should also correct any issues with the `Normal` bg being off or incorrect, as this color (i.e. the `Normal` bg, the global bg) ultimately derives from `canvas.default` (found in the palette). Fixes projekt0n#298
Thanks, @tmillr for the PR! I appreciate your work on this issue. The patched theme's canvas color is noticeably darker, almost like the |
hmmm do you mean in comparison to GitHub.com (the center window), or to what it was before the patch (i.e. topleft vs bottomleft)? As for the former, on the bottom img, the bg of the left and center window match. If you mean the latter (which I think is probably what you meant?), well for me here on the website while using the regular dark "theme" (i.e. Dark default) that is indeed the correct bg of the code viewer it seems: It is close to the same color used for dark high contrast, but it is not the same color, it is just a touch lighter. This is correct (as far as matching the website) and is just what GitHub has decided to go with. So it looks correct to me (based off this), if the goal is to match the colors of GitHub.com precisely. It just seems like GitHub chose 2 very similar bg colors for regular dark and dark high contrast. Note I used a color meter. On a somewhat unrelated note, it looks like |
The white theme is still different from the codeviewer white theme too |
@tmillr It looks like deprecating |
Just wondering, is there any blocker to #299? |
Sorry I can't merge it, but it should be good to go |
Yeah it's hard not to mix it up. In the web ui they call it i.e.
I believe this is what you're saying as well but just making sure. I suppose you could either leave it as it is now and remove the |
@ful1e5, the color is back again after the latest commits |
@NikoKS, consider switching your color scheme to The |
Hello, Thank you very much for this theme!
I was wondering if there is a colorscheme that match Github's native dark default code viewer? Here is a sample screenshot what it looks like:
I found that github_dark_dimmed is too light, and github_dark_high_contrast is too dark. Github's native dark default sits just right in the middle.
The text was updated successfully, but these errors were encountered: