Author Topic: Scrollable Mapview on tmtest  (Read 29856 times)

0 Members and 1 Guest are viewing this topic.

Offline michih

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 4878
  • Last Login:Today at 11:45:31 am
Re: Scrollable Mapview on tmtest
« Reply #30 on: June 01, 2020, 03:21:13 am »
-When you open the page, the first thing you see is something to enter in the lat/lon. Only after a few seconds of me looking at that and going "uhhh..." then I saw that I could enter in a city to go to. I think you should make this more prominent because nobody is going to know what lat/lon coordinates they want to start out at.

yep! I suggest moving "Find place" on top and hide the lon/lat (to be opened by "Extend").
And I like to have a button that my actual coordinates I directly loaded (well, only if the privacy settings of my device allows you to know my location)

It would be great to get any feedback that the location was found and that I need to press OK to load the map. It's confusing even after some attempts now.
edit: I realized now that there is a feedback. When the location was found, lon/lat is updated. There should be something more visible though (a check icon, OK button color change,...).


-Also, add an option to that screen to input username. It's kinda clunky to have to type that into the URL.

I reported this. It only happens when you open it from the motd link. I works when you open it from the user page. I suggest removing the link from the motd.
« Last Edit: June 01, 2020, 03:26:22 am by michih »

Offline yakra

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 4445
  • Last Login:Today at 04:12:46 pm
  • I like C++
Re: Scrollable Mapview on tmtest
« Reply #31 on: June 01, 2020, 08:08:26 am »
Putting the "Find place" box on top is fine.
But "Set Initial Location and Zoom Level" isn't much real estate to lose; hiding it would just serve to make things less usable: more clicking, a more convoluted interface...
If possible, maybe accept both lat & lon values in a single field, to facilitate pasting coords E.G. from popups from the HB or wptedit.

As for the functionality of OSM Nominatim itself, it thinks "Portland" is somewhere in the Pacific coast. Which we all know is factually incorrect. "Portland CT" maps to a road by that name in Great Yarmouth, Norfolk, and "Portland ME" maps to a local road in Hermosillo, Sonora. Oh what, I gawtta type out the full name theyah, Emm Eh Aye Enn Ee? That's a bit frickin' much bub.
Sri Syadasti Syadavaktavya Syadasti Syannasti Syadasti Cavaktavyasca Syadasti Syannasti Syadavatavyasca Syadasti Syannasti Syadavaktavyasca

Online Jim

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 2860
  • Last Login:Today at 08:28:42 pm
Re: Scrollable Mapview on tmtest
« Reply #32 on: June 01, 2020, 08:38:04 am »
-When you open the page, the first thing you see is something to enter in the lat/lon. Only after a few seconds of me looking at that and going "uhhh..." then I saw that I could enter in a city to go to. I think you should make this more prominent because nobody is going to know what lat/lon coordinates they want to start out at.

Definitely need to reorganize that.  Putting the Nominatim search field first as mentioned here and below makes a lot of sense.

Quote
-When you have the route table open, and the route table doesn't take up the whole length of the screen, you can't drag the map if you have the mouse cursor in that area.

That's been a longstanding issue with the table CSS we've been using to keep the table above the map and all of the overlays.  https://github.com/TravelMapping/Web/issues/314
Other than that, I think this is a great feature that will add a lot to this site.
[/quote]

Online Jim

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 2860
  • Last Login:Today at 08:28:42 pm
Re: Scrollable Mapview on tmtest
« Reply #33 on: June 01, 2020, 08:41:01 am »
The reason it's expensive (as far as I can tell) is that it needs to look at both endpoints of each of 874,000+ segments and see if either is within the visible area of the map, and that we need to join the segments with waypoints (twice) to get locations, and with clinched to get whether each segment is clinched for the traveler.
Thoughts on including beginning/end latitude/longitude directly in the segments table and eliminating the JOINs for a potential speedup?

Maybe.  Better might be to include a bounding box as an SQL GEOMETRY that would be more readily searchable.  I tried this on the waypoints table with just POINT values but it actually slowed things down.  For the smaller-sized queries, splitting out waypoint queries first then searching for segments sped things up enough that I stopped looking at the spatial functions in SQL.

Offline yakra

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 4445
  • Last Login:Today at 04:12:46 pm
  • I like C++
Re: Scrollable Mapview on tmtest
« Reply #34 on: June 01, 2020, 01:18:54 pm »
Quote
splitting out waypoint queries first then searching for segments
Got a link to where this is done in the new code?
Sri Syadasti Syadavaktavya Syadasti Syannasti Syadasti Cavaktavyasca Syadasti Syannasti Syadavatavyasca Syadasti Syannasti Syadavaktavyasca

