-
Notifications
You must be signed in to change notification settings - Fork 188
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
Conversation
Yaml testing using Travis (DLu#102)
Just added preloader, still not added markers yet. |
@DLu can I move the data from yaml to json? As it has many advantages over the current yaml,
|
you can check the current map at https://prajankya.github.io/ros_map/map.html before merging |
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. |
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. |
@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.. |
@DLu can you me tell if anything is wrong with this PR? |
@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? |
Sure, Will do..! and congrats for your first kid !! |
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.. |
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." |
Sure, will do.!! I agree on your point of view. I am a bit pre-occupied, but I'lll soon start on this one.. |
Closing for now. |
No description provided.