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

Single file copies of our GeoJSON examples are now rendered nicely on Github #12

Merged
merged 1 commit into from
Jun 17, 2013
Merged

Single file copies of our GeoJSON examples are now rendered nicely on Github #12

merged 1 commit into from
Jun 17, 2013

Conversation

sthagen
Copy link
Member

@sthagen sthagen commented Jun 15, 2013

I simply renamed the .js single file copies of the I-D submission candidates GeoJSON examples into .geojson files, so that we all have a taste of the new GeoJSON awsomeness when we browse the js folder in our github repo.

Kudos from me to Tim for reminding me once again, how cool git (and hg are) and how deep the viral patterns of svn induced branch laziness really invaded my brain ;-)

Hope you like it. Don't expect all samples to cause stunning display effects. Some may even result in a cannot render message.

All the best,
Stefan.

…ger the new GeoJSON display rendering on github
@tschaub
Copy link
Member

tschaub commented Jun 17, 2013

Looks good @sdrees. Please merge.

@sgillies
Copy link
Contributor

I'm okay with a merge, though it might be good to alter the values in these examples so that they overlay interesting map data and fall within range of Leaftlet zoom levels.

@tschaub
Copy link
Member

tschaub commented Jun 17, 2013

@sgillies - agreed. Was going to address that in a separate PR.

@sthagen
Copy link
Member Author

sthagen commented Jun 17, 2013

Am 17.06.13 20:25, schrieb Tim Schaub:

@sgillies https://github.com/sgillies - agreed. Was going to address
that in a separate PR.


Reply to this email directly or view it on GitHub
#12 (comment).

so I merge and Tim adds beauty to the awsomeness ;-?)

Stefan.

sthagen pushed a commit that referenced this pull request Jun 17, 2013
…awesomeness

Single file copies of our GeoJSON examples are now rendered nicely on Github
@sthagen sthagen merged commit 762fcc0 into geojson:master Jun 17, 2013
@sthagen sthagen deleted the rename-geojson-samples-for-github-awesomeness branch June 17, 2013 19:41
@sthagen
Copy link
Member Author

sthagen commented Jun 17, 2013

@tschaub: All set with the awsomeness, now for the beauty 👍 ...

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