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.

Sign up for GitHub

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

Jump to bottom

Spaces needed #4010

Closed
raymondadouglas opened this issue Oct 29, 2015 · 2 comments
Closed

Spaces needed #4010

raymondadouglas opened this issue Oct 29, 2015 · 2 comments
Assignees
@ltegman

Comments

@raymondadouglas
Copy link

There needs to be a space after Latitude, and one before and after Longitude, and maybe a comma as well - $("#data").html("Latitiude " + position.coords.latitude + ", Longitude " + position.coords.longitude);

Turns
latitude-1.2677468longitude-1.2677568
into
Latitude -1.2677468, Longitude -1.2677568
@ltegman
Copy link
Member

ltegman commented Oct 29, 2015

Thanks for the report! We currently have PR #3986 open to fix this. It just needs a small update and then will be merged into the codebase 😄

@ltegman ltegman mentioned this issue Oct 30, 2015
Merged
@dcsan dcsan added the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label Oct 30, 2015
@bugron
Copy link
Contributor

bugron commented Oct 30, 2015

Closing this issue as #3986 is merged.

@bugron bugron closed this as completed Oct 30, 2015
@bugron bugron removed the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label Oct 30, 2015
ltegman added a commit to ltegman/FreeCodeCamp that referenced this issue Oct 30, 2015
@ltegman
JSON/API Lesson Cleanup
086b0bb 
+ Fixes Convert JSON to HTML lesson (Closes freeCodeCamp#4016)
+ Add Formatting to Geolocation lesson output (Closes freeCodeCamp#4010)
+ Fixes Render Images lesson
+ Removes unnecessary `?callback=` from JSON challenges. (Closes freeCodeCamp#3972)
+ Lots of other formatting and consistancy changes.
+ Also closes freeCodeCamp#3962 and closes freeCodeCamp#3969
@ghost ghost removed the in progress label May 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ltegman ltegman

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@dcsan @ltegman @raymondadouglas @bugron