Online Jim

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 2860
  • Last Login:Today at 08:28:42 pm
Re: Scrollable Mapview on tmtest
« Reply #35 on: June 01, 2020, 01:46:37 pm »
Quote
splitting out waypoint queries first then searching for segments
Got a link to where this is done in the new code?

https://github.com/TravelMapping/Web/blob/master/lib/getVisibleSegments.php

Offline vdeane

  • Sr. Member
  • ****
  • Posts: 430
  • Gender: Female
  • Last Login:Yesterday at 10:03:01 pm
    • New York State Roads
Re: Scrollable Mapview on tmtest
« Reply #36 on: June 02, 2020, 01:00:43 pm »
One weird thing I noticed: where state or US routes cross interstates, the state or US route draws on top of the interstate.  Overlaps are handled correctly, and this doesn't happen between US and state routes, so it's a little odd.  Regular mapview like we have for the regions doesn't do this.
Please note: All comments here represent my own personal opinion and do not reflect the official position of NYSDOT or its affiliates.

Offline yakra

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 4445
  • Last Login:Today at 04:12:46 pm
  • I like C++
Re: Scrollable Mapview on tmtest
« Reply #37 on: June 02, 2020, 01:23:19 pm »
One weird thing I noticed: where state or US routes cross interstates, the state or US route draws on top of the interstate.  Overlaps are handled correctly, and this doesn't happen between US and state routes, so it's a little odd.  Regular mapview like we have for the regions doesn't do this.
Polylines do not appear to be stacked by tier.
Looking at my travels in Worcester MA, MA70 renders atop I-290.
Already noted: https://github.com/TravelMapping/Web/issues/436
Sri Syadasti Syadavaktavya Syadasti Syannasti Syadasti Cavaktavyasca Syadasti Syannasti Syadavatavyasca Syadasti Syannasti Syadavaktavyasca

Online Jim

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 2860
  • Last Login:Today at 08:28:42 pm
Re: Scrollable Mapview on tmtest
« Reply #38 on: June 02, 2020, 01:57:22 pm »
The tmtest version for scollable Mapview now has:

- rearranged and clarified search box.
- user and units selection, stored in cookies for persistence.

I'm going to see if I can knock off the stacking order before putting this live on the main site.

Online Jim

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 2860
  • Last Login:Today at 08:28:42 pm
Re: Scrollable Mapview on tmtest
« Reply #39 on: June 02, 2020, 02:07:39 pm »
Stacking order is good on tmtest now too.  I'm going to see if I can tackle any other scrollable Mapview problems I'm aware of before a 3:30 meeting.

Offline michih

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 4878
  • Last Login:Today at 11:45:31 am
Re: Scrollable Mapview on tmtest
« Reply #40 on: June 02, 2020, 02:23:33 pm »
The tmtest version for scollable Mapview now has:
- rearranged and clarified search box.

Much better now! I just don't like the look. E.g. the OK button is almost invisible...

New: Reopen start position popup

Online Jim

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 2860
  • Last Login:Today at 08:28:42 pm
Re: Scrollable Mapview on tmtest
« Reply #41 on: June 02, 2020, 02:47:18 pm »
Now live on the main site:

- rearranged and clarified search box
- user and units selection in search box
- stacking order of polylines honors tiers
- can click the map in the space below the routes table

Offline yakra

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 4445
  • Last Login:Today at 04:12:46 pm
  • I like C++
Re: Scrollable Mapview on tmtest
« Reply #42 on: June 02, 2020, 02:49:02 pm »
- can click the map in the space below the routes table
:D
Sri Syadasti Syadavaktavya Syadasti Syannasti Syadasti Cavaktavyasca Syadasti Syannasti Syadavatavyasca Syadasti Syannasti Syadavaktavyasca

Offline michih

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 4878
  • Last Login:Today at 11:45:31 am
Re: Scrollable Mapview on tmtest
« Reply #43 on: June 02, 2020, 03:22:39 pm »
Does anyone else want routes 100% clinched being indicated with a different backcolor similar to how traveled segments on the HB route table are indicated?
https://github.com/TravelMapping/Web/issues/440

(or a filter to "remove" all routes 100% clinched to ease seeing untraveled segments for planning trips?)

Offline Markkos1992

  • TM Collaborator
  • Hero Member
  • *****
  • Posts: 3316
  • Last Login:Today at 07:41:36 pm
Re: Scrollable Mapview on tmtest
« Reply #44 on: June 02, 2020, 03:34:08 pm »
Does anyone else want routes 100% clinched being indicated with a different backcolor similar to how traveled segments on the HB route table are indicated?
https://github.com/TravelMapping/Web/issues/440

(or a filter to "remove" all routes 100% clinched to ease seeing untraveled segments for planning trips?)

I would support both.