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.

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

UX: SourceRef should be default-expanded in Notebooks? #5866

Closed
starpit opened this issue Oct 6, 2020 · 3 comments · Fixed by #5867
Closed

UX: SourceRef should be default-expanded in Notebooks? #5866

starpit opened this issue Oct 6, 2020 · 3 comments · Fixed by #5867

Comments

@starpit
Copy link
Contributor

starpit commented Oct 6, 2020

The suggestion here is that, when viewing a Notebook, the referenced Source (e.g. the foo.yaml in kubectl apply -f foo.yaml) is an important part of the educational potential of the Notebook.

@starpit
Copy link
Contributor Author

starpit commented Oct 6, 2020

not expanded:

job notebook

@starpit
Copy link
Contributor Author

starpit commented Oct 6, 2020

Expanded: Screen Shot 2020-10-06 at 8 37 56 AM

@starpit
Copy link
Contributor Author

starpit commented Oct 6, 2020

If we do this, we probably want to finagle SimpleEditor so that it is sized to fit, in terms of height. For this, we can use the relatively new (introduced in 0.20.x) editor.getContentHeight() API from monaco: microsoft/monaco-editor#794 (comment)

starpit added a commit to starpit/kui that referenced this issue Oct 6, 2020
…ded in Notebooks?

This PR updates SimpleEditor so that, for `simple` props, it is sized to fit, in terms of height.

Fixes kubernetes-sigs#5866
starpit added a commit to starpit/kui that referenced this issue Oct 6, 2020
…ded in Notebooks?

This PR updates SimpleEditor so that, for `simple` props, it is sized to fit, in terms of height.

Fixes kubernetes-sigs#5866
starpit added a commit to starpit/kui that referenced this issue Oct 6, 2020
…ded in Notebooks?

This PR updates SimpleEditor so that, for `simple` props, it is sized to fit, in terms of height.

This PR also fixes a longer-standing problem where reducing the width of the window would not result in a re-layout of monaco for SourceRef; we need a min-width: 0 on flex-fill

Fixes kubernetes-sigs#5866
starpit added a commit to starpit/kui that referenced this issue Oct 6, 2020
…ded in Notebooks?

This PR updates SimpleEditor so that, for `simple` props, it is sized to fit, in terms of height.

This PR also fixes a longer-standing problem where reducing the width of the window would not result in a re-layout of monaco for SourceRef; we need a min-width: 0 on flex-fill

Fixes kubernetes-sigs#5866
starpit added a commit to starpit/kui that referenced this issue Oct 6, 2020
…ded in Notebooks?

This PR updates SimpleEditor so that, for `simple` props, it is sized to fit, in terms of height.

This PR also fixes a longer-standing problem where reducing the width of the window would not result in a re-layout of monaco for SourceRef; we need a min-width: 0 on flex-fill

Fixes kubernetes-sigs#5866
starpit added a commit that referenced this issue Oct 6, 2020
…ded in Notebooks?

This PR updates SimpleEditor so that, for `simple` props, it is sized to fit, in terms of height.

This PR also fixes a longer-standing problem where reducing the width of the window would not result in a re-layout of monaco for SourceRef; we need a min-width: 0 on flex-fill

Fixes #5866
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant