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

Extra margins added by EMBL-EBI dropdown #114

Closed
gpmaccari opened this issue Sep 26, 2017 · 4 comments
Closed

Extra margins added by EMBL-EBI dropdown #114

gpmaccari opened this issue Sep 26, 2017 · 4 comments

Comments

@gpmaccari
Copy link

The EMBL-EBI dropdown menu adds extra margins in screens < 53em, causing the page not to be properly responsive.
screen shot 2017-09-26 at 11 42 04

.embl-selector .button[aria-expanded="false"]::after that adds some extra margin to allow the location name (Hinxton in this case) to be correctly visualized. When the screen width is < 53em, the location name is not displayed, but the ::after element still adds extra right margins.

@gpmaccari
Copy link
Author

This is the same issue as #63

khawkins98 added a commit that referenced this issue Oct 2, 2017
@khawkins98
Copy link
Contributor

I've just committed a fix for v1.2 -- it should show shortly on the dev server: https://wwwdev.ebi.ac.uk/web_guidelines/EBI-Framework/v1.2/css/ebi-global.css

Here it is on the boilerplate page: https://wwwdev.ebi.ac.uk/style-lab/websites/sample-site/boilerplate/blank.html

Hopefully in v1.3 the approach will be a bit more refined.

@gpmaccari
Copy link
Author

The display: nonein the .embl-selector .button:after element doesn't seems to override the original display: block style.

Changing the code to

@media screen and (max-width: 53em){
  .embl-selector .button[aria-expanded="false"]::after{ 
     display:none; 
  }
}

seems to work.

@khawkins98
Copy link
Contributor

Supporting this method (which requires Foundation JS) was proving difficult and problematic, so in v1.3 we moved to a more "simple" and cleaner approach.

This should now be resolved; example: https://www.ebi.ac.uk/style-lab/websites/sample-site/boilerplate/blank.html

Closing this issue, but please shout if issues persist -- or if there are other concerns.

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

No branches or pull requests

2 participants