dce-reactkit 3.2.2-beta.4 → 3.2.2-beta.40

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,22 +573,27 @@ 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;
       });
     }
   });
 
   // Create initial tag filter state
   const initTagFilterState: TagFilterState = {};
-  Object.values(LogMetadata.Tag ?? {}).forEach((tagValue) => {
-    initTagFilterState[tagValue] = true;
+  Object.keys(LogMetadata.Tag ?? {}).forEach((tag) => {
+    initTagFilterState[tag] = 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;
@@ -727,16 +813,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 +841,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 +857,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 +901,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 +932,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 +948,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 +995,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 +1027,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
@@ -950,17 +1054,22 @@ const LogReviewer: React.FC<Props> = (props) => {
         filterDrawer = (
           <TabBox title="Tags">
             {
-              Object.keys(tagFilterState)
+              Object.keys(LogMetadata.Tag ?? {})
                 .map((tag, i) => {
                   const description = genHumanReadableName(tag);
                   return (
                     <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;
+                        dispatch({
+                          type: ActionType.UpdateTagFilterState,
+                          tagFilterState,
+                        });
                       }}
                     />
                   );
@@ -1023,19 +1132,25 @@ 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) => {
                           const description = genHumanReadableName(action);
                           return (
                             <CheckboxButton
+                              key={action}
                               id={`LogReviewer-action-${action}-checkbox`}
                               text={description}
                               ariaLabel={`include logs with action type "${description}" in results`}
                               noMarginOnRight={i === Object.keys(LogAction).length - 1}
+                              checked={actionErrorFilterState.action[action]}
                               onChanged={(checked) => {
                                 actionErrorFilterState.action[action] = checked;
+                                console.log(actionErrorFilterState, action, checked);
                                 dispatch({
                                   type: ActionType.UpdateActionErrorFilterState,
                                   actionErrorFilterState,
@@ -1048,17 +1163,27 @@ 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) => {
                           const description = genHumanReadableName(target);
                           return (
                             <CheckboxButton
+                              key={target}
                               id={`LogReviewer-target-${target}-checkbox`}
                               text={description}
                               ariaLabel={`include logs with target "${description}" in results`}
+                              checked={actionErrorFilterState.target[target]}
                               onChanged={(checked) => {
                                 actionErrorFilterState.target[target] = checked;
+                                console.log(actionErrorFilterState, target, checked);
                                 dispatch({
                                   type: ActionType.UpdateActionErrorFilterState,
                                   actionErrorFilterState,
@@ -1108,7 +1233,7 @@ const LogReviewer: React.FC<Props> = (props) => {
                       type="text"
                       className="form-control"
                       aria-label="query for error code"
-                      value={actionErrorFilterState.errorMessage}
+                      value={actionErrorFilterState.errorCode}
                       onChange={(e) => {
                         actionErrorFilterState.errorCode = (
                           (e.target.value)
@@ -1394,53 +1519,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,7 +1593,7 @@ 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 ---------- */
           
@@ -1513,19 +1643,25 @@ 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;
                 })
             )
           ) {
+            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]
@@ -1542,14 +1678,16 @@ 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];
+            })
           ) {
             return;
           }
@@ -1731,7 +1869,7 @@ const LogReviewer: React.FC<Props> = (props) => {
             // Device info exists
             && log.device
             // Mobile filter doesn't match
-            && (advancedFilterState.isMobile === log.device.isMobile)
+            && (advancedFilterState.isMobile !== log.device.isMobile)
           ) {
             return;
           }
@@ -1807,18 +1945,18 @@ const LogReviewer: React.FC<Props> = (props) => {
         type: ParamType.Int,
       },
       {
-        title: 'Student',
+        title: 'Student?',
         param: 'isLearner',
         type: ParamType.Boolean,
       },
       {
-        title: 'Teaching Staff',
+        title: 'Teaching Staff?',
         param: 'isTTM',
         type: ParamType.Boolean,
         startsHidden: true,
       },
       {
-        title: 'Admin',
+        title: 'Admin?',
         param: 'isAdmin',
         type: ParamType.Boolean,
         startsHidden: true,
@@ -1853,7 +1991,7 @@ const LogReviewer: React.FC<Props> = (props) => {
         startsHidden: true,
       },
       {
-        title: 'Mobile',
+        title: 'Mobile?',
         param: 'device.isMobile',
         type: ParamType.Boolean,
         startsHidden: true,
@@ -1972,10 +2110,27 @@ const LogReviewer: React.FC<Props> = (props) => {
       },
     ];
 
+    // Nothing to show notice
+    const noLogsNotice = (
+      logs.length === 0
+        ? (
+          <div className="alert alert-warning text-center mt-2">
+            <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>
+        )
+        : undefined
+    );
+
     // Create intelliTable
     const dataTable = (
       <IntelliTable
-        title="Matching Logs"
+        title="Matching Logs:"
         id="logs"
         data={logs}
         columns={columns}
@@ -1986,7 +2141,10 @@ const LogReviewer: React.FC<Props> = (props) => {
     body = (
       <>
         {filters}
-        {dataTable}
+        <div className="mt-2">
+          {dataTable}
+          {noLogsNotice}
+        </div>
       </>
     );
   }
@@ -2001,26 +2159,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}