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

winblend with opaque terminal #10685

Open
talbergs opened this issue Aug 3, 2019 · 5 comments
Open

winblend with opaque terminal #10685

talbergs opened this issue Aug 3, 2019 · 5 comments
Assignees
Labels
bug issues reporting wrong behavior highlight ui
Milestone

Comments

@talbergs
Copy link

talbergs commented Aug 3, 2019

Terminal is transparent except for floating windows.

I'm not sure if this is a bug. In the snippet below, I create floating window, my terminal app is opaque, so I can see desktop background, that's not the case for floating window though.

  • NVIM v0.4.0-1534-g24fb7eefa | Build type: Debug
  • Operating system/version: 5.1.2-arch1-1-ARCH | sway (wayland wm)
  • Terminal name/version: termite v15

Steps to reproduce using nvim -u NORC

let l = []| let w = winwidth(0)| let h = winheight(0)| let s = ""
while w > 0| let w -= 1| let s .= "."| endwhile
while h > 0| let h -= 1| let l += [s]| endwhile
call append("$", l)

set showtabline=2
set termguicolors

hi TabLine gui=NONE cterm=NONE
hi TabLineFill gui=NONE cterm=NONE
hi TabLineSel gui=NONE cterm=NONE
hi Normal guibg=NONE

let buf = nvim_create_buf(v:false, v:true)
let win = nvim_open_win(buf, v:true, {
            \'row':0,
            \'col':0,
            \'width':20,
            \'height':10,
            \'relative':'editor',
            \'focusable':v:true})

call nvim_win_set_option(win, "winblend", 100)
call nvim_buf_set_lines(buf, 0, 1, v:false, ["   TEXT", ""])

Actual behaviour

Desktop background not visible only through floating window.

Expected behaviour

To be visible. (for winblend value of 100)

@talbergs talbergs added the bug issues reporting wrong behavior label Aug 3, 2019
@bfredl
Copy link
Member

bfredl commented Aug 3, 2019

It is because with blend>0 we always blend the color. We could add an edge case for blend=100 to preserve the back layer background exactly (including bg=NONE)

@bfredl bfredl self-assigned this Aug 3, 2019
@bfredl bfredl added this to the 0.4 milestone Aug 3, 2019
@bfredl bfredl added the ui label Aug 3, 2019
@justinmk
Copy link
Member

@bfredl is this a blocker ? Seems like it could wait until 0.5

@bfredl
Copy link
Member

bfredl commented Sep 11, 2019

Yeah it is just "nice to have". Most people do not use winblend=100

@justinmk justinmk modified the milestones: 0.4, 0.5 Sep 11, 2019
@janlazo janlazo modified the milestones: 0.5, 0.5.1 Feb 14, 2021
@justinmk justinmk modified the milestones: 0.5.1, 0.5.2 Sep 26, 2021
@zeertzjq zeertzjq modified the milestones: 0.6.1, 0.7 Dec 27, 2021
@Maswor
Copy link

Maswor commented Mar 7, 2022

Confirmed that the bug is still around for Alacritty on MacOS. It would be nice to have :)

@Chaitanyabsprip
Copy link

still an issue, should be prominent among transparent theme users along with noice/drop/notify, etc

@justinmk justinmk modified the milestones: 0.9, backlog Jan 26, 2023
@justinmk justinmk changed the title Opaque terminal and winblend winblend with opaque terminal Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug issues reporting wrong behavior highlight ui
Projects
None yet
Development

No branches or pull requests

7 participants