Rework CodeViewer Implementation

[camielvs]

Description

Rework the codeviewer so the fullscreen button is not embedded into the CodeVier component itself, and change the CodeViewer component to be controller rather than uncontrolled. it now focusses purely on displaying code inline or fullscreen, and does not concern itself with its triggering button.

Toggling functionality is moved to the TaskImplementation component, allowing the CodeViewer to be used without specifically requiring a button (i.e. it can be controlled programmatically). As such, this requires a slightly more manual implementation across current CodeViewer usage.

Note: I didn't want to get bogged down with trying to move all the divs in CodeViewer onto BlockStack. I feel that given it's a bespoke component it's probably okay for now to leave as-is.

Related Issue and Pull requests

Type of Change

• Cleanup/Refactor

Checklist

• I have tested this does not break current pipelines / runs functionality
• I have tested the changes on staging

Screenshots (if applicable)

No change to app functionality. Code architecture update only. Confirm that the CodeViewer still behaves as expected.

Test Instructions

Additional Comments

[camielvs]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Break TaskActions into two sections #1550
• Rework Pipeline Action Buttons into Action Framework #1544
• Rework Task Action Buttons into Action Framework #1534
• Rework Run Action Buttons into Action Framework #1524
• Rework CodeViewer Implementation #1527 👈 (View in Graphite)
• Cleanup RunDetails #1508
• Cleanup PipelineDetails #1468
• Cleanup TaskDetails #1464
• Cleanup ExecutionDetails #1471
• Add UI Blocks to Context Panel #1470
• Update Link Component #1472
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.