Add status_height to the dict items returned by getwininfo()
#18841
Conversation
| {only with the +quickfix feature} | ||
| quickfix 1 if quickfix or location list window | ||
| {only with the +quickfix feature} | ||
| status_height status lines height (0 or 1) |
There was a problem hiding this comment.
But this is not really the height, but rather whether a statusline is displayed, so
shouldn't that be rather:
| status_height status lines height (0 or 1) | |
| statusline window has a status line displayed (0 or 1) |
There was a problem hiding this comment.
At the moment it is, but I'm working on a multi-line-statusline patch.
There was a problem hiding this comment.
okay, but then we would have an entry statusline which indicates whether the window has a statusline and stl_height which gives the line height, no?
There was a problem hiding this comment.
I think a single entry is sufficient. A value of 0 means no statusline. Any other value specifies the statusline’s height. Negative values are not allowed.
|
okay, fair thanks |
|
Thanks for the merging. There may be room to discuss the item name
Does anyone have any thoughts? |
2 participants
I think it's strange that this hasn't existed until now.
(In addition, the order of items in the
getwininfo()documentation has been changed to alphabetical order)