InputText: Reworked prev/next-word behavior . Include period as delimiter and tweak prev/next words logic. (#6067)
diff --git a/docs/CHANGELOG.txt b/docs/CHANGELOG.txt
index fa8f50b..f110193 100644
--- a/docs/CHANGELOG.txt
+++ b/docs/CHANGELOG.txt
@@ -44,6 +44,9 @@
retaining underlying data. While we don't really want to encourage user not retaining
underlying data, in the absence of a "late commit" behavior/flag we understand it may
be desirable to take advantage of this trick. (#4714)
+- InputText: Reworked prev/next-word behavior to more closely match Visual Studio
+ text editor. Include '.' as a delimiter and alter varying subtle behavior with how
+ blanks and separators are treated when skipping words. (#6067) [@ajweeks]
- Drag, Sliders: Fixed parsing of text input when '+' or '#' format flags are used
in the format string. (#6259) [@idbrii]
- Nav: Made Ctrl+Tab/Ctrl+Shift+Tab windowing register ownership to held modifier so
diff --git a/imgui.h b/imgui.h
index 92f448d..72947b3 100644
--- a/imgui.h
+++ b/imgui.h
@@ -23,7 +23,7 @@
// Library Version
// (Integer encoded as XYYZZ for use in #if preprocessor conditionals, e.g. '#if IMGUI_VERSION_NUM > 12345')
#define IMGUI_VERSION "1.89.5 WIP"
-#define IMGUI_VERSION_NUM 18944
+#define IMGUI_VERSION_NUM 18945
#define IMGUI_HAS_TABLE
/*
diff --git a/imgui_widgets.cpp b/imgui_widgets.cpp
index eb9c590..b1e3fb4 100644
--- a/imgui_widgets.cpp
+++ b/imgui_widgets.cpp
@@ -3700,10 +3700,34 @@
r->num_chars = (int)(text_remaining - (text + line_start_idx));
}
-// When ImGuiInputTextFlags_Password is set, we don't want actions such as CTRL+Arrow to leak the fact that underlying data are blanks or separators.
-static bool is_separator(unsigned int c) { return ImCharIsBlankW(c) || c==',' || c==';' || c=='(' || c==')' || c=='{' || c=='}' || c=='[' || c==']' || c=='|' || c=='\n' || c=='\r'; }
-static int is_word_boundary_from_right(ImGuiInputTextState* obj, int idx) { if (obj->Flags & ImGuiInputTextFlags_Password) return 0; return idx > 0 ? (is_separator(obj->TextW[idx - 1]) && !is_separator(obj->TextW[idx]) ) : 1; }
-static int is_word_boundary_from_left(ImGuiInputTextState* obj, int idx) { if (obj->Flags & ImGuiInputTextFlags_Password) return 0; return idx > 0 ? (!is_separator(obj->TextW[idx - 1]) && is_separator(obj->TextW[idx])) : 1; }
+static bool is_separator(unsigned int c)
+{
+ return c==',' || c==';' || c=='(' || c==')' || c=='{' || c=='}' || c=='[' || c==']' || c=='|' || c=='\n' || c=='\r' || c=='.' || c=='!';
+}
+
+static int is_word_boundary_from_right(ImGuiInputTextState* obj, int idx)
+{
+ // When ImGuiInputTextFlags_Password is set, we don't want actions such as CTRL+Arrow to leak the fact that underlying data are blanks or separators.
+ if ((obj->Flags & ImGuiInputTextFlags_Password) || idx <= 0)
+ return 0;
+
+ bool prev_white = ImCharIsBlankW(obj->TextW[idx - 1]);
+ bool prev_separ = is_separator(obj->TextW[idx - 1]);
+ bool curr_white = ImCharIsBlankW(obj->TextW[idx]);
+ bool curr_separ = is_separator(obj->TextW[idx]);
+ return ((prev_white || prev_separ) && !(curr_separ || curr_white)) || (curr_separ && !prev_separ);
+}
+static int is_word_boundary_from_left(ImGuiInputTextState* obj, int idx)
+{
+ if ((obj->Flags & ImGuiInputTextFlags_Password) || idx <= 0)
+ return 0;
+
+ bool prev_white = ImCharIsBlankW(obj->TextW[idx]);
+ bool prev_separ = is_separator(obj->TextW[idx]);
+ bool curr_white = ImCharIsBlankW(obj->TextW[idx - 1]);
+ bool curr_separ = is_separator(obj->TextW[idx - 1]);
+ return ((prev_white) && !(curr_separ || curr_white)) || (curr_separ && !prev_separ);
+}
static int STB_TEXTEDIT_MOVEWORDLEFT_IMPL(ImGuiInputTextState* obj, int idx) { idx--; while (idx >= 0 && !is_word_boundary_from_right(obj, idx)) idx--; return idx < 0 ? 0 : idx; }
static int STB_TEXTEDIT_MOVEWORDRIGHT_MAC(ImGuiInputTextState* obj, int idx) { idx++; int len = obj->CurLenW; while (idx < len && !is_word_boundary_from_left(obj, idx)) idx++; return idx > len ? len : idx; }
static int STB_TEXTEDIT_MOVEWORDRIGHT_WIN(ImGuiInputTextState* obj, int idx) { idx++; int len = obj->CurLenW; while (idx < len && !is_word_boundary_from_right(obj, idx)) idx++; return idx > len ? len : idx; }
