Web components based app to view dart docs. Still has rough edges.

client/web/model.dart

 library apidoc_model;
 
 import 'package:web_ui/watcher.dart' as watchers;
 import 'package:web_ui/safe_html.dart';
 import 'markdown.dart' as md;
 import 'dart:html' as html;
 import 'dart:json';
 import 'ast.dart';
 
+// TODO(jacobr): specify the version # in the JSON file.

[Siggi Cherem (dart-lang), 2012/12/19 19:47:33]

+1 :)

[Jacob, 2013/01/02 19:54:58]

acknowledged :)

 String svnRevisionNumber = "15605";
 
-String _activeReferenceId;
+/**
+ * Reference id of [currentElement].
+ * *

[Siggi Cherem (dart-lang), 2012/12/19 19:47:33]

remove extra *

[Jacob, 2013/01/02 19:54:58]

rm * 

:)

+ * Stored in addition to [currentElement] as [currentElement] may
+ * not yet be available if the data model for the library it is part of has
+ * not yet been loaded.
+ */
+String _currentReferenceId;
 
-/// Current state of the application.
+/**
+ * Current library the user is browsing if any.
+ */
 LibraryElement currentLibrary;
+
+/**
+ * Current type the user is viewing if any.
+ * Should be either a [ClassElement] or a [TypedefElement].
+ */
 Element currentType;
+/**
+ * Current member of either [currentLibrary] or [currentType] that the user is
+ * viewing.
+ */
 Element currentMember;
+
+/**
+ * Element corresponding to [_currentReferenceId].
+ * The most specific element of [currentLibrary]. [currentType], and
+ * [currentMember].
+ */
 Element currentElement;
 
