Tweaks to checkUser code to reduce unnecessary signin boxes. two more icons

Author: Tim Berners-Lee

lib/icons/noun_19460_green.svg

@@ -133,7 +133,7 @@ UI.widgets.loadPreferences = function(context) {
         var pending;
         if (box) {
             pending = UI.widgets.errorMessageBlock(context.dom,
-                "(loading preferences " + preferencesFile+ ")");
+                "(loading preferences " + preferencesFile+ ")", "straw");
             box.appendChild(pending);
         }
         kb.fetcher.load(preferencesFile).then(function(xhr){
@@ -142,7 +142,7 @@ UI.widgets.loadPreferences = function(context) {
             }
             resolve(context);
         }).catch(function(err){
-            reject("Error loading preferences file" + err)
+            reject("Error loading preferences file. " + err)
         });
     });
 };
@@ -529,13 +529,17 @@ UI.widgets.userCheckSite = 'https://databox.me/';
 
 
 UI.widgets.checkUser = function(doc, setIt) {
+    var kb = UI.store
 
     if (typeof $SolidTestEnvironment !== 'undefined' && $SolidTestEnvironment.username){ // Test setup
+      tabulator.preferences.set('me', $SolidTestEnvironment.username)
       return setIt($SolidTestEnvironment.username)
     }
+    var me_uri = tabulator.preferences.get('me')
+    if (me_uri) return setIt(me_uri)
+
     var userMirror = UI.store.any(doc, UI.ns.link('userMirror'));
     if (!userMirror) userMirror = doc;
-    var kb = UI.store
     kb.fetcher.nowOrWhenFetched(userMirror.uri, undefined, function(ok, body) {
         if (!ok) {
           var message = "checkUser: Unable to load " + userMirror.uri + ": " + body;

lib/icons/noun_45961.svg

@@ -934,7 +934,7 @@ widgetModule.fieldLabel = function (dom, property, form) {
 
 widgetModule.errorMessageBlock = function (dom, msg, backgroundColor) {
   var div = dom.createElement('div')
-  div.setAttribute('style', 'padding: 0.5em; border: 0.5px solid black; background-color: ' +
+  div.setAttribute('style', 'margin: 0.1em; padding: 0.5em; border: 0.05em solid gray; background-color: ' +
     (backgroundColor || '#fee') + '; color:black;')
   div.textContent = msg
   return div