From a30b08d01f2e7b379798e8fa330cfdef67bd89e0 Mon Sep 17 00:00:00 2001
From: "Joshua A. Horton" <joshua_horton@sil.org>
Date: Thu, 21 Mar 2019 15:10:22 +0700
Subject: [PATCH] Minor documentation tweaks.

---
 web/source/osk/activeLayout.ts   | 5 +----
 web/source/osk/visualKeyboard.ts | 6 +++---
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/web/source/osk/activeLayout.ts b/web/source/osk/activeLayout.ts
index 4193dcc0cf..b9125bc730 100644
--- a/web/source/osk/activeLayout.ts
+++ b/web/source/osk/activeLayout.ts
@@ -47,11 +47,8 @@ namespace com.keyman.osk {
         }
       }
 
-      // Calculate actual key widths by summing defined widths and scaling each key to %,
-      // adjusting the width of the last key to make the total exactly 100%
+      // Calculate percentage-based scalings by summing defined widths and scaling each key to %.
       // Save each percentage key width as a separate member (do *not* overwrite layout specified width!)
-      // NB: the 'percent' suffix is historical, units are percent on desktop devices, but pixels on touch devices
-      // All key widths and paddings are rounded for uniformity
       var keyPercent: number, padPercent: number, totalPercent=0;
       for(let j=0; j<keys.length-1; j++) {
         keyPercent=parseInt(keys[j]['width'],10)/totalWidth;
diff --git a/web/source/osk/visualKeyboard.ts b/web/source/osk/visualKeyboard.ts
index 39fe10e415..230b0edf8c 100644
--- a/web/source/osk/visualKeyboard.ts
+++ b/web/source/osk/visualKeyboard.ts
@@ -762,9 +762,9 @@ namespace com.keyman.osk {
           // Apply defaults, setting the width and other undefined properties for each key
           keys=row['key'];
 
-          // Calculate actual key widths by summing defined widths and scaling each key to %,
-          // adjusting the width of the last key to make the total exactly 100%
-          // Save each percentage key width as a separate member (do *not* overwrite layout specified width!)
+          // Calculate actual key widths by multiplying by the OSK's width and rounding appropriately,
+          // adjusting the width of the last key to make the total exactly 100%.
+          // Overwrite the previously-computed percent.
           // NB: the 'percent' suffix is historical, units are percent on desktop devices, but pixels on touch devices
           // All key widths and paddings are rounded for uniformity
           var keyPercent: number, padPercent: number, totalPercent=0;