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

Add powered by Mapbox #438

Merged
merged 9 commits into from Mar 4, 2022
Merged

Add powered by Mapbox #438

merged 9 commits into from Mar 4, 2022

Conversation

mapbox-danny
Copy link
Contributor

@mapbox-danny mapbox-danny commented Oct 25, 2021

  • Add "powered by Mapbox" notation to the results list
  • write tests for all new functionality
  • run npm run docs and commit changes to API.md
  • update CHANGELOG.md with changes under master heading before merging

@mapbox-danny mapbox-danny self-assigned this Oct 28, 2021
@mbullington
Copy link
Contributor

Screen Shot 2022-01-25 at 3 47 40 PM

To make the text nicer I also ended up upgrading the system font stack, so it includes Apple's San Francisco font if available. This was surely the intent originally, but San Francisco was first released near when this started development.

We also use a similar font stack in GL JS now: mapbox/mapbox-gl-js#11029

.vscode/settings.json Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
lib/index.js Outdated Show resolved Hide resolved
mbullington and others added 5 commits January 26, 2022 12:22
Co-authored-by: mpothier <39810086+mpothier@users.noreply.github.com>
Co-authored-by: mpothier <39810086+mpothier@users.noreply.github.com>
@mpothier mpothier merged commit 3e3f316 into main Mar 4, 2022
@mpothier mpothier deleted the powered-by branch March 4, 2022 15:37
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.

None yet

3 participants