-
-
Notifications
You must be signed in to change notification settings - Fork 552
Add background image continuation #3067
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
base: main
Are you sure you want to change the base?
Conversation
opacity -> normal_opacity transparency -> opacity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for attempting to continue this!
Note: I didn't review very carefully yet, due to so many basics things being broken and I did not check the code at all.
website/docs/configuration.md
Outdated
|
||
Use the `g:neovide_background_image` variable to set a background image for Neovide. It is set to a string url. | ||
|
||
You can alter the background images transparency by changing the `g:neovide_background_transparency` variable. It takes a value ranging from 0.0 (transparent) to 1.0 (opaque). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could be renamed to opacity
to be consistent with the other _opacity
settings. I also think that it would make sense to include image
to avoid confusion. So, perhaps neovide_bgimage_opacity
?
&screen_rect, | ||
grid_scale, | ||
) | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
&screen_rect, | ||
grid_scale, | ||
) | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
&screen_rect, | ||
grid_scale, | ||
) | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vim.g.neovide_background_image = "~/image.jpg" | ||
``` | ||
|
||
Using a background image in conjunction with `g:neovide_opacity` can lead to some nice looking results. Especially when using a more subtle texture instead of a regular image. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need to document that the feature requires either neovide_opacity
or neovide_normal_opacity
, otherwise the image will be hidden behind the background
This pr relates to issue: #342 and pr: #2419.
This is a continuation on mysty0s pr.
Adds two new settings
vim.g.neovide_background_image
andvim.g.neovide_background_transparency
.What kind of change does this PR introduce?
Did this PR introduce a breaking change?
A breaking change includes anything that breaks backwards compatibility either at compile or run time.