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 issue 2635 #3104

Closed
wants to merge 1 commit into from
Closed

Conversation

ShreyaSingh10
Copy link

Description

This fixes the overlapping problem with the WordPress admin sidebar.

How Has This Been Tested?

Tested with chrome inspector too.

@@ -15,7 +15,7 @@ $blocks-button__height: 46px;
z-index: 2;
top: $blocks-button__height + 2px;
left: 50%;
transform: translateX( -50% );
Copy link
Contributor

Choose a reason for hiding this comment

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

Which bug is this fixing exactly? and why 13%?

Copy link
Author

@ShreyaSingh10 ShreyaSingh10 Oct 23, 2017

Choose a reason for hiding this comment

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

This fix is for #2635

@Soean
Copy link
Member

Soean commented Nov 13, 2017

Issue #2635 is fixed, so we can close this PR.

@youknowriad
Copy link
Contributor

youknowriad commented Jan 31, 2018

Thanks for your work on this PR @ShreyaSingh10 The issue is fixed now.

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 this pull request may close these issues.

3 participants