lint fixes

src/components/structures/LoggedInView.tsx

@@ -473,7 +473,7 @@ class LoggedInView extends React.Component<IProps, IState> {
         const navAction = getKeyBindingsManager().getNavigationAction(ev);
         switch (navAction) {
             case KeyBindingAction.NextLandmark:
-                // The next landmark completes the cycle, back to the active space button.
+                // When no landmark area is selected, this is reached and we start at the first landmark, the active space button.
                 document.querySelector<HTMLElement>(".mx_SpaceButton_active")?.focus();
                 handled = true;
                 break;

src/components/structures/MatrixChat.tsx

@@ -315,6 +315,7 @@ export default class MatrixChat extends React.PureComponent<IProps, IState> {
     }
 
     private onKeyDown = (e: KeyboardEvent | React.KeyboardEvent): void => {
+        // Pass up the key event to the parent view (LoggedInView).
         this.props.onKeyDown?.(e);
 
         const action = getKeyBindingsManager().getAccessibilityAction(e);

src/components/views/rooms/BasicMessageComposer.tsx

@@ -539,11 +539,13 @@ export default class BasicMessageEditor extends React.Component<IProps, IState>
         const navAction = getKeyBindingsManager().getNavigationAction(event);
         switch (navAction) {
             case KeyBindingAction.PreviousLandmark:
-                document.querySelector(".mx_RoomTile_selected")?.focus();
+                // The previous landmark is the selected room (we know a room is selected because we're in its message composer).
+                document.querySelector<HTMLElement>(".mx_RoomTile_selected")?.focus();
                 handled = true;
                 break;
             case KeyBindingAction.NextLandmark:
-                document.querySelector(".mx_SpaceButton_active")?.focus();
+                // The next landmark completes the cycle, back to the active space button.
+                document.querySelector<HTMLElement>(".mx_SpaceButton_active")?.focus();
                 handled = true;
                 break;
         }