Skip to content

Commit

Permalink
updated sample code and issue template (#1288)
Browse files Browse the repository at this point in the history
* updated sample code and issue template

* clarify link

* Update .github/ISSUE_TEMPLATE/bug_report.md

Co-authored-by: Jacob Wasilkowski <[email protected]>

Co-authored-by: Jacob Wasilkowski <[email protected]>
  • Loading branch information
gavinr and jwasilgeo authored Aug 11, 2021
1 parent d81df03 commit 0b1a152
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 7 deletions.
10 changes: 7 additions & 3 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,15 @@ A clear and concise description of what the bug is.
- Describe what occurs in your code.
- Specifically, what seems to work differently than you intended?
- Provide any error messages you see in the console.
- If possible, create a [jsbin](http://jsbin.com/dagilag/edit?html,output) that demonstrates the problem.
- Please create a [jsbin](https://jsbin.com/yewimat/edit?html,output) that demonstrates the problem.

## To Reproduce

Please share the steps to reproduce the behavior.
Please share and fill in the steps below to reproduce the behavior.

**THIS IS REQUIRED** - If you do not provide the steps for others to reproduce the issue you're seeing, it is impossible for them to assist you. Please create a [jsbin](https://jsbin.com/yewimat/edit?html,output) that demonstrates the problem and include the steps to see the issue:

[Example](https://LINK_TO_YOUR_JS_BIN)

1. Go to '...'
2. Click on '....'
Expand All @@ -45,4 +49,4 @@ If applicable, add screenshots to help explain your problem.

## Additional context

Add any other context about the problem here. If you're are *not* using the [CDN](http://www.jsdelivr.com/projects/leaflet.esri), please note what loading/bundling library you are using ([webpack](https://webpack.github.io/), [browserify](http://browserify.org/), [RequireJS](http://requirejs.org/))?
Add any other context about the problem here. If you're are *not* using the [CDN](http://www.jsdelivr.com/projects/leaflet.esri), please note what loading/bundling library you are using ([webpack](https://webpack.github.io/), [browserify](http://browserify.org/), [RequireJS](http://requirejs.org/))?
9 changes: 5 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -70,17 +70,18 @@ The easiest way to get started is to load Esri Leaflet via [CDN](https://unpkg.c
<body>
<div id="map"></div>
<script>
var map = L.map('map').setView([45.528, -122.680], 13);
const map = L.map('map').setView([34.39, -117.189], 6);
L.esri.basemapLayer("Gray").addTo(map);
var parks = L.esri.featureLayer({
url: "https://services.arcgis.com/rOo16HdIMeOBI4Mb/arcgis/rest/services/Portland_Parks/FeatureServer/0",
// https://www.arcgis.com/home/item.html?id=8b98df1300b749eabc3142864bb9a119
const parks = L.esri.featureLayer({
url: "https://services2.arcgis.com/FiaPA4ga0iQKduv3/arcgis/rest/services/NPS_Park_Unit_Boundaries/FeatureServer/0",
style: function () {
return { color: "#70ca49", weight: 2 };
}
}).addTo(map);
var popupTemplate = "<h3>{NAME}</h3>{ACRES} Acres<br><small>Property ID: {PROPERTYID}<small>";
const popupTemplate = "<h3>{UNIT_NAME}</h3><p>Type: {UNIT_TYPE}</p><p>Region: {REGION}</p>";
parks.bindPopup(function(e){
return L.Util.template(popupTemplate, e.feature.properties)
Expand Down
Binary file modified example.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.

0 comments on commit 0b1a152

Please sign in to comment.