Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Notebook output scrollable #6490

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Notebook output scrollable #6490

wants to merge 9 commits into from
+39 −5

Conversation

Tom-Szendrey
Copy link
Contributor

@Tom-Szendrey Tom-Szendrey commented Feb 7, 2025 •
edited
Loading

Description

  • Allow user to scroll the output console
  • Allow user to clear the output console
  • Add this output section to other drilldowns utilizing the notebook.

Note I could rewrite how we are using
tera-notebook-jupyter-input, v-ace-editor, and tera-notebook-output but I wanted this to be a small PR. If we want to clean that up we can prioritize this issue

Screen.Recording.2025-02-07.at.9.30.35.AM.mov

@Tom-Szendrey Tom-Szendrey marked this pull request as draft February 7, 2025 14:45
@Tom-Szendrey Tom-Szendrey marked this pull request as ready for review February 14, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YohannParis YohannParis Awaiting requested review from YohannParis YohannParis is a code owner

@mwdchang mwdchang Awaiting requested review from mwdchang mwdchang is a code owner

@jryu01 jryu01 Awaiting requested review from jryu01 jryu01 is a code owner

@blanchco blanchco Awaiting requested review from blanchco blanchco is a code owner

@mloppie mloppie Awaiting requested review from mloppie mloppie is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@YohannParis YohannParis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Tom-Szendrey @YohannParis