+/**
+ * Recomputes the Elements part of the current active state from the data model.
+ *
+ * This method should be invoked after additional libraries are loaded from the
+ * server or after the user navigates to a different element in the UI.
+ */
 void _recomputeActiveState() {
   currentLibrary = null;
   currentType = null;
   currentMember = null;
   currentElement = null;
-  if (_activeReferenceId != null) {
-    var path = lookupReferenceId(_activeReferenceId).path;
-
-    if (path.length > 0) {
-      currentLibrary = path[0];
-    }
-    if (path.length > 1) {
-      if (path[1] is ClassElement || path[1] is TypedefElement) {
-        currentType = path[1];
-        if (path.length > 2) {
-          currentMember = path[2];
+  if (_currentReferenceId != null) {
+    var referenceElement = lookupReferenceId(_currentReferenceId);
+    if (referenceElement != null) {
+      var path = referenceElement.path;
+      if (path.length > 0) {
+        currentLibrary = path[0];
+      }
+      if (path.length > 1) {
+        if (path[1] is ClassElement || path[1] is TypedefElement) {
+          currentType = path[1];
+          if (path.length > 2) {
+            currentMember = path[2];
+          }
+        } else {
+          currentMember = path[1];
         }
+      }
+      if (currentMember != null) {
+        currentElement = currentMember;
+      } else if (currentType != null) {
+        currentElement = currentType;
       } else {
-        currentMember = path[1];
+        currentElement = currentLibrary;
       }
     }
-    if (currentMember != null) {
-      currentElement = currentMember;
-    } else if (currentType != null) {
-      currentElement = currentType;
-    } else {
-      currentElement = currentLibrary;
-    }
   }
 }
 
+/**
+ * Scrolls the [currentElement] into view.
+ */
 void scrollIntoView() {
   // TODO(jacobr): there should be a cleaner way to run code that executes
   // after the UI updates.

[Siggi Cherem (dart-lang), 2012/12/19 19:47:33]

let's add an issue for this in web-ui.

[Jacob, 2013/01/02 19:54:58]

Referenced https://github.com/dart-lang/web-ui/issues/188 in the comment.

[Siggi Cherem (dart-lang), 2013/01/02 21:40:55]

Thanks! Sorry that I asked you to filed it and then marked it as a duplicate.
Somehow I didn't realize that this was already filed until you filed it and I
searched for other similar bugs =(.

   html.window.setTimeout(() {
     if (currentElement != null) {
       for (var e in html.queryAll('[data-id="${currentElement.id}"]')) {
         e.scrollIntoView(false);
       }
     }
   }, 0);
 }
 
-onDataModelChanged() {
+/**
+ * Invoke every time the data model changes.
+ */
+void _onDataModelChanged() {
   _recomputeActiveState();
   scrollIntoView();
 }
 
 /**
  * Generate a CSS class given an element that may be a class, member, method,
  * etc.
  */
 String kindCssClass(Element element) {
-  String classes = 'kind kind-${normalizedKind(element)}';
+  String classes = 'kind kind-${_normalizedKind(element)}';
   if (element.isPrivate == true) {
     classes = '$classes private';
   } else if (element is MethodElementBase && element.isStatic) {
     classes = '$classes static';
   }
 
   // Setters are viewed as methods by the AST.
   if (element is PropertyElement) {
     classes = '$classes getter';
   }
 
   if (element is MethodElementBase && element.isSetter) {
     classes = '$classes setter';
   }
 
   return classes;
 }
 
-String normalizedKind(obj) {
-  if (obj is Element) return normalizedKindFromElement(obj);
-  return obj;
-}
-
-String normalizedKindFromElement(Element element) {
+String _normalizedKind(Element element) {
   var kind = element.kind;
   var name = element.name;
   if (kind == 'method' && element.isOperator) {
     kind = 'operator';
   }
   // TODO(jacobr): this is horrible but matches what DartDoc does
   if (kind == 'class' && name.endsWith('Exception')) {
     kind = 'exception';
   }
   return kind;
 }
 
 String toUserVisibleKind(Element element) {
-  return KIND_TITLES[normalizedKind(element)];
+  return KIND_TITLES[_normalizedKind(element)];
 }
 
 /**
  * [obj] shoudl be a [Reference] or [Element].
  */
 String permalink(var obj) {
   var data = {'id': obj.refId};
+  // TODO(jacobr): evaluate whether the persistent UI state will stay just a
+  // single reference ID in which case this is overkill.
   return "#!${JSON.stringify(data)}";
 }
 
 void loadStateFromUrl() {
   String link = html.window.location.hash;
   var data = {};
   if (link.length > 2) {
     try {
       // strip #! and parse json.
       data = JSON.parse(link.substring(2));
     } catch (e) {
       html.window.console.error("Invalid link url");
       // TODO(jacobr): redirect to default page or better yet attempt to fixup.
     }
   }
-  _activeReferenceId = data['id'];
+  _currentReferenceId = data['id'];
   _recomputeActiveState();
   scrollIntoView();
 }
 
 Future loadModel() {
   html.window.on.popState.add((e) {

[Siggi Cherem (dart-lang), 2012/12/19 19:47:33]

in our todomvc sample we had to listen for both popState and hashChange to
ensure thigns work in IE9 and Opera 12. 

See http://code.google.com/p/dart/issues/detail?id=5483

[Siggi Cherem (dart-lang), 2013/01/02 21:40:55]

you might have missed this comment...

[Jacob, 2013/01/03 00:09:33]

I did... done :)

     loadStateFromUrl();
     watchers.dispatch();
   });
 
   // Patch in support for [:...:]-style code to the markdown parser.
   // TODO(rnystrom): Markdown already has syntax for this. Phase this out?
   md.InlineParser.syntaxes.insertRange(0, 1,
       new md.CodeSyntax(r'\[\:((?:.|\n)*?)\:\]'));
 
   md.setImplicitLinkResolver(_resolveNameReference);
   var completer = new Completer();
   // TODO(jacobr): shouldn't have to get this from the parent directory.
-  new html.HttpRequest.get('../static/apidoc.json', onSuccess(html.HttpRequest req) {
+  new html.HttpRequest.get('../static/apidoc.json', (req) {
     for (var libraryJson in JSON.parse(req.responseText)) {
       var library = new LibraryElement(libraryJson, null);
       libraries[library.id] = library;
     }
-    onDataModelChanged();
+    _onDataModelChanged();
     completer.complete(true);
   });
   return completer.future;
 }
 
-/** XXX NOT USED TODO(jacobr) remove.
-class DocComment {
-  final String text;
-
-  /**
-   * Non-null if the comment is inherited from another declaration.
-   */
-  final inheritedFrom; // InterfaceMirror?
-
-  DocComment(this.text, [this.inheritedFrom = null]) {
-    assert(text != null && !text.trim().isEmpty);
-  }
-
-  SafeHtml get html => new SafeHtml.unsafe(md.markdownToHtml(text));
-
-  String toString() => text;
-}
-
-*/
-
+// TODO(jacobr): remove this method and resolve refences to types in the json
+// generation. That way the existing correct logic in Dart2Js can be used rather
+// than this rather busted logic.
 /**
  * This will be called whenever a doc comment hits a `[name]` in square
  * brackets. It will try to figure out what the name refers to and link or
  * style it appropriately.
  */
 md.Node _resolveNameReference(String name) {
   // TODO(jacobr): this isn't right yet and we have made this code quite ugly
   // by using the verbose universal permalink member even though library is
   // always currentLibrary.
   makeLink(String href) {
     return new md.Element.text('a', name)
       ..attributes['href'] = href
       ..attributes['class'] = 'crossref';
   }
 
   // See if it's a parameter of the current method.
   if (currentMember != null && currentMember.kind == 'method') {
     var parameters = currentMember.children;
     for (final parameter in parameters) {
       if (parameter.name == name) {
         final element = new md.Element.text('span', name);
         element.attributes['class'] = 'param';
         return element;
       }
     }
   }
 
   // See if it's another member of the current type.
-  // TODO(jacobr): fixme. this is wrong... members are by id now not simple string name...
+  // TODO(jacobr): fixme. this is wrong... members are by id now not simple
+  // string name...
   if (currentType != null) {
     final foundMember = currentType.members[name];
     if (foundMember != null) {
       return makeLink(permalink(foundMember));
     }
   }
 
   // See if it's another type or a member of another type in the current
   // library.
   if (currentLibrary != null) {
@@ skipping 38 matching lines @@
     }
   }
 
   // TODO(jacobr): Should also consider:
   // * Names imported by libraries this library imports. Don't think we even
   //   store this in the AST.
   // * Type parameters of the enclosing type.
 
   return new md.Element.text('code', name);
 }