fix(plugins/plugin-client-common): when rerunning blocks, output alwa…

…ys scrolls to bottom

part of kubernetes-sigs#7527

plugins/plugin-client-common/src/components/Views/Terminal/Block/BlockModel.ts

@@ -204,6 +204,10 @@ export function hasOriginalUUID(block: BlockModel | WithOriginalExecUUID): block
   return typeof (block as WithOriginalExecUUID).originalExecUUID === 'string'
 }
 
+export function hasBeenRerun(block: BlockModel): boolean {
+  return hasOriginalUUID(block)
+}
+
 /** Transform to Processing */
 export function Processing(
   block: BlockModel,

plugins/plugin-client-common/src/components/Views/Terminal/Block/Output.tsx

@@ -45,6 +45,7 @@ import {
   FinishedBlock,
   hasUUID,
   hasCommand,
+  hasBeenRerun,
   isBeingRerun,
   isFinished,
   isProcessing,
@@ -171,7 +172,8 @@ export default class Output extends React.PureComponent<Props, State> {
   }
 
   private onRender(assertHasContent: boolean): void {
-    if (this.props.onRender) {
+    if (this.props.onRender && !this.props.isBeingRerun && !hasBeenRerun(this.props.model)) {
+      // we don't want reruns to trigger any scrolling behavior
       this.props.onRender()
     }
     this.setState({ assertHasContent })

plugins/plugin-client-common/src/components/Views/Terminal/ScrollableTerminal.tsx

@@ -143,6 +143,7 @@ type ScrollbackState = ScrollbackOptions & {
   onFocus: (evt: React.FocusEvent) => void
   onOutputRender: () => void
   navigateTo: (dir: 'first' | 'last' | 'previous' | 'next') => void
+  setActiveBlock: (c: Block) => void
   willFocusBlock: (evt: React.SyntheticEvent) => void
   willRemoveBlock: (evt: React.SyntheticEvent, idx?: number) => void
   willUpdateCommand: (idx: number, command: string) => void
@@ -403,6 +404,7 @@ export default class ScrollableTerminal extends React.PureComponent<Props, State
       onFocus: undefined,
       onOutputRender: undefined,
       navigateTo: undefined,
+      setActiveBlock: undefined,
       willFocusBlock: undefined,
       willRemoveBlock: undefined,
       willUpdateCommand: undefined
@@ -462,6 +464,13 @@ export default class ScrollableTerminal extends React.PureComponent<Props, State
       })
     }
 
+    /** Update the active block */
+    state.setActiveBlock = (c: Block) => {
+      if (c && c.props && c.props.model && isActive(c.props.model)) {
+        state._activeBlock = c
+      }
+    }
+
     /**
      * User clicked to focus a block.
      *
@@ -1327,7 +1336,7 @@ export default class ScrollableTerminal extends React.PureComponent<Props, State
                     isWidthConstrained={isWidthConstrained}
                     onOutputRender={scrollback.onOutputRender}
                     navigateTo={scrollback.navigateTo}
-                    ref={idx !== this.findActiveBlock(scrollback) ? undefined : c => (scrollback._activeBlock = c)}
+                    ref={scrollback.setActiveBlock}
                   />
                 )
               })