dce-reactkit 3.2.2-beta.3 → 3.2.2-beta.31

package/src/components/LogReviewer.tsx

@@ -21,7 +21,7 @@ import {
 // Import shared helpers
 import visitServerEndpoint from '../helpers/visitServerEndpoint';
 import getTimeInfoInET from '../helpers/getTimeInfoInET';
-import { showFatalError } from './AppWrapper';
+import { alert, showFatalError } from './AppWrapper';
 
 // Import shared constants
 import LOG_REVIEW_ROUTE_PATH_PREFIX from '../constants/LOG_REVIEW_ROUTE_PATH_PREFIX';
@@ -34,6 +34,7 @@ import LogType from '../types/LogType';
 import LogAction from '../types/LogAction';
 import ParamType from '../types/ParamType';
 import IntelliTableColumn from '../types/IntelliTableColumn';
+import LogBuiltInMetadata from '../types/LogBuiltInMetadata';
 
 // Import shared components
 import SimpleDateChooser from './SimpleDateChooser';
@@ -121,7 +122,7 @@ type ContextFilterState = {
 
 // Tag filter state
 type TagFilterState = {
-  [k: string]: boolean
+  [k: string]: boolean // tag => true if in the list of tags to show
 };
 
 // Action filter state (only relevant for action logs)
@@ -206,6 +207,11 @@ const style = `
     border: 0.05rem solid black;
     border-radius: 0.5rem;
     overflow: hidden;
+    padding: 0.7rem;
+
+    /* Solid background */
+    background-color: white;
+    color: black;
 
     /* Place contents in flex column */
     flex-direction: column;
@@ -232,6 +238,27 @@ const style = `
     /* Vertical scroll */
     overflow-y: auto;
   }
+
+  .LogReviewer-header-close-button {
+    border: 0 !important;
+    background-color: transparent !important;
+    padding-top: 0 !important;
+    padding-bottom: 0 !important;
+    padding-right: 1em !important;
+    margin: 0 !important;
+    color: #444 !important;
+
+    right: 0 !important;
+    position: absolute !important;
+  }
+  .LogReviewer-header-close-button:hover {
+    border: 0 !important;
+    background-color: transparent !important;
+    padding-top: 0 !important;
+    padding-bottom: 0 !important;
+    margin: 0 !important;
+    color: #000 !important;
+  }
 `;
 
 /*------------------------------------------------------------------------*/
@@ -254,7 +281,7 @@ const genHumanReadableName = (machineReadableName: string) => {
       // Uppercase! Add a space before
       humanReadableName += ' ';
     }
-    humanReadableName += chars;
+    humanReadableName += char;
   });
 
   // Trim and return
@@ -437,9 +464,25 @@ const reducer = (state: State, action: Action): State => {
       };
     }
     case ActionType.UpdateTagFilterState: {
+      const { tagFilterState } = action;
+
+      // Select all if every tag is deselected
+      const numTagsSelected = (
+        Object.values(tagFilterState)
+          .filter((isSelected) => {
+            return isSelected;
+          })
+          .length
+      );
+      if (numTagsSelected === 0) {
+        Object.keys(tagFilterState).forEach((tag) => {
+          tagFilterState[tag] = true;
+        });
+      }
+
       return {
         ...state,
-        tagFilterState: action.tagFilterState,
+        tagFilterState,
       };
     }
     case ActionType.UpdateActionErrorFilterState: {
@@ -477,6 +520,35 @@ const LogReviewer: React.FC<Props> = (props) => {
     onClose,
   } = props;
 
+  // Add built-in LogMetadata
+  // > Add "uncategorized" subcontext to each context
+  Object.keys(LogMetadata.Context ?? {}).forEach((context) => {
+    if (
+      // Context exists
+      LogMetadata.Context
+      // Context has children already
+      && typeof LogMetadata.Context[context] !== 'string'
+    ) {
+      (LogMetadata.Context as any)[context][LogBuiltInMetadata.Context.Uncategorized] = (
+        LogBuiltInMetadata.Context.Uncategorized
+      );
+    }
+  })
+  // > Add built-in contexts
+  LogMetadata.Context = (LogMetadata.Context ?? {});
+  Object.keys(LogBuiltInMetadata.Context).forEach((context) => {
+    if (LogMetadata.Context) {
+      LogMetadata.Context[context] = context;
+    }
+  });
+  // > Add built-in targets
+  LogMetadata.Target = (LogMetadata.Target ?? {});
+  Object.keys(LogBuiltInMetadata.Target).forEach((target) => {
+    if (LogMetadata.Target) {
+      LogMetadata.Target[target] = target;
+    }
+  });
+
   /* -------------- State ------------- */
 
   // Create initial date filter state
@@ -501,14 +573,19 @@ const LogReviewer: React.FC<Props> = (props) => {
   Object.keys(LogMetadata.Context ?? {}).forEach((context) => {
     const contextValue = (LogMetadata.Context ?? {})[context];
     if (typeof contextValue === 'string') {
-      // Case: no subcontexts
+      // Case: no subcontexts, init as checked
       initContextFilterState[contextValue] = true;
     } else {
       // Case: subcontexts exist
-      initContextFilterState[contextValue._] = {};
+      initContextFilterState[context] = {};
       Object.values((LogMetadata.Context ?? {})[context]).forEach((subcontext) => {
-        const subcontextValue = contextValue[subcontext];
-        (initContextFilterState[contextValue._] as { [k: string]: boolean })[subcontextValue] = true;
+        // Skip self ("_")
+        if (subcontext === '_') {
+          return;
+        }
+
+        // Initialize as checked
+        (initContextFilterState[context] as any)[subcontext] = true;
       });
     }
   });
@@ -516,7 +593,7 @@ const LogReviewer: React.FC<Props> = (props) => {
   // Create initial tag filter state
   const initTagFilterState: TagFilterState = {};
   Object.values(LogMetadata.Tag ?? {}).forEach((tagValue) => {
-    initTagFilterState[tagValue] = true;
+    initTagFilterState[tagValue] = false;
   });
 
   // Create advanced filter state
@@ -550,6 +627,10 @@ const LogReviewer: React.FC<Props> = (props) => {
   Object.values(LogAction).forEach((action) => {
     initActionErrorFilterState.action[action] = true;
   });
+  // Add built-in targets
+  Object.values(LogBuiltInMetadata.Target).forEach((target) => {
+    initActionErrorFilterState.target[target] = true;
+  });
 
   // Initial state
   const initialState: State = {
@@ -600,18 +681,23 @@ const LogReviewer: React.FC<Props> = (props) => {
     let month = newDateFilterState.startDate.month;
     while (
       // Earlier year
-      (year <= newDateFilterState.endDate.year)
+      (year < newDateFilterState.endDate.year)
       // Current year but included month
       || (
         year === newDateFilterState.endDate.year
         && month <= newDateFilterState.endDate.month
       )
     ) {
-      // Add to list
-      toLoad.push({
-        year,
-        month,
-      });
+      // Add to list if not already loaded
+      if (
+        !logMap[year]
+        || !logMap[year][month]
+      ) {
+        toLoad.push({
+          year,
+          month,
+        });
+      }
       
       // Increment
       month += 1;
@@ -641,6 +727,7 @@ const LogReviewer: React.FC<Props> = (props) => {
 
     // Check which year/month combos we need to load
     const toLoad = listMonthsToLoad(newDateFilterState);
+    console.log('Filter update:', newDateFilterState, toLoad, logMap);
 
     // If nothing to load, finished
     if (toLoad.length === 0) {
@@ -727,16 +814,16 @@ const LogReviewer: React.FC<Props> = (props) => {
 
     // Filter toggle
     const filterToggles = (
-      <div className="LogReviewer-filter-toggles d-flex align-items-center justify-content-center">
+      <div className="LogReviewer-filter-toggles">
         <h3 className="m-0">
           Filters:
         </h3>
-        <div className="LogReviewer-filter-toggle-buttons">
+        <div className="LogReviewer-filter-toggle-buttons alert alert-secondary p-2 m-0">
           {/* Date */}
           <button
             type="button"
             id="LogReviewer-toggle-date-filter-drawer"
-            className={`btn btn-${FilterDrawer.Date === expandedFilterDrawer} me-2`}
+            className={`btn btn-${FilterDrawer.Date === expandedFilterDrawer ? 'warning' : 'light'} me-2`}
             aria-label="toggle date filter drawer"
             onClick={() => {
               dispatch({
@@ -755,7 +842,7 @@ const LogReviewer: React.FC<Props> = (props) => {
           <button
             type="button"
             id="LogReviewer-toggle-context-filter-drawer"
-            className={`btn btn-${FilterDrawer.Context === expandedFilterDrawer} me-2`}
+            className={`btn btn-${FilterDrawer.Context === expandedFilterDrawer ? 'warning' : 'light'} me-2`}
             aria-label="toggle context filter drawer"
             onClick={() => {
               dispatch({
@@ -771,29 +858,32 @@ const LogReviewer: React.FC<Props> = (props) => {
             Context
           </button>
           {/* Tag */}
-          <button
-            type="button"
-            id="LogReviewer-toggle-tag-filter-drawer"
-            className={`btn btn-${FilterDrawer.Tag === expandedFilterDrawer} me-2`}
-            aria-label="toggle tag filter drawer"
-            onClick={() => {
-              dispatch({
-                type: ActionType.ToggleFilterDrawer,
-                filterDrawer: FilterDrawer.Tag,
-              });
-            }}
-          >
-            <FontAwesomeIcon
-              icon={faTag}
-              className="me-2"
-            />
-            Tag
-          </button>
+          {/* Skip if no tags are used */}
+          {(LogMetadata.Tag && Object.keys(LogMetadata.Tag).length > 0) && (
+            <button
+              type="button"
+              id="LogReviewer-toggle-tag-filter-drawer"
+              className={`btn btn-${FilterDrawer.Tag === expandedFilterDrawer ? 'warning' : 'light'} me-2`}
+              aria-label="toggle tag filter drawer"
+              onClick={() => {
+                dispatch({
+                  type: ActionType.ToggleFilterDrawer,
+                  filterDrawer: FilterDrawer.Tag,
+                });
+              }}
+            >
+              <FontAwesomeIcon
+                icon={faTag}
+                className="me-2"
+              />
+              Tag
+            </button>
+          )}
           {/* Action */}
           <button
             type="button"
             id="LogReviewer-toggle-action-filter-drawer"
-            className={`btn btn-${FilterDrawer.Action === expandedFilterDrawer} me-2`}
+            className={`btn btn-${FilterDrawer.Action === expandedFilterDrawer ? 'warning' : 'light'} me-2`}
             aria-label="toggle action and error filter drawer"
             onClick={() => {
               dispatch({
@@ -812,7 +902,7 @@ const LogReviewer: React.FC<Props> = (props) => {
           <button
             type="button"
             id="LogReviewer-toggle-advanced-filter-drawer"
-            className={`btn btn-${FilterDrawer.Advanced === expandedFilterDrawer}`}
+            className={`btn btn-${FilterDrawer.Advanced === expandedFilterDrawer ? 'warning' : 'light'}`}
             aria-label="toggle advanced filter drawer"
             onClick={() => {
               dispatch({
@@ -843,14 +933,11 @@ const LogReviewer: React.FC<Props> = (props) => {
               year={dateFilterState.startDate.year}
               month={dateFilterState.startDate.month}
               day={dateFilterState.startDate.day}
+              chooseFromPast
+              numMonthsToShow={12}
               onChange={(month, day, year) => {
-                dispatch({
-                  type: ActionType.UpdateDateFilterState,
-                  dateFilterState: {
-                    ...dateFilterState,
-                    startDate: { month, day, year },
-                  },
-                });
+                dateFilterState.startDate = { month, day, year };
+                handleDateRangeUpdated(dateFilterState);
               }}
             />
             {' '}
@@ -862,14 +949,28 @@ const LogReviewer: React.FC<Props> = (props) => {
               year={dateFilterState.endDate.year}
               month={dateFilterState.endDate.month}
               day={dateFilterState.endDate.day}
+              chooseFromPast
+              numMonthsToShow={12}
               onChange={(month, day, year) => {
-                dispatch({
-                  type: ActionType.UpdateDateFilterState,
-                  dateFilterState: {
-                    ...dateFilterState,
-                    endDate: { month, day, year },
-                  },
-                });
+                if (
+                  year < dateFilterState.startDate.year
+                  || (
+                    year === dateFilterState.startDate.year
+                    && month < dateFilterState.startDate.month
+                  )
+                  || (
+                    year === dateFilterState.startDate.year
+                    && month === dateFilterState.startDate.month
+                    && day < dateFilterState.startDate.day
+                  )
+                ) {
+                  return alert(
+                    'Invalid Start Date',
+                    'The start date cannot be before the end date.',
+                  );
+                }
+                dateFilterState.endDate = { month, day, year };
+                handleDateRangeUpdated(dateFilterState);
               }}
             />
           </TabBox>
@@ -895,20 +996,20 @@ const LogReviewer: React.FC<Props> = (props) => {
               const children: PickableItem[] = (
                 Object.keys(value)
                   .filter((subcontext) => {
-                    return subcontext !== '_'
+                    return subcontext !== '_';
                   })
                   .map((subcontext) => {
                     return {
-                      id: `${context}-${subcontext}`,
+                      id: subcontext,
                       name: genHumanReadableName(subcontext),
                       isGroup: false,
-                      checked: !!value[subcontext],
+                      checked: (contextFilterState[context] as any)[subcontext],
                     };
                   })
               );
               const item: PickableItem = {
                 id: context,
-                name: context,
+                name: genHumanReadableName(context),
                 isGroup: true,
                 children,
               };
@@ -927,9 +1028,13 @@ const LogReviewer: React.FC<Props> = (props) => {
                 if (pickableItem.isGroup) {
                   // Has subcontexts
                   pickableItem.children.forEach((subcontextItem) => {
-                    (contextFilterState as any)[pickableItem.id][subcontextItem.id] = (
-                      (subcontextItem as any).checked
-                    );
+                    if (!subcontextItem.isGroup) {
+                      (
+                        contextFilterState[pickableItem.id] as { [k: string]: boolean }
+                      )[subcontextItem.id] = (
+                        subcontextItem.checked
+                      );
+                    }
                   });
                 } else {
                   // No subcontexts
@@ -957,8 +1062,9 @@ const LogReviewer: React.FC<Props> = (props) => {
                     <CheckboxButton
                       id={`LogReviewer-tag-${tag}-checkbox`}
                       text={description}
-                      ariaLabel={`include logs tagged with "${description}" in results`}
+                      ariaLabel={`require that logs be tagged with "${description}" or any other selected tag`}
                       noMarginOnRight={i === Object.keys(tagFilterState).length - 1}
+                      checked={tagFilterState[tag]}
                       onChanged={(checked) => {
                         tagFilterState[tag] = checked;
                       }}
@@ -1023,7 +1129,10 @@ const LogReviewer: React.FC<Props> = (props) => {
               ) && (
                 <TabBox title="Action Log Details">
                   {/* Action */}
-                  <ButtonInputGroup label="Action">
+                  <ButtonInputGroup
+                    label="Action"
+                    className="mb-2"
+                  >
                     {
                       Object.keys(LogAction)
                         .map((action, i) => {
@@ -1048,6 +1157,13 @@ const LogReviewer: React.FC<Props> = (props) => {
                   </ButtonInputGroup>
                   {/* Target */}
                   <ButtonInputGroup label="Target">
+                    {/* Nothing here */}
+                    {(Object.keys(LogMetadata.Target ?? {}).length === 0) && (
+                      <div>
+                        This app does not have any targets yet.
+                      </div>
+                    )}
+                    {/* List of targets */}
                     {
                       Object.keys(LogMetadata.Target ?? {})
                         .map((target, i) => {
@@ -1394,53 +1510,58 @@ const LogReviewer: React.FC<Props> = (props) => {
                 />
               </ButtonInputGroup>
 
-              {/* Route path */}
-              <div className="input-group mb-2">
-                <span className="input-group-text">
-                  Server Route Path
-                </span>
-                <input
-                  type="text"
-                  className="form-control"
-                  aria-label="query for server route path"
-                  placeholder="e.g. /api/ttm/courses/12345"
-                  value={advancedFilterState.routePath}
-                  onChange={(e) => {
-                    advancedFilterState.courseName = (
-                      (e.target.value)
-                        .trim()
-                    );
-                    dispatch({
-                      type: ActionType.UpdateAdvancedFilterState,
-                      advancedFilterState,
-                    });
-                  }}
-                />
-              </div>
+              {/* Server filters */}
+              {advancedFilterState.source !== LogSource.Client && (
+                <div className="mt-2">
+                  {/* Route path */}
+                  <div className="input-group mb-2">
+                    <span className="input-group-text">
+                      Server Route Path
+                    </span>
+                    <input
+                      type="text"
+                      className="form-control"
+                      aria-label="query for server route path"
+                      placeholder="e.g. /api/ttm/courses/12345"
+                      value={advancedFilterState.routePath}
+                      onChange={(e) => {
+                        advancedFilterState.courseName = (
+                          (e.target.value)
+                            .trim()
+                        );
+                        dispatch({
+                          type: ActionType.UpdateAdvancedFilterState,
+                          advancedFilterState,
+                        });
+                      }}
+                    />
+                  </div>
 
-              {/* Route template */}
-              <div className="input-group mb-2">
-                <span className="input-group-text">
-                  Server Route Template
-                </span>
-                <input
-                  type="text"
-                  className="form-control"
-                  aria-label="query for server route template"
-                  value={advancedFilterState.routeTemplate}
-                  placeholder="e.g. /api/ttm/courses/:courseId"
-                  onChange={(e) => {
-                    advancedFilterState.courseName = (
-                      (e.target.value)
-                        .trim()
-                    );
-                    dispatch({
-                      type: ActionType.UpdateAdvancedFilterState,
-                      advancedFilterState,
-                    });
-                  }}
-                />
-              </div>
+                  {/* Route template */}
+                  <div className="input-group mb-2">
+                    <span className="input-group-text">
+                      Server Route Template
+                    </span>
+                    <input
+                      type="text"
+                      className="form-control"
+                      aria-label="query for server route template"
+                      value={advancedFilterState.routeTemplate}
+                      placeholder="e.g. /api/ttm/courses/:courseId"
+                      onChange={(e) => {
+                        advancedFilterState.courseName = (
+                          (e.target.value)
+                            .trim()
+                        );
+                        dispatch({
+                          type: ActionType.UpdateAdvancedFilterState,
+                          advancedFilterState,
+                        });
+                      }}
+                    />
+                  </div>
+                </div>
+              )}
             </TabBox>
           </>
         );
@@ -1463,9 +1584,10 @@ const LogReviewer: React.FC<Props> = (props) => {
     // > Perform filters
     const logs: Log[] = [];
     Object.keys(logMap).forEach((year) => {
-      Object.keys(logMap).forEach((month) => {
+      Object.keys(logMap[year]).forEach((month) => {
         logMap[year][month].forEach((log) => {
           /* ----------- Date Filter ---------- */
+          console.log('Log:', log);
           
           // Before start date
           if (
@@ -1483,6 +1605,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               && (log.day < dateFilterState.startDate.day)
             )
           ) {
+            console.log('RULED OUT: START');
             return;
           }
 
@@ -1502,6 +1625,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               && (log.day > dateFilterState.endDate.day)
             )
           ) {
+            console.log('RULED OUT: END');
             return;
           }
 
@@ -1513,19 +1637,27 @@ const LogReviewer: React.FC<Props> = (props) => {
             contextFilterState[log.context] === false
             // None of the subcontexts are selected
             || (
-              Object.values(contextFilterState[log.context] ?? {})
+              // Has subcontexts
+              typeof contextFilterState[log.context] !== 'boolean'
+              // None of the subcontexts are selected
+              && Object.values(contextFilterState[log.context] ?? {})
                 .every((isSelected) => {
                   return !isSelected;
                 })
             )
           ) {
+            // TODO: figure out why context filter is still not working, why actions and targets filter UI sucks and doesn't wrap and doesn't select
+            console.log('RULED OUT: CONTEXT');
+            console.log(log.context, contextFilterState);
             return;
           }
 
           // Subcontext doesn't match
           if (
+            // Log context is not "uncategorized" (no point in further filters)
+            log.context !== LogBuiltInMetadata.Context.Uncategorized
             // Log has a subcontext
-            log.subcontext
+            && log.subcontext
             // Context has subcontexts
             && (
               contextFilterState[log.context]
@@ -1535,6 +1667,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // Subcontext is not selected
             && !(contextFilterState as any)[log.context][log.subcontext]
           ) {
+            console.log('RULED OUT: SUBCONTEXT');
             return;
           }
 
@@ -1542,15 +1675,18 @@ const LogReviewer: React.FC<Props> = (props) => {
 
           // No tags match
           if (
-            // Log has at least one tag
-            log.tags.length > 0
-            // No tags match
-            && (
-              log.tags.every((tag) => {
-                return !tagFilterState[tag];
+            // At least one tag is required
+            Object.values(tagFilterState)
+              .filter((isSelected) => {
+                return isSelected;
               })
-            )
+              .length > 0
+            // No tags match
+            && log.tags.every((tag) => {
+              return !tagFilterState[tag];
+            })
           ) {
+            console.log('RULED OUT: TAGS');
             return;
           }
 
@@ -1563,6 +1699,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // Log type doesn't match
             && actionErrorFilterState.type !== log.type
           ) {
+            console.log('RULED OUT: TYPE');
             return;
           }
 
@@ -1579,6 +1716,7 @@ const LogReviewer: React.FC<Props> = (props) => {
                 actionErrorFilterState.errorMessage.trim().toLowerCase(),
               )
             ) {
+              console.log('RULED OUT: ERROR MESSAGE');
               return;
             }
 
@@ -1593,6 +1731,7 @@ const LogReviewer: React.FC<Props> = (props) => {
                 actionErrorFilterState.errorCode.trim().toUpperCase(),
               )
             ) {
+              console.log('RULED OUT: ERROR CODE');
               return;
             }
           }
@@ -1606,6 +1745,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               // Target isn't selected
               && !actionErrorFilterState.target[log.target]
             ) {
+              console.log('RULED OUT: TARGET');
               return;
             }
 
@@ -1616,6 +1756,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               // Action isn't selected
               && !actionErrorFilterState.action[log.action]
             ) {
+              console.log('RULED OUT: ACTION');
               return;
             }
           }
@@ -1631,6 +1772,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               advancedFilterState.userFirstName.toLowerCase().trim(),
             )
           ) {
+            console.log('RULED OUT: FIRST');
             return;
           }
 
@@ -1643,6 +1785,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               advancedFilterState.userLastName.toLowerCase().trim(),
             )
           ) {
+            console.log('RULED OUT: LAST');
             return;
           }
 
@@ -1655,6 +1798,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               advancedFilterState.userEmail.toLowerCase().trim(),
             )
           ) {
+            console.log('RULED OUT: EMAIL');
             return;
           }
 
@@ -1667,6 +1811,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               advancedFilterState.userId.trim(),
             )
           ) {
+            console.log('RULED OUT: USER ID');
             return;
           }
 
@@ -1677,6 +1822,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // Learners aren't included
             && !advancedFilterState.includeLearners
           ) {
+            console.log('RULED OUT: LEARNER');
             return;
           }
 
@@ -1687,6 +1833,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // TTMs aren't included
             && !advancedFilterState.includeTTMs
           ) {
+            console.log('RULED OUT: TTM');
             return;
           }
 
@@ -1697,6 +1844,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // Admins aren't included
             && !advancedFilterState.includeAdmins
           ) {
+            console.log('RULED OUT: ADMIN');
             return;
           }
 
@@ -1709,6 +1857,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               advancedFilterState.courseId.trim(),
             )
           ) {
+            console.log('RULED OUT: COURSE ID');
             return;
           }
 
@@ -1721,6 +1870,7 @@ const LogReviewer: React.FC<Props> = (props) => {
               advancedFilterState.courseName.trim(),
             )
           ) {
+            console.log('RULED OUT: COURSE NAME');
             return;
           }
 
@@ -1733,6 +1883,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // Mobile filter doesn't match
             && (advancedFilterState.isMobile === log.device.isMobile)
           ) {
+            console.log('RULED OUT: MOBILE');
             return;
           }
 
@@ -1745,6 +1896,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // Source filter doesn't match
             && (advancedFilterState.source !== log.source)
           ) {
+            console.log('RULED OUT: SOURCE');
             return;
           }
 
@@ -1757,6 +1909,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // Route path doesn't match
             && !(log.routePath.includes(advancedFilterState.routePath.trim()))
           ) {
+            console.log('RULED OUT: PATH');
             return;
           }
 
@@ -1769,6 +1922,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // Route template doesn't match
             && !(log.routeTemplate.includes(advancedFilterState.routeTemplate.trim()))
           ) {
+            console.log('RULED OUT: TEMPLATE');
             return;
           }
 
@@ -1974,19 +2128,40 @@ const LogReviewer: React.FC<Props> = (props) => {
 
     // Create intelliTable
     const dataTable = (
-      <IntelliTable
-        title="Matching Logs"
-        id="logs"
-        data={logs}
-        columns={columns}
-      />
+      logs.length === 0
+        ? (
+          <>
+            <h3 className="m-0">
+              Matching Logs:
+            </h3>
+            <div className="alert alert-warning text-center">
+              <h4 className="m-1">
+                No Logs to Show
+              </h4>
+              <div>
+                Either your filters are too strict or no matching logs have been
+                created yet.
+              </div>
+            </div>
+          </>
+        )
+        : (
+          <IntelliTable
+            title="Matching Logs:"
+            id="logs"
+            data={logs}
+            columns={columns}
+          />
+        )
     );
 
     // Main body
     body = (
       <>
         {filters}
-        {dataTable}
+        <div className="mt-2">
+          {dataTable}
+        </div>
       </>
     );
   }
@@ -2001,26 +2176,22 @@ const LogReviewer: React.FC<Props> = (props) => {
       <div className="LogReviewer-inner-container">
         <div className="LogReviewer-header">
           <div className="LogReviewer-header-title">
-            <h1 className="m-0">
+            <h3 className="text-center m-0">
               Log Review Dashboard
-            </h1>
+            </h3>
+          </div>
+          <div style={{ width: 0 }}>
+            <button
+              type="button"
+              className="LogReviewer-header-close-button btn btn-dark btn-lg pe-0"
+              aria-label="close log reviewer panel"
+              onClick={onClose}
+            >
+              <FontAwesomeIcon
+                icon={faTimes}
+              />
+            </button>
           </div>
-          <button
-            type="button"
-            className="LogReviewer-header-close-button btn btn-lg"
-            aria-label="close log reviewer panel"
-            onClick={onClose}
-            style={{
-              border: 0,
-              backgroundColor: 'transparent',
-              padding: 0,
-              margin: 0,
-            }}
-          >
-            <FontAwesomeIcon
-              icon={faTimes}
-            />
-          </button>
         </div>
         <div className="LogReviewer-contents">
           {body}