minor code review: render domain= option as the last one

gorhill · Aug 30, 2016 · c3582e1 · c3582e1
1 parent 7073737
commit c3582e1
Showing 1 changed file with 17 additions and 16 deletions.
diff --git a/src/js/logger-ui.js b/src/js/logger-ui.js
@@ -195,6 +195,23 @@ var filterDecompiler = (function() {
         var tfields = fid !== '.' ? vfields[3].split('\t') : [];
         var tfield0 = tfields[0];
 
+        // Filter options
+        // Importance
+        if ( bits & 0x02 ) {
+            opts.push('important');
+        }
+        // Party
+        if ( bits & 0x08 ) {
+            opts.push('third-party');
+        } else if ( bits & 0x04 ) {
+            opts.push('first-party');
+        }
+        // Type
+        var typeVal = bits >>> 4 & 0x0F;
+        if ( typeVal ) {
+            opts.push(typeValToTypeName[typeVal]);
+        }
+
         switch ( fid ) {
         case '.':
             filter += '||' + vfields[2] + '^';
@@ -258,22 +275,6 @@ var filterDecompiler = (function() {
             break;
         }
 
-        // Filter options
-        // Importance
-        if ( bits & 0x02 ) {
-            opts.push('important');
-        }
-        // Party
-        if ( bits & 0x08 ) {
-            opts.push('third-party');
-        } else if ( bits & 0x04 ) {
-            opts.push('first-party');
-        }
-        // Type
-        var typeVal = bits >>> 4 & 0x0F;
-        if ( typeVal ) {
-            opts.push(typeValToTypeName[typeVal]);
-        }
         if ( opts.length !== 0 ) {
             filter += '$' + opts.join(',');
         }