Fixes from review.

src/lib/annotations/annotations.html

@@ -90,6 +90,10 @@
 
     _bindingRegex: /([^{[]*)({{|\[\[)([^}\]]*)(?:]]|}})/g,
 
+    // TODO(kschaaf): We could modify this to allow an escape mechanism by
+    // looking for the escape sequence in each of the matches and converting
+    // the part back to a literal type, and then bailing if only literals
+    // were found
     _parseBindings: function(text) {
       var re = this._bindingRegex;
       var parts = [];
@@ -114,7 +118,7 @@
         var negate = false;
         if (value[0] == '!') {
           negate = true;
-          value = value.substring(1);
+          value = value.substring(1).trim();
         }
         var customEvent, notifyEvent, colon;
         if (mode == '{' && (colon = value.indexOf('::')) > 0) {
@@ -277,25 +281,25 @@
     },
 
     // construct annotation data from a generic attribute, or undefined
-    _parseNodeAttributeAnnotation: function(node, n, v) {
-      var parts = this._parseBindings(v);
+    _parseNodeAttributeAnnotation: function(node, name, value) {
+      var parts = this._parseBindings(value);
       if (parts) {
         // Attribute or property
-        var name = n;
+        var origName = name;
         var kind = 'property';
-        if (n[n.length-1] == '$') {
-          name = n.slice(0, -1);
+        if (name[name.length-1] == '$') {
+          name = name.slice(0, -1);
           kind = 'attribute';
         }
         // Clear attribute before removing, since IE won't allow removing
         // `value` attribute if it previously had a value (can't
         // unconditionally set '' before removing since attributes with `$`
         // can't be set using setAttribute)
-        if (node.localName == 'input' && n == 'value') {
-          node.setAttribute(n, '');
+        if (node.localName == 'input' && name == 'value') {
+          node.setAttribute(origName, '');
         }
         // Remove annotation
-        node.removeAttribute(n);
+        node.removeAttribute(origName);
         // Case hackery: attributes are lower-case, but bind targets
         // (properties) are case sensitive. Gambit is to map dash-case to
         // camel-case: `foo-bar` becomes `fooBar`.

src/lib/bind/effects.html

@@ -15,6 +15,7 @@
     _shouldAddListener: function(effect) {
       return effect.name &&
              effect.kind != 'attribute' &&
+             effect.kind != 'text' &&
              !effect.isCompound &&
              effect.parts[0].mode === '{' &&
              !effect.parts[0].negate;