Skip to content

[docs] Address user feedback for /api: sources, images, layers - #8754

Merged
colleenmcginnis merged 8 commits into
masterfrom
cmcg-reader-feedback-map-pt2
Sep 17, 2019
Merged

[docs] Address user feedback for /api: sources, images, layers#8754
colleenmcginnis merged 8 commits into
masterfrom
cmcg-reader-feedback-map-pt2

Conversation

@colleenmcginnis

Copy link
Copy Markdown
Contributor

This PR is one in a series that will address user feedback on the contents of the API docs including adding inline examples to all sections of the API docs and auditing Related links for relevance.

As always, all feedback is welcome! But, there are some specific things that need a close look:

  • Am I talking about sprites + images added using addImage correctly?
  • Thoughts on adding context to @see resources (for example, * @see Vector source: [Show and hide layers](https://docs.mapbox.com/mapbox-gl-js/example/toggle-layers/))?
  • Style layer zoom extents vs. source layer zoom extents in the setLayerZoomRange description.

cc @mapbox/docs

@colleenmcginnis
colleenmcginnis requested a review from a team September 14, 2019 00:54
Comment thread src/ui/map.js Outdated

@mourner mourner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Wow, those updates are so good! This will be super helpful for new users.

Comment thread src/ui/map.js
* @returns {Map} `this`
* @see [Draw GeoJSON points](https://www.mapbox.com/mapbox-gl-js/example/geojson-markers/)
* @see [Style circles using data-driven styling](https://www.mapbox.com/mapbox-gl-js/example/data-driven-circle-colors/)
* @see [Set a point after Geocoder result](https://www.mapbox.com/mapbox-gl-js/example/point-from-geocoder-result/)

@andrewharvey andrewharvey Sep 17, 2019

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ha, I just discovered this independently, was about to submit my PR when I noticed this.
Since 6e6a005#diff-44befe964c5c5c476298e33cc55ce3e7 the geocoder example no longer uses addSource.

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