Tweaks to the injector UI

pkg/polymer/lib/src/build/log_injector.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /// This library provides a single function called injectLogs which when called
 /// will request a logs json file and build a small widget out of them which
 /// groups the logs by level.
 library polymer.build.log_injector;
 
 import 'dart:async';
 import 'dart:convert';
 import 'dart:html';
 
 import 'package:path/path.dart' as path;
 
 class LogInjector {
   Element selectedMenu;
   Element selectedContent;
 
   // Gets the logs from a url and inject them into the dom.
-  Future injectLogsFromUrl([String url]) {
-    if (url == null) url = '${Uri.base.path}._buildLogs';
-    return HttpRequest.getString(url).then((data) => injectLogs(data));
-  }
+  Future injectLogsFromUrl(String url) =>
+      HttpRequest.getString(url).then((data) => injectLogs(data));
 
   // Builds the html for the logs element given some logs, and injects that
   // into the dom. Currently, we do not  use Polymer just to ensure that the
   // page works regardless of the state of the app. Ideally, we could have
   // multiple scripts running independently so we could ensure that this would
   // always be running.
   injectLogs(String data) {
     // Group all logs by level.
     var logsByLevel = {
     };
@@ skipping 44 matching lines @@
         }
 
         menuItem.classes.toggle('active');
         contentItem.classes.toggle('active');
       });
 
       // Add the logs to the content item.
       for (var log in logs) {
         var logHtml = new StringBuffer();
         logHtml.write('<div class="log">');
-        logHtml.write(
-            '<div class="message $levelClassName">${log['message']}</div>');
+        var message = log['message'].replaceAllMapped(_urlRegex,
+            (m) => '<a href="${m.group(0)}" target="blank">${m.group(0)}</a>');
+        logHtml.write('<div class="message $levelClassName">$message</div>');
         var assetId = log['assetId'];
+        var span = log['span'];
+        bool hasLocation = assetId != null || span != null;
+        if (hasLocation) logHtml.write('<div class="location">');
         if (assetId != null) {
           logHtml.write(
-              '<div class="asset">'
-              '  <span class="package">${assetId['package']}</span>:'
-              '  <span class="path">${assetId['path']}</span>''</div>');
+              '  <span class="package">${assetId['package']}</span>:');
+          if (span == null) {
+            logHtml.write('  <span class="location">${assetId['path']}</span>');
+          }
         }
-        var span = log['span'];
         if (span != null) {
           logHtml.write(
-              '<div class="span">'
-              '  <div class="location">${span['location']}</div>'
-              '  <code class="text">${span['text']}</code>''</div>');
+              '  <span class="location">${span['location']}</span></div>'
+              '  <span class="text">${span['text']}</span>''</div>');

[Siggi Cherem (dart-lang), 2014/08/08 00:40:45]

in the end, I switch to span -- when I was checking this in other apps, I run it
in an app that used bootstrap's css. It looked nice, but it totally changed how
the <code> block looked like. This way we have more control over it.

+        } else if (hasLocation) {
+          logHtml.write('</div>');
         }
         logHtml.write('</div>');
 
-        contentItem.append(new Element.html(logHtml.toString()));
+        var logElement = new Element.html(logHtml.toString(),
+            validator: new NodeValidatorBuilder.common()
+              ..allowNavigation(new _OpenUriPolicy()));

[Siggi Cherem (dart-lang), 2014/08/08 00:40:45]

otherwise `new Element.html` sanitizes the html and deletes the 'href'  :(

+        contentItem.append(logElement);
+        var messageElement = logElement.querySelector('.message');
+        messageElement.onClick.listen((e) {
+          if (e.target == messageElement) {
+            messageElement.classes.toggle('expanded');
+          }
+        });
       };
     });
 
     document.body.append(wrapperDiv);
   }
 
-}
+}
+
+final _urlRegex = new RegExp('http://[^ ]*');
+class _OpenUriPolicy implements UriPolicy {
+  bool allowsUri(String uri) => true;
+}