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

Map page massive upgrade #103

Closed
wants to merge 12 commits into from
Closed

Map page massive upgrade #103

wants to merge 12 commits into from

Conversation

prajankya
Copy link
Contributor

No description provided.

@prajankya
Copy link
Contributor Author

Just added preloader, still not added markers yet.

@prajankya
Copy link
Contributor Author

@DLu can I move the data from yaml to json?

As it has many advantages over the current yaml,

  • KML layer is not necessary and can just be made with markers.
  • If KML is not required then generate_kml.py is also not required.
  • Also yaml by itself would need conversion library, so json would be better.
  • As JSON is a bit better tolerable to indentations, it is lesser prone to creating problems in PRs, hence no Linter required.

@prajankya
Copy link
Contributor Author

@DLu few other queries,

Do you need to have that Pie chart ? I was planning on putting check-boxes in the legend of the map, which can be selected to show or hide that type.

Also, I have addressed #95 any other things you wish to address?

@prajankya
Copy link
Contributor Author

you can check the current map at https://prajankya.github.io/ros_map/map.html before merging

@prajankya
Copy link
Contributor Author

Had to add API key for the https://prajankya.github.io/ros_map/map.html to work,

so added an API key with limits of referrer, so only two domains can use that API key,

prajankya.github.io and metrorobots.com

of which first one is just temporary for to check the current status of the upgrade.

@prajankya
Copy link
Contributor Author

You can check the Map at https://prajankya.github.io/ros_map/map.html

I have moved from yaml to json, hope you consider what I have done. Any suggestions greatly appreciated.

@prajankya
Copy link
Contributor Author

@DLu I have done how much I have thought, if any ideas or changes please tell. Hope you like it, if its too much UI(I am feeling like it) just tell..

@prajankya prajankya changed the title Started with upgrading the Map page Map page massive upgrade Aug 7, 2017
@prajankya
Copy link
Contributor Author

@DLu can you me tell if anything is wrong with this PR?

@DLu
Copy link
Owner

DLu commented Dec 6, 2017

@prajankya My sincere apologies for ignoring this for so long. Right as you sent it to me was right around when I had my first kid, and that has been distracting me for the past few months.

I really appreciate you putting some much-needed love into this repo, but right now, there's a bit too much going on in this one pull request. Would you be willing to break this up into a few smaller PRs with clear descriptions of what the new features are?

@prajankya
Copy link
Contributor Author

Sure, Will do..!

and congrats for your first kid !!

@prajankya
Copy link
Contributor Author

Can you just check out the link https://prajankya.github.io/ros_map/map.html as I frankly am not a UI designer, and hence am not confident upon whether the community would like it. If any suggestions, please tell me..

@DLu
Copy link
Owner

DLu commented Dec 11, 2017

On the whole, I think it is a nice interace. My major complaint is that it doesn't show all of the points when zoomed out. There's a tradeoff between showing too much and too little, but one of the cool things about the ROS users map is being able to take a quick glance and say "wow there's a lot of ROS users."

@prajankya
Copy link
Contributor Author

Sure, will do.!! I agree on your point of view. I am a bit pre-occupied, but I'lll soon start on this one..

@DLu
Copy link
Owner

DLu commented Jul 13, 2022

Closing for now.

@DLu DLu closed this Jul 13, 2022
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.

2 participants