aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar Jan Mulder <jlmulder@xs4all.nl>2017-11-16 12:02:01 +0100
committerGravatar Jan Mulder <jlmulder@xs4all.nl>2017-11-17 09:19:03 +0100
commitc8a7712ea5587af54c590ad1b57142a3d9af122a (patch)
treed5aedced06ce535b7961e3c0738549bafdda403e
parent010f5f2abc6f6781282ec55b4749eb632d505be1 (diff)
downloadsubsurface-c8a7712ea5587af54c590ad1b57142a3d9af122a.tar.gz
Show all autocompletes after entry of comma in tag list
The almost 3 year old commit e1db5f21b15b2 introduced the behavior that you need to type at least 1 character for a tag to show any autocompleted data. While this went unnoticed for years, there is always a user that notices this changed behavior. The solution is rather simple. Do allow the 0 lenght string to act as seed for autocompletion instead of just returning, and doing no autocomplete at all. As the afore mentioned commit was explicitly meant to prevent over active firing of the tag list autocompleter, this (simple) change is carefully tested on this, and no adverse effects are visible. Fixes: #605 (wrt the 1 character issue) Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>
-rw-r--r--desktop-widgets/tagwidget.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/desktop-widgets/tagwidget.cpp b/desktop-widgets/tagwidget.cpp
index b0e6e3249..b88f967d2 100644
--- a/desktop-widgets/tagwidget.cpp
+++ b/desktop-widgets/tagwidget.cpp
@@ -91,7 +91,7 @@ void TagWidget::reparse()
* Do not show the completer when not in edit mode - basically
* this returns when we are accepting or discarding the changes.
*/
- if (MainWindow::instance()->information()->isEditing() == false || currentText.length() == 0) {
+ if (MainWindow::instance()->information()->isEditing() == false) {
return;
}