aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/nvim/api/vim.c11
-rw-r--r--src/nvim/api/window.c17
-rw-r--r--src/nvim/buffer_defs.h4
-rw-r--r--src/nvim/ui.c4
-rw-r--r--src/nvim/window.c49
5 files changed, 55 insertions, 30 deletions
diff --git a/src/nvim/api/vim.c b/src/nvim/api/vim.c
index a773234ea0..37bad4d517 100644
--- a/src/nvim/api/vim.c
+++ b/src/nvim/api/vim.c
@@ -1004,8 +1004,6 @@ Buffer nvim_create_buf(Boolean listed, Boolean scratch, Error *err)
///
/// @param buffer handle of buffer to be displayed in the window
/// @param enter whether the window should be entered (made the current window)
-/// @param width width of window (in character cells)
-/// @param height height of window (in character cells)
/// @param options dict of options for configuring window positioning
/// accepts the following keys:
/// `relative`: If set, the window becomes a floating window. The window
@@ -1023,6 +1021,8 @@ Buffer nvim_create_buf(Boolean listed, Boolean scratch, Error *err)
/// `focusable`: Whether window can be focused by wincmds and
/// mouse events. Defaults to true. Even if set to false, the window
/// can still be entered using |nvim_set_current_win()| API call.
+/// `height`: window height (in character cells). Cannot be smaller than 1.
+/// `width`: window width (in character cells). Cannot be smaller than 2.
/// `row`: row position. Screen cell height are used as unit. Can be
/// floating point.
/// `col`: column position. Screen cell width is used as unit. Can be
@@ -1047,16 +1047,15 @@ Buffer nvim_create_buf(Boolean listed, Boolean scratch, Error *err)
///
/// @param[out] err Error details, if any
/// @return the window handle or 0 when error
-Window nvim_open_win(Buffer buffer, Boolean enter,
- Integer width, Integer height,
- Dictionary options, Error *err)
+Window nvim_open_win(Buffer buffer, Boolean enter, Dictionary options,
+ Error *err)
FUNC_API_SINCE(6)
{
FloatConfig config = FLOAT_CONFIG_INIT;
if (!parse_float_config(options, &config, false, err)) {
return 0;
}
- win_T *wp = win_new_float(NULL, (int)width, (int)height, config, err);
+ win_T *wp = win_new_float(NULL, config, err);
if (!wp) {
return 0;
}
diff --git a/src/nvim/api/window.c b/src/nvim/api/window.c
index f8bcaddb36..ba505af87e 100644
--- a/src/nvim/api/window.c
+++ b/src/nvim/api/window.c
@@ -439,13 +439,12 @@ Boolean nvim_win_is_valid(Window window)
/// types).
///
/// See documentation at |nvim_open_win()|, for the meaning of parameters. Pass
-/// in -1 for 'witdh' and 'height' to keep exiting size.
+/// in 0 for 'witdh' and 'height' to keep exiting size.
///
/// When reconfiguring a floating window, absent option keys will not be
/// changed. The following restriction apply: `row`, `col` and `relative`
/// must be reconfigured together. Only changing a subset of these is an error.
-void nvim_win_set_config(Window window, Integer width, Integer height,
- Dictionary options, Error *err)
+void nvim_win_set_config(Window window, Dictionary options, Error *err)
FUNC_API_SINCE(6)
{
win_T *win = find_window_by_handle(window, err);
@@ -453,21 +452,21 @@ void nvim_win_set_config(Window window, Integer width, Integer height,
return;
}
bool new_float = !win->w_floating;
- width = width > 0 ? width: win->w_width;
- height = height > 0 ? height : win->w_height;
// reuse old values, if not overriden
FloatConfig config = new_float ? FLOAT_CONFIG_INIT : win->w_float_config;
if (!parse_float_config(options, &config, !new_float, err)) {
return;
}
+ config.height = config.height > 0 ? config.height : win->w_height;
+ config.width = config.width > 0 ? config.width : win->w_width;
if (new_float) {
- if (!win_new_float(win, (int)width, (int)height, config, err)) {
+ if (!win_new_float(win, config, err)) {
return;
}
redraw_later(NOT_VALID);
} else {
- win_config_float(win, (int)width, (int)height, config);
+ win_config_float(win, config);
win->w_pos_changed = true;
}
}
@@ -490,8 +489,8 @@ Dictionary nvim_win_get_config(Window window, Error *err)
return rv;
}
- PUT(rv, "height", INTEGER_OBJ(wp->w_height));
- PUT(rv, "width", INTEGER_OBJ(wp->w_width));
+ PUT(rv, "width", INTEGER_OBJ(wp->w_float_config.width));
+ PUT(rv, "height", INTEGER_OBJ(wp->w_float_config.height));
PUT(rv, "focusable", BOOLEAN_OBJ(wp->w_float_config.focusable));
PUT(rv, "external", BOOLEAN_OBJ(wp->w_float_config.external));
PUT(rv, "anchor", STRING_OBJ(cstr_to_string(
diff --git a/src/nvim/buffer_defs.h b/src/nvim/buffer_defs.h
index 199399cf4d..f11f3e2332 100644
--- a/src/nvim/buffer_defs.h
+++ b/src/nvim/buffer_defs.h
@@ -979,6 +979,7 @@ typedef enum {
typedef struct {
Window window;
+ int height, width;
double row, col;
FloatAnchor anchor;
FloatRelative relative;
@@ -986,7 +987,8 @@ typedef struct {
bool focusable;
} FloatConfig;
-#define FLOAT_CONFIG_INIT ((FloatConfig){ .row = 0, .col = 0, .anchor = 0, \
+#define FLOAT_CONFIG_INIT ((FloatConfig){ .height = 0, .width = 0, \
+ .row = 0, .col = 0, .anchor = 0, \
.relative = 0, .external = false, \
.focusable = true })
diff --git a/src/nvim/ui.c b/src/nvim/ui.c
index d07ea3179e..0dddfe5f59 100644
--- a/src/nvim/ui.c
+++ b/src/nvim/ui.c
@@ -460,7 +460,9 @@ void ui_grid_resize(handle_T grid_handle, int width, int height, Error *error)
if (wp->w_floating) {
if (width != wp->w_width && height != wp->w_height) {
- win_config_float(wp, (int)width, (int)height, wp->w_float_config);
+ wp->w_float_config.width = width;
+ wp->w_float_config.height = height;
+ win_config_float(wp, wp->w_float_config);
}
} else {
// non-positive indicates no request
diff --git a/src/nvim/window.c b/src/nvim/window.c
index a4d2189ac1..dc043c3c45 100644
--- a/src/nvim/window.c
+++ b/src/nvim/window.c
@@ -502,10 +502,11 @@ wingotofile:
break;
}
FloatConfig config = FLOAT_CONFIG_INIT;
+ config.width = curwin->w_width;
+ config.height = curwin->w_height;
config.external = true;
Error err = ERROR_INIT;
- if (!win_new_float(curwin, curwin->w_width, curwin->w_height, config,
- &err)) {
+ if (!win_new_float(curwin, config, &err)) {
EMSG(err.msg);
api_clear_error(&err);
beep_flush();
@@ -538,8 +539,7 @@ static void cmd_with_count(char *cmd, char_u *bufp, size_t bufsize,
/// float. It must then already belong to the current tabpage!
///
/// config must already have been validated!
-win_T *win_new_float(win_T *wp, int width, int height, FloatConfig config,
- Error *err)
+win_T *win_new_float(win_T *wp, FloatConfig config, Error *err)
{
if (wp == NULL) {
wp = win_alloc(lastwin_nofloating(), false);
@@ -571,17 +571,16 @@ win_T *win_new_float(win_T *wp, int width, int height, FloatConfig config,
// TODO(bfredl): use set_option_to() after merging #9110 ?
wp->w_p_nu = false;
wp->w_allbuf_opt.wo_nu = false;
- win_config_float(wp, width, height, config);
+ win_config_float(wp, config);
wp->w_pos_changed = true;
redraw_win_later(wp, VALID);
return wp;
}
-void win_config_float(win_T *wp, int width, int height,
- FloatConfig config)
+void win_config_float(win_T *wp, FloatConfig config)
{
- wp->w_height = MAX(height, 1);
- wp->w_width = MAX(width, 2);
+ wp->w_width = MAX(config.width, 2);
+ wp->w_height = MAX(config.height, 1);
if (config.relative == kFloatRelativeCursor) {
config.relative = kFloatRelativeWindow;
@@ -729,6 +728,22 @@ bool parse_float_config(Dictionary config, FloatConfig *out, bool reconf,
"'col' option has to be Integer or Float");
return false;
}
+ } else if (strequal(key, "width")) {
+ if (val.type == kObjectTypeInteger && val.data.integer >= 0) {
+ out->width = val.data.integer;
+ } else {
+ api_set_error(err, kErrorTypeValidation,
+ "'width' option has to be a positive Integer");
+ return false;
+ }
+ } else if (strequal(key, "height")) {
+ if (val.type == kObjectTypeInteger && val.data.integer >= 0) {
+ out->height= val.data.integer;
+ } else {
+ api_set_error(err, kErrorTypeValidation,
+ "'height' option has to be a positive Integer");
+ return false;
+ }
} else if (!strcmp(key, "anchor")) {
if (val.type != kObjectTypeString) {
api_set_error(err, kErrorTypeValidation,
@@ -3804,7 +3819,9 @@ static void tabpage_check_windows(tabpage_T *old_curtab)
for (win_T *wp = firstwin; wp; wp = wp->w_next) {
if (wp->w_floating && !wp->w_float_config.external) {
- win_config_float(wp, wp->w_width, wp->w_height, wp->w_float_config);
+ wp->w_float_config.width = wp->w_width;
+ wp->w_float_config.height = wp->w_height;
+ win_config_float(wp, wp->w_float_config);
}
wp->w_pos_changed = true;
}
@@ -4664,7 +4681,9 @@ int win_comp_pos(void)
// Too often, but when we support anchoring floats to split windows,
// this will be needed
for (win_T *wp = lastwin; wp && wp->w_floating; wp = wp->w_prev) {
- win_config_float(wp, wp->w_width, wp->w_height, wp->w_float_config);
+ wp->w_float_config.width = wp->w_width;
+ wp->w_float_config.height = wp->w_height;
+ win_config_float(wp, wp->w_float_config);
}
return row;
@@ -4737,7 +4756,9 @@ void win_setheight_win(int height, win_T *win)
if (win->w_floating) {
if (win->w_float_config.external) {
- win_config_float(win, win->w_width, height, win->w_float_config);
+ win->w_float_config.width = win->w_width;
+ win->w_float_config.height = height;
+ win_config_float(win, win->w_float_config);
} else {
beep_flush();
return;
@@ -4942,7 +4963,9 @@ void win_setwidth_win(int width, win_T *wp)
}
if (wp->w_floating) {
if (wp->w_float_config.external) {
- win_config_float(wp, width, wp->w_height, wp->w_float_config);
+ wp->w_float_config.width = width;
+ wp->w_float_config.height = wp->w_height;
+ win_config_float(wp, wp->w_float_config);
} else {
beep_flush();
return;
generated by cgit (git 2.34.1) at 2025-05-09 14:26:24 +0000