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

only compact attribution on screens #6506

Merged
merged 1 commit into from
May 2, 2018

Conversation

andrewharvey
Copy link
Collaborator

Launch Checklist

  • briefly describe the changes in this PR

On small maps, the AttributionControl uses an i icon expanded to the full attribution on mouseover.

This PR ensures that only happens for the screen media, so that if a user prints a web page displaying a GL JS map the attribution text is still shown.

  • write tests for all new functionality
  • document any changes to public APIs
  • post benchmark scores
  • manually test the debug page

@andrewharvey andrewharvey force-pushed the only-compact-attribution-on-screen branch from 887b7b1 to e157848 Compare May 1, 2018 23:37
Copy link
Contributor

@jfirebaugh jfirebaugh left a comment

Choose a reason for hiding this comment

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

Thanks!

@jfirebaugh jfirebaugh merged commit ad0c81f into master May 2, 2018
@jfirebaugh jfirebaugh deleted the only-compact-attribution-on-screen branch May 2, 2018 20:29
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