messagegui: wip small improvements

Author: thyttan

apps/messagegui/app.js

@@ -281,6 +281,7 @@ function showMessageScroller2(msg, persist, initAtEndOfMsg, alreadyProcessed) {
   //  alreadyProcessed = { lines : [], startIdx : undefined, stopIdx : undefined };
   //}
 
+  //msg.new = false;
   require("widget_utils").hide();
 
   if (alreadyProcessed) print("alreadyPeocessed start and stop idx: ",alreadyProcessed.startIdx, alreadyProcessed.stopIdx);
@@ -325,6 +326,7 @@ function showMessageScroller2(msg, persist, initAtEndOfMsg, alreadyProcessed) {
   var messagesWrapped = [];
   for (let j=startIdx; j<stopIdx; j++) {
     let msgLocal = MESSAGES[j];
+    msgLocal.new = false;
 
     if (msgLocal.id=="music" || msgLocal.source=="maps" || msgLocal.id =="call") {
       stopIdx++
@@ -341,7 +343,7 @@ function showMessageScroller2(msg, persist, initAtEndOfMsg, alreadyProcessed) {
     }
     var titleCnt = lines.length;
     if (titleCnt) lines.push(""); // add blank line after title
-    lines = lines.concat(g.wrapString(msgLocal.body, g.getWidth()-10), [ "------" ]);
+    lines = lines.concat(g.wrapString(msgLocal.body, g.getWidth()-10), [ "---------" ]);
     messagesWrapped.push(lines);
     //print(lines);
     //  allLines = allLines.concat(lines);
@@ -361,14 +363,13 @@ function showMessageScroller2(msg, persist, initAtEndOfMsg, alreadyProcessed) {
   // }
 
   print("titlelines", titleLines)
-  var initScroll;
+  var initScroll = messagesWrapped[0].length;
   if (alreadyProcessed===undefined){
-    initScroll = messagesWrapped[0].length;
+    if (msgIdx==0) initScroll = 0;
   } else if (msgIdx <= alreadyProcessed.startIdx) {
-    initScroll = messagesWrapped[0].length;
     titleLines = titleLines.concat(alreadyProcessed.titleLines.map((x) => x + allLines2.length));
   } else if (msgIdx >= alreadyProcessed.stopIdx-1){
-    initScroll = alreadyProcessed.lines.length-8;
+    initScroll = alreadyProcessed.lines.length-9;
     titleLines = alreadyProcessed.titleLines.concat(titleLines.map((x) => x + alreadyProcessed.lines.length));
   }
 
@@ -724,7 +725,7 @@ function checkMessages(options) {
   if (!options.ignoreUnread && newMessages.length) {
     delete newMessages[0].show; // stop us getting stuck here if we're called a second time
     //showMessage(newMessages[0].id, false);
-    showMessageScroller2(MESSAGES[2], false, false);
+    showMessageScroller2(newMessages[0], false, false);
     // buzz after showMessage, so being busy during layout doesn't affect the buzz pattern
     if (global.BUZZ_ON_NEW_MESSAGE) {
       // this is set if we entered the messages app by loading `messagegui.new.js`