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

Fix for Code Mirror width. Requested changes implemented #11809

Closed
wants to merge 3 commits into from

Conversation

pree-T
Copy link
Contributor

@pree-T pree-T commented Jan 10, 2022

Fix for the Codemirror Width issue #11805 ,implemented the changes asked in the last PR review.
Kindly check.

Fixes #11696

@jupyterlab-probot
Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@github-actions github-actions bot added Design System CSS pkg:cells pkg:outputarea tag:CSS For general CSS related issues and pecadilloes labels Jan 10, 2022
@fcollonval fcollonval added the bug label Jan 10, 2022
@fcollonval fcollonval modified the milestones: 3.2.x, 3.3.x Jan 10, 2022
Copy link
Member

@fcollonval fcollonval left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @pree-T

This looks good on Binder.

The linter is complaining. It should be happy with the suggestions I added.

@@ -59,7 +60,7 @@
.jp-InputArea-editor {
display: table-row;
margin-left: var(--jp-notebook-padding);
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing white spaces:

Suggested change
}
}

@@ -93,7 +93,7 @@
}

.jp-CodeCell.jp-mod-outputsScrolled .jp-OutputArea-prompt {
min-width: calc(
width: calc(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Over indentation

Suggested change
width: calc(
width: calc(

@pree-T pree-T closed this Jan 10, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Design System CSS pkg:cells pkg:outputarea tag:CSS For general CSS related issues and pecadilloes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[JLab v4a17] CodeMirror width is not limited when text overflow
2 participants