Fix TextHandle min/max position handling

When not explicitly specified, the maximum is the length of the editor
content, not minus one (cursor can be moved "past" the last character).

Furthermore, due to binding evaluation orders, the minimum/maximum
properties might be temporarily pointing outside the range so we must
validate them to avoid clients (TextField/TextArea) having to do that.

Change-Id: I1faddd1e7ac4f12d7a802484ceb8eea500ad8bfc
Reviewed-by: Gabriel de Dietrich <gabriel.dedietrich@digia.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/src/controls/Private/TextHandle.qml b/src/controls/Private/TextHandle.qml
index 147f61fa8..558d0fa37 100644
--- a/src/controls/Private/TextHandle.qml
+++ b/src/controls/Private/TextHandle.qml
@@ -93,7 +93,7 @@ Loader {
 
             // limit vertically within mix/max coordinates or content bounds
             var min = (minimum !== -1) ? minimum : 0
-            var max = (maximum !== -1) ? maximum : editor.length - 1
+            var max = (maximum !== -1) ? maximum : editor.length
             pt.y = Math.max(pt.y, editor.positionToRectangle(min).y)
             pt.y = Math.min(pt.y, editor.positionToRectangle(max).y)
 
@@ -102,8 +102,10 @@ Loader {
             // limit horizontally within min/max character positions
             if (minimum !== -1)
                 pos = Math.max(pos, minimum)
+            pos = Math.max(pos, 0)
             if (maximum !== -1)
                 pos = Math.min(pos, maximum)
+            pos = Math.min(pos, editor.length)
 
             handle.position = pos
         }