Skip to content
This repository has been archived by the owner on Sep 25, 2018. It is now read-only.

Make tab bottom border positioned relative to bottom #37

Merged
merged 1 commit into from
Jan 6, 2015

Conversation

ultramiraculous
Copy link

Both this theme and the light theme have the tab bar's underline positioned assuming the tab bar is the only thing in the top section. Anything added using atom.workspaceView.prependToTop will cause layout issues:

screen shot 2014-12-22 at 11 23 02 pm

@kevinsawicki
Copy link
Contributor

@simurai can you take a look at this?

position: fixed;
top: @tab-height + @tab-bottom-border-height;
position: absolute;
bottom: @tab-bottom-border-height - 1px; // - 1px border width
Copy link
Contributor

Choose a reason for hiding this comment

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

@ultramiraculous This line should stay as is. Otherwise it's a bit too high:

screen shot 2014-12-24 at 8 40 06 pm

Or replacing it with bottom: 0; would work too.

@simurai
Copy link
Contributor

simurai commented Dec 24, 2014

@ultramiraculous I see the problem, yeah.. good point. If you fix the above issue, this should be good to merge.

@ultramiraculous
Copy link
Author

Fixed!

simurai added a commit that referenced this pull request Jan 6, 2015
Make tab bottom border positioned relative to bottom
@simurai simurai merged commit 52ba787 into atom:master Jan 6, 2015
@simurai
Copy link
Contributor

simurai commented Jan 6, 2015

@ultramiraculous Thanks for the fix! It will introduce a new issue #38, but I think this one is still the more important to fix.

@mark-hahn
Copy link

When is this going to be released? It is still broken in 169.

@simurai
Copy link
Contributor

simurai commented Jan 11, 2015

Should be in the next release (170).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants