Skip to content
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

[Bug]: Vibrancy not applied in fullscreen (macOS) #37

Closed
1 task done
Spiderpig86 opened this issue Oct 24, 2022 · 8 comments
Closed
1 task done

[Bug]: Vibrancy not applied in fullscreen (macOS) #37

Spiderpig86 opened this issue Oct 24, 2022 · 8 comments
Labels
wontfix This will not be worked on

Comments

@Spiderpig86
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Using Vscode 1.71.2, when I change the window to use fullscreen mode (f11), vibrancy disappears. It appears again once it is no longer fullscreen.

Expected Behavior

Vibrancy should still be applied on the window even in fullscreen mode.

Steps To Reproduce

  1. Hit f11 or enter fullscreen on Vscode.

Environment

- OS: Windows 10
- VSCode: 1.71.2
- Extension: 1.1.9

Anything else?

No response

@Spiderpig86 Spiderpig86 added bug Something isn't working triage Cause of the issue needs to be triaged labels Oct 24, 2022
@ynsgnr
Copy link

ynsgnr commented Nov 21, 2022

I think this functions as intended, Windows and MacOS disables vibrancy on full screen to improve focus

@illixion
Copy link
Owner

I think this functions as intended, Windows and MacOS disables vibrancy on full screen to improve focus

Can confirm, this is likely intentional Electron behavior, you could maximize the window as an alternative.

@illixion illixion closed this as not planned Won't fix, can't repro, duplicate, stale Nov 21, 2022
@illixion illixion added wontfix This will not be worked on and removed bug Something isn't working triage Cause of the issue needs to be triaged labels Nov 21, 2022
@EmreSURK
Copy link

EmreSURK commented Dec 8, 2022

Nope, it is not.
Both Safari and Finder are still vibrant. Blur radius becomes very large but they are still being affected by the desktop wallpaper. You can see a slightly different colors for different wallpaper images.

Screenshot 2022-12-08 at 4 28 08 PM

Screenshot 2022-12-08 at 4 28 29 PM

As an alternative, we may put theme color on the background.
Here is an issue I opened for this:
#47

@MrConnorKenway
Copy link

This is intended. If you look at Default Dark.css, you will find

.monaco-workbench.fullscreen {
	background-color: #202020 !important; /* make the background-color non transparent when in fullscreen */
}

To reenable vibrancy in fullscreen mode, change it to

.monaco-workbench.fullscreen {
	background-color: transparent !important;
}

@illixion
Copy link
Owner

illixion commented Dec 9, 2022

@EmreSURK

Nope, it is not. Both Safari and Finder are still vibrant. Blur radius becomes very large but they are still being affected by the desktop wallpaper.

I believe what you're seeing is not transparency, but rather wallpaper tinting, which is a macOS setting that is on by default. With that setting off, there is no transparency.

@MrConnorKenway

This is intended. If you look at Default Dark.css, you will find

Good catch, though after checking this on macOS, it doesn't appear to actually enable transparency. Instead it just takes the background color average and applies that. I'll look into this later, but feel free to use the Vibrancy custom import feature in the meantime.

@EmreSURK
Copy link

@illixion Yes, apparently you are right.
Good Point.
I checked the image pixel colors with Digital Color Meter if there is a slight difference we can not release it. No, no different, only one color.

@minhiu
Copy link

minhiu commented Mar 22, 2023

This is intended. If you look at Default Dark.css, you will find

.monaco-workbench.fullscreen {
	background-color: #202020 !important; /* make the background-color non transparent when in fullscreen */
}

To reenable vibrancy in fullscreen mode, change it to

.monaco-workbench.fullscreen {
	background-color: transparent !important;
}

I have a dumb question but how do we modify this css file in VSCode?

@illixion
Copy link
Owner

This is intended. If you look at Default Dark.css, you will find

.monaco-workbench.fullscreen {
	background-color: #202020 !important; /* make the background-color non transparent when in fullscreen */
}

To reenable vibrancy in fullscreen mode, change it to

.monaco-workbench.fullscreen {
	background-color: transparent !important;
}

I have a dumb question but how do we modify this css file in VSCode?

You can use the import feature for that. Create a CSS file somewhere on the disk with your desired contents, copy the full path (on Windows, Shift+Right click then choose Copy as path) and add it to your VSCode config under Vibrancy settings as described here: https://github.com/illixion/vscode-vibrancy-continued#vscode_vibrancyimports

If you're on Windows, you'll also need to replace all backslashes with forward slashes.

@illixion illixion changed the title [Bug]: Vibrancy not applied in fullscreen [Bug]: Vibrancy not applied in fullscreen (macOS) Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

6 participants