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: copied div margin-top #22

Merged
merged 1 commit into from
Mar 14, 2014
Merged

Conversation

jenil
Copy link
Contributor

@jenil jenil commented Mar 14, 2014

I just noticed that the icons and the white band were overlapping at times when the icon was big. I have bummed up the margin-top a bit to fix this issue.

I just noticed that the icons and the white band were overlapping at times when the icon was big. I have bummed up the margin-top a bit to fix this issue.
@thomaspark
Copy link
Owner

Thanks. We should make this overlay more responsive at some point.

thomaspark added a commit that referenced this pull request Mar 14, 2014
fix: copied div margin-top
@thomaspark thomaspark merged commit 382f01c into thomaspark:gh-pages Mar 14, 2014
@jenil
Copy link
Contributor Author

jenil commented Mar 14, 2014

I think it is pretty responsive, because the overlay doesn't show up on mobiles and tablets which don't support flash anyway.

@thomaspark
Copy link
Owner

But people on PCs have windows in all sorts of configurations, especially for a utility site like this. It wouldn't take much to ensure the overlay is centered and displayed nicely at all sizes.

@jenil
Copy link
Contributor Author

jenil commented Mar 14, 2014

I checked it, it works well only issues is we need to reduce the font-size with respect to the viewport.

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.

2 participants