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

[FIX] #452 #459

Closed
wants to merge 2 commits into from
Closed

[FIX] #452 #459

wants to merge 2 commits into from

Conversation

mrleblanc101
Copy link

@mrleblanc101 mrleblanc101 commented Jan 22, 2018

Fix an issue with multiline-tab package
Fix an issue with tool-bar package separator in vertical mode.

Description of the Change

multiline-tab:

Fix an issue that caused the .placeholder when dragging tabs to be 10rem height. Since multiline-tabs package disabled the vertical overflow of the tab bar to allow wrapping, this caused an issue where the placeholder would span multiple row of tabs. Now the placeholder with be the same height as the tab( 3rem, or 2.5rem if compact mode is enabled)

tool-bar:

Fix and issue that caused separator to be left-aligned instead of centered when the tool-bar is set in vertical mode. Also correct the separator color variable so it's not as light and work with more theme The color is now 5% lighter then the icon instead of 5% lighter than the text which make more sense.

Benefits

Fix a compatibility issue with two widely used package

Possible Drawbacks

None that I see.

Applicable Issues

#452, #460

Fix an issue with multiline-tab package
Fix tool-bar separator not being centered when in vertical-mode. Fix separator color being way too light.
Comment on lines +193 to 203
content: "";
background: linear-gradient(to bottom, black, transparent);
position: absolute;
top: 0;
left: 0;
width: 100%;
height: 3px;
opacity: .125;
opacity: 0.125;
pointer-events: none;
z-index: 10;
}

Choose a reason for hiding this comment

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

:octocat:

@mrleblanc101
Copy link
Author

@mjrael89 wut ?

@atom-material atom-material deleted a comment from mjrael89 Dec 14, 2020
@atom-material atom-material deleted a comment Dec 14, 2020
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