kassio / neoterm

Wrapper of some vim/neovim's :terminal functions.
Other
1.32k stars 117 forks source link

neoterm size doesn't work #129

Closed musaffa closed 7 years ago

musaffa commented 7 years ago

I'm using neoterm with vim-test. I've tried to set vertical size with neoterm_size but it doesn't seem to work. It always takes the half of the current window.

let test#strategy = 'neoterm'
let g:neoterm_size = 30

I also tried with quoted number: let g:neoterm_size = "30". No result.

kassio commented 7 years ago

I wasn't able to replicate the behaviour. Can you give me more information, or a gif? Thanks

kassio commented 7 years ago

btw, I tried with:

  let g:neoterm_position = "vertical"
  let g:neoterm_size = 30
musaffa commented 7 years ago

Size works with vertical positioning. But it doesn't seem to work with horizontal positioning. Regardless of the size I set, this is what I see in the editor:

screenshot from 2017-02-21 00-01-19

kassio commented 7 years ago

Can you put here your neoterm configurations?

musaffa commented 7 years ago

It is just:

let test#strategy = 'neoterm'
let g:neoterm_size = 30
kassio commented 7 years ago

Hum.. but this seems to be right, if you put the focus on the neoterm window you can do :echo winheight('%') and this will return 30. Now if you want you neoterm to be vertical you'll need to set let g:neoterm_position = "vertical".

musaffa commented 7 years ago

But I prefer horizontal positioning. :(

kassio commented 7 years ago

So, it's already working?! right?! have you checked the window size with :echo winheight('%')?

musaffa commented 7 years ago

Oh didn't realize 30 is the half of the whole window. I was thinking in terms of vimux where 30 produced much smaller height.

Thanks for your time. :)

ghost commented 7 years ago

Same issue here, g:neoterm_size seems doesn't work with horizontal positioning. neoterm

codeinabox commented 6 years ago

How does this option now work in relation to g:neoterm_default_mod? I have the following settings but I noticed the size option no longer has any effect

let g:neoterm_size = 10
let g:neoterm_default_mod = 'split'
kassio commented 6 years ago

@codeinabox with the new api something probably broke. Can you open a new issue, please?

codeinabox commented 6 years ago

Done, new issue is #202