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

Sprite docs: format list of file types correctly, and document that pixelRatio is required #364

Merged
merged 1 commit into from
Oct 6, 2023
Merged

Sprite docs: format list of file types correctly, and document that pixelRatio is required #364

merged 1 commit into from
Oct 6, 2023

Conversation

flother
Copy link
Contributor

@flother flother commented Oct 4, 2023

Following on from #355 and #356, here are another couple of issues I've noticed on the sprite documentation page.

  1. The list of the two file types (index file and image file) was formatted as a list with three items. The second item should be a paragraph within the first item.

  2. The paragraph following that also needs to include pixelRatio in the list of required properties (there are five required properties, not four).

Launch Checklist

  • Confirm your changes do not include backports from Mapbox projects (unless with compliant license) - if you are not sure about this, please ask!
  • Briefly describe the changes in this PR.
  • Link to related issues.

The list of the two file types (index file and image file) was formatted
as a list with three items. The second item should be a paragraph within
the first item.

The paragraph following that also needs to include `pixelRatio` in the
list of required properties.
@codecov-commenter
Copy link

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (edeaee3) 78.02% compared to head (1e99118) 78.02%.
Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #364   +/-   ##
=======================================
  Coverage   78.02%   78.02%           
=======================================
  Files         101      101           
  Lines        4142     4142           
  Branches     1184     1184           
=======================================
  Hits         3232     3232           
  Misses        910      910           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@HarelM HarelM merged commit 5b5b6cd into maplibre:main Oct 6, 2023
6 checks passed
@HarelM
Copy link
Collaborator

HarelM commented Oct 6, 2023

THANKS!!

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