Merge branch 'master' into globals

Author: quantimnot

nimx/formatted_text.nim

@@ -665,17 +665,24 @@ void compose()
 """, false, "mediump")
 
 type ForEachLineAttributeCallback = proc(gfx: GraphicsContext, t: FormattedText, p: var Point, curLine, endIndex: int, str: string) {.nimcall.}
-proc forEachLineAttribute(gfx: GraphicsContext, origP: Point, t: FormattedText, cb: ForEachLineAttributeCallback) =
+proc forEachLineAttribute(gfx: GraphicsContext, inRect: Rect, origP: Point, t: FormattedText, cb: ForEachLineAttributeCallback) =
     template fontCtx: untyped = gfx.fontCtx
     template gl: untyped = gfx.gl
     var p = origP
     let numLines = t.lines.len
     var curLine = 0
     let top = topOffset(fontCtx, gl, t) + origP.y
+    let inRectValid = (inRect.y + inRect.height) > 0.01
 
     while curLine < numLines:
         p.x = origP.x + lineLeft(fontCtx, gl, t, curLine)
         p.y = t.lines[curLine].top + t.lines[curLine].baseline + top
+        if inRectValid:
+            if p.y < inRect.y:
+                curLine.inc
+                continue
+            elif p.y > inRect.height:
+                break
 
         var lastCurAttrIndex: int
         var lastAttrStartIndex: int
@@ -741,9 +748,9 @@ proc forEachLineAttribute(gfx: GraphicsContext, origP: Point, t: FormattedText,
         curLine.inc
 
 
-proc drawShadow(gfx: GraphicsContext, origP: Point, t: FormattedText) =
+proc drawShadow(gfx: GraphicsContext, inRect: Rect, origP: Point, t: FormattedText) =
     # TODO: Optimize heavily
-    forEachLineAttribute(gfx, origP, t) do(gfx: GraphicsContext, t: FormattedText, p: var Point, curLine, curAttrIndex: int, str: string):
+    forEachLineAttribute(gfx, inRect, origP, t) do(gfx: GraphicsContext, t: FormattedText, p: var Point, curLine, curAttrIndex: int, str: string):
         template gl: untyped = gfx.gl
         gfx.fillColor = t.mAttributes[curAttrIndex].shadowColor
         let font = t.mAttributes[curAttrIndex].font
@@ -784,9 +791,9 @@ proc drawShadow(gfx: GraphicsContext, origP: Point, t: FormattedText) =
         font.baseline = oldBaseline
         p.x += pp.x - ppp.x
 
-proc drawStroke(gfx: GraphicsContext, origP: Point, t: FormattedText) =
+proc drawStroke(gfx: GraphicsContext, inRect: Rect, origP: Point, t: FormattedText) =
     # TODO: Optimize heavily
-    forEachLineAttribute(gfx, origP, t) do(gfx: GraphicsContext, t: FormattedText, p: var Point, curLine, curAttrIndex: int, str: string):
+    forEachLineAttribute(gfx, inRect, origP, t) do(gfx: GraphicsContext, t: FormattedText, p: var Point, curLine, curAttrIndex: int, str: string):
         const magicStrokeMaxSizeCoof = 0.46
         let font = t.mAttributes[curAttrIndex].font
 
@@ -828,16 +835,16 @@ proc drawStroke(gfx: GraphicsContext, origP: Point, t: FormattedText) =
             # Dirty hack to advance x position. Should be optimized, of course.
             gfx.drawText(font, p, str)
 
-proc drawText*(gfx: GraphicsContext, origP: Point, t: FormattedText) =
+proc drawText*(gfx: GraphicsContext, origP: Point, t: FormattedText, inRect: Rect = zeroRect) =
     template fontCtx: untyped = gfx.fontCtx
     template gl: untyped = gfx.gl
     updateCacheIfNeeded(fontCtx, gl, t)
 
     if t.overrideColor.a == 0:
-        if t.shadowAttrs.len > 0: gfx.drawShadow(origP, t)
-        if t.strokeAttrs.len > 0: gfx.drawStroke(origP, t)
+        if t.shadowAttrs.len > 0: gfx.drawShadow(inRect, origP, t)
+        if t.strokeAttrs.len > 0: gfx.drawStroke(inRect, origP, t)
 
-    forEachLineAttribute(gfx, origP, t) do(gfx: GraphicsContext, t: FormattedText, p: var Point, curLine, curAttrIndex: int, str: string):
+    forEachLineAttribute(gfx, inRect, origP, t) do(gfx: GraphicsContext, t: FormattedText, p: var Point, curLine, curAttrIndex: int, str: string):
         let font = t.mAttributes[curAttrIndex].font
         let oldBaseline = font.baseline
         font.baseline = bAlphabetic

nimx/scroll_bar.nim

@@ -10,20 +10,21 @@ type ScrollBar* = ref object of Slider
                     # document is 0.5.
     trackingPos: Coord # Position of mouse coordinate (x or y depending on orientation) within knob
 
+const minKnobSize = 0.05
 method init*(s: ScrollBar, gfx: GraphicsContext, r: Rect) =
     procCall s.Slider.init(gfx, r)
     s.mKnobSize = 0.2
 
 proc knobRect(s: ScrollBar): Rect =
     result = s.bounds
     if s.isHorizontal:
-        result.size.width *= s.mKnobSize
+        result.size.width *= max(s.mKnobSize, minKnobSize)
         result.origin.x = max((s.bounds.width - result.width) * s.value, 3)
         if result.maxX > s.bounds.width - 3:
             result.size.width = s.bounds.width - result.x - 3
         result = inset(result, 0, 2)
     else:
-        result.size.height *= s.mKnobSize
+        result.size.height *= max(s.mKnobSize, minKnobSize)
         result.origin.y = max((s.bounds.height - result.height) * s.value, 3)
         if result.maxY > s.bounds.height - 3:
             result.size.height = s.bounds.height - result.y - 3

nimx/text_field.nim

@@ -262,6 +262,15 @@ proc drawSelection(t: TextField) {.inline.} =
         r.size.width = endOff
         gfx.drawRect(r)
 
+#todo: replace by generic visibleRect which should be implemented in future
+proc visibleRect(t: TextField): Rect =
+    let wndRect = t.convertRectToWindow(t.bounds)
+    let wndBounds = t.window.bounds
+
+    result.origin.y = if wndRect.y < 0.0: abs(wndRect.y) else: 0.0
+    result.size.width = t.bounds.width
+    result.size.height = min(t.bounds.height, wndBounds.height) + result.y - max(wndRect.y, 0.0)
+
 method draw*(t: TextField, r: Rect) =
     procCall t.View.draw(r)
 
@@ -284,7 +293,11 @@ method draw*(t: TextField, r: Rect) =
         t.mText.overrideColor = whiteColor()
     else:
         t.mText.overrideColor.a = 0
-    gfx.drawText(pt, t.mText)
+
+    if t.bounds.height > t.window.bounds.height:
+        gfx.drawText(pt, t.mText, t.visibleRect())
+    else:
+        gfx.drawText(pt, t.mText)
 
     if t.isEditing:
         if t.hasBezel: