ui; add note about Session; allow clicking on table header not just link

Author: gadicc

msgfmt:ui/lib/client.js

@@ -1,3 +1,11 @@
+/*
+ * NOTE ON USE OF SESSION.SET() ETC
+ *   The messageformat project is around since Meteor 0.5, when this
+ *   was recommended practice.  There are better ways to do this now,
+ *   however, with the way things are going, we're going to need to
+ *   rewrite this all with React soon, so it's not a priority.
+ */
+
 // TODO, if we add a new lang, need to update mfPkg.timestamps
 
 // Setup in msgfmt:core on server, only used on the client in msgfmt:ui
@@ -265,13 +273,13 @@ Template.mfTransLang.helpers({
     return Session.get('caseInsensitiveOrdering');
   },
   sortOrderHeaderClass: function(headerSortField) {
-    var classes = '';
+    var classes = 'translationSort';
     var sortField = Session.get('translationSortField');
     var sortOrder = Session.get('translationSortOrder');
     if (headerSortField === sortField) {
       classes += ' active-sort ' + sortOrder;
     }
-    return classes
+    return classes;
   },
   stateClass: function() {
     if (this.fuzzy)

msgfmt:ui/lib/ui.html

@@ -74,15 +74,15 @@ <h2>{{mf 'mf_site_translations' 'Site Translations'}}</h2>
         <thead>
           <tr>
             {{#if showKey}}
-              <th class="{{sortOrderHeaderClass 'key'}}">
-                <a href="#" class='translationSort' data-sortField="key">Key</a>
+              <th class="{{sortOrderHeaderClass 'key'}}" data-sortField="key">
+                <a href="#">Key</a>
               </th>
             {{/if}}
-            <th class="{{sortOrderHeaderClass 'orig'}}">
-              <a href="#" class="translationSort" data-sortField="orig">Original String ({{origLang}})</a>
+            <th class="{{sortOrderHeaderClass 'orig'}}" data-sortField="orig">
+              <a href="#">Original String ({{origLang}})</a>
             </th>
-            <th class="{{sortOrderHeaderClass 'trans'}}">
-              <a href="#" class="translationSort" data-sortField="trans">Translation ({{destLang}})</a>
+            <th class="{{sortOrderHeaderClass 'trans'}}" data-sortField="trans">
+              <a href="#">Translation ({{destLang}})</a>
             </th>
           </tr>
         </thead>