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

Layout fixes for tab bar and tab stack panel #770

Merged
merged 3 commits into from Aug 26, 2016

Conversation

ellisonbg
Copy link
Contributor

@ellisonbg ellisonbg commented Aug 26, 2016

This brings up the core layout of the dock area to where it was before the mono-repo. I am also using less to better parametrize the border/width/height of the tabs.

@ellisonbg ellisonbg added this to the 0.90 milestone Aug 26, 2016
@ellisonbg
Copy link
Contributor Author

I should mention that I have a lot more work like this on the way...

Here is a screenshot of what this looks like:

screen shot 2016-08-25 at 6 08 13 pm

.p-TabBar {
overflow: visible;
color: #777777;
font-size: 13px;
font-size: @jp-ui-font-size;
Copy link
Member

Choose a reason for hiding this comment

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

Should we follow our CSS naming convention and use jp-ui-fontSize here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thought about that - the main difference is that things like font-size are actual CSS properties. Many/most of our less variables will have names like @jp-component-css-property so I would prefer to use a slightly modified naming convention here.

@blink1073
Copy link
Member

Should this explicitly close #680?

@ellisonbg
Copy link
Contributor Author

This should not close #680 as there are a bunch of other things I need to fix as well...

@ellisonbg
Copy link
Contributor Author

@blink1073 once this has passed Travis again, this should be ready for merge. See above note about less variable names.

@ellisonbg
Copy link
Contributor Author

Merging to keep iterating...

@ellisonbg ellisonbg merged commit d27f686 into jupyterlab:master Aug 26, 2016
@ellisonbg ellisonbg deleted the layout-fixes branch June 14, 2017 04:12
@lock lock bot added the status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. label Aug 10, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Aug 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:resolved-locked Closed issues are locked after 30 days inactivity. Please open a new issue for related discussion. tag:Design and UX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants