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

Update index.html #20

Merged
merged 6 commits into from
Jul 29, 2016
Merged

Update index.html #20

merged 6 commits into from
Jul 29, 2016

Conversation

shadowsyllvet
Copy link
Contributor

Initial map location automatically updates based on user settings "unique_coordinates"
If "unique_coordinates" is set to false the map will initialize on user 0 lat, lng
if set to true the map will initialize on "standard_coordinates"

This can probably be better optimized but I just do patch work

Initial map location automatically updates based on user settings "unique_coordinates"
If "unique_coordinates" is set to false the map will initialize on user 0 lat, lng
if set to true the map will initialize on "standard_coordinates"
@shadowsyllvet
Copy link
Contributor Author

@seikur0 I just wanted you to see the html change before I added the PB support. This is my first time with a github project and I didn't realize it would push commits initiated after the pull request was made.

@seikur0
Copy link
Owner

seikur0 commented Jul 29, 2016

Okay I'll merge this, but I'll change some things after that, I'll comment some of the changes, so you can see why.


LAT_C=settings['standard_coordinates']['lat'];
LNG_C=settings['standard_coordinates']['lng'];
var centralized=settings['unique_coordinates'];
Copy link
Owner

Choose a reason for hiding this comment

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

No need to create a new variable here, just use it directly in the next line.

@shadowsyllvet
Copy link
Contributor Author

I only had a bare minimum of C++ experience with this.It was fun trying to figure out my syntax errors without a compiler telling me where I messed up. I appreciate you showing me better ways to do this.

@shadowsyllvet
Copy link
Contributor Author

@seikur0 Also, do you need me to make the changes on my end (or need me to do anything) before you merge this? I'm new to this.

@seikur0
Copy link
Owner

seikur0 commented Jul 29, 2016

Nope, I'll merge this and immediately update the files then, it's good.

@seikur0 seikur0 merged commit da3ef8a into seikur0:master Jul 29, 2016
@shadowsyllvet
Copy link
Contributor Author

I love how much neater and functional your code looks. The difference in skill is kind of staggering. My only experience with coding comes from cludge jobs like this using messy code until it does what I want it to.

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.

None yet

2 participants