Travel Mapping

Web Design Discussion => General Web Design Discussion => Topic started by: Jim on June 05, 2020, 09:45:34 pm

Title: New Mapview on tmtest
Post by: Jim on June 05, 2020, 09:45:34 pm
A new version of Mapview is available for testing on tmtest.  It's getting much closer to the vision I've had for it for a long time.  Here's the quick summary:


I think this allows for some nice functionality. 


There are also some things not yet done.


I'm sure there are glitches.  Please give it a try and let me know both the good and the bad.
Title: Re: New Mapview on tmtest
Post by: yakra on June 05, 2020, 10:27:01 pm
http://tmtest.teresco.org/user/mapview.php?units=miles&u=harvey&v works, but
http://tmtest.teresco.org/user/mapview.php?units=miles&u=harvey does not.

I input a location, E.G. "portsmouth NH" and click OK, and the "Loading Data..." msg remains indefinitely, with no signs of progress.

Quote
I have not yet added rte= support, pending some discussions in the thread I created earlier today.  I am unconvinced that it's useful.
Very useful IMO, and a feature I use frequently. I'll post more in the other thread.
Title: Re: New Mapview on tmtest
Post by: Jim on June 05, 2020, 11:01:33 pm
http://tmtest.teresco.org/user/mapview.php?units=miles&u=harvey&v works, but
http://tmtest.teresco.org/user/mapview.php?units=miles&u=harvey does not.

I input a location, E.G. "portsmouth NH" and click OK, and the "Loading Data..." msg remains indefinitely, with no signs of progress.

The progress was very quick - no data was loaded, which resulted in the message remaining.  In any case, I've just fixed it so if no rg= or sys= (or later, rte= or country=) are specified, it forces the behavior as if the v QS parameter was there so we see something at least.

Quote from: yakra
Quote
I have not yet added rte= support, pending some discussions in the thread I created earlier today.  I am unconvinced that it's useful.
Very useful IMO, and a feature I use frequently. I'll post more in the other thread.

OK, should be easy enough to get the old behavior, or at least most of it.
Title: Re: New Mapview on tmtest
Post by: Jim on June 06, 2020, 05:23:18 pm
Added feature: dropdowns to color segments by system (original behavior), number of concurrent segments, or all black, and to highlight traveled, untraveled, all, or no segments.

More to come.
Title: Re: New Mapview on tmtest
Post by: vdeane on June 06, 2020, 05:53:03 pm
I think I'm falling in love with this version of mapview.  The box to enable/disable data update is REALLY useful, and makes it possible to browse TM data the same way one would browse Google Maps.

One thing I have noticed: disabling data update, scrolling to another continent, and then enabling data update can result in a state where the update gets hung and no data loads (this is also the case if you scroll somewhere not contiguous with currently visible data in the one live on the main site); in that case, using the jump function and searching for the location you want is necessary to get data displaying again.

I've been loving the highlight and coloring options.  It's really great for browsing what routes are around in different locations, even in regions I haven't traveled in.
Title: Re: New Mapview on tmtest
Post by: Jim on June 06, 2020, 08:53:01 pm
Another long-time personal wish list item is in.  You can now color by number of TM users who have traveled each segment.  It's another option in the dropdown for how to choose colors.  It uses a similar color scale as in the tables with blue being many (relative to visible data, so the most-traveled route will show up as blue) down through the greens, yellows, oranges, to red being few travelers.  Special cases of purple for 1 (a TM "pioneer") and gray for 0 (not yet relevant to any TM user).
Title: Re: New Mapview on tmtest
Post by: Jim on June 06, 2020, 10:33:32 pm
Noted a couple problems: the more complicated queries to get traveler counts seems to have changed the order in which segments come out and has broken the code in new Mapview that merges segments across hidden waypoints, and moving around on the map can sometimes break the color scheme based on traveler counts...

Edit: both were fixed late Saturday evening.
Title: Re: New Mapview on tmtest
Post by: si404 on June 07, 2020, 07:27:54 am
At first I was a little bit skeptical that scrolling map would be useful, but the number of times I've loaded the whole of England (which is a lot) to just look at one small area.

I prefer trying to find broken concurrencies with mapview as the interface is easier than HDX, so the addition of graph data like concurrencies is really good - I can check for broken concurrencies with ease.

I never particularly cared about the number of travelers, but scrolling around South East England and seeing a variety of colors - even on minor routes/Central London routes that people probably walked rather than drove - including quite a bit of purple that isn't me (though I do have a lot of purple), and not big chunks of grey - is rather encouraging.
Title: Re: New Mapview on tmtest
Post by: Jim on June 07, 2020, 12:16:53 pm
Fun (to me, anyway) new feature just now:

In any popup when you click on a connection, if you click on the "X Travelers" line and subsequently hover over it, you get the list of travelers of that segment.  If you hover before clicking, you get instructions to do so.  It's done this way to avoid what seems to be an expensive operation to get all that info up front, but really quick to get for individual segments on-demand.  Open to ideas of a better UI, subject to the requirement that the user would need to take some action to trigger loading of the data to prevent lots of apache and DB hits to load info that wouldn't be used in many cases.
Title: Re: New Mapview on tmtest
Post by: michih on June 07, 2020, 12:45:14 pm
Fun (to me, anyway) new feature just now:

Thanks :) I already missed this feature the earlier today. Is it possible to open the list with the click? It's a little bit tricky right now.

And about the location selection window... I'm used to press Enter after selection from HDX. Is it possible to execute the "Press to search and set coordinats" command with Enter from the input field?
Title: Re: New Mapview on tmtest
Post by: Jim on June 07, 2020, 01:31:06 pm
Is it possible to open the list with the click? It's a little bit tricky right now.

Yes, that's what I'd like.  I probably need to implement a custom tooltip rather than relying on the title attribute to have more control over.  https://github.com/TravelMapping/Web/issues/468

Quote
And about the location selection window... I'm used to press Enter after selection from HDX. Is it possible to execute the "Press to search and set coordinats" command with Enter from the input field?

Just figured it out and it's on tmtest.
Title: Re: New Mapview on tmtest
Post by: mapcat on June 07, 2020, 02:05:48 pm
Very addicting! Love the new features.

@Jim, since you indicated before that this was not your intention, I'll point out again that devel routes (cannf, mexsf, probably others) are still visible.
Title: Re: New Mapview on tmtest
Post by: Jim on June 07, 2020, 03:06:26 pm
Very addicting! Love the new features.

@Jim, since you indicated before that this was not your intention, I'll point out again that devel routes (cannf, mexsf, probably others) are still visible.

Thanks!

I believe I just put in the fix to filter out devel routes.
Title: Re: New Mapview on tmtest
Post by: mapcat on June 07, 2020, 03:54:36 pm
I believe I just put in the fix to filter out devel routes.

That was quick. Not seeing them anymore.
Title: Re: New Mapview on tmtest
Post by: yakra on June 07, 2020, 04:57:41 pm
Open to ideas of a better UI, subject to the requirement that the user would need to take some action to trigger loading of the data to prevent lots of apache and DB hits to load info that wouldn't be used in many cases.
Is it possible to open the list with the click? It's a little bit tricky right now.
Not terribly often, but occasionally when using traveled graphs in the HDX I'd find myself wishing for something (Dialog box? Alert?) I could highlight & copy, for debug purposes.
Title: Re: New Mapview on tmtest
Post by: Jim on June 07, 2020, 10:34:30 pm
A couple more features to wrap up this Sunday:


Title: Re: New Mapview on tmtest
Post by: Jim on June 07, 2020, 10:54:34 pm
You can specify a cr= to give any root (such as ny.i084) and you will see all routes in the same connected route.  rg= or sys= restrictions here don't make much sense in my opinion, so they are ignored if specified in conjunction with cr=.

I've replaced the links on the table of routes in systems.php to use cr= so you now get only the actual connected route listed in that table.
Title: Re: New Mapview on tmtest
Post by: Jim on June 07, 2020, 11:01:50 pm
Is anyone aware of any old Mapview functionality that's not been replaced with equivalent or better functionality on the new version on tmtest?  I'd like to have it go live soon.
Title: Re: New Mapview on tmtest
Post by: mapcat on June 07, 2020, 11:08:51 pm
I've replaced the links on the table of routes in systems.php to use cr= so you now get only the actual connected route listed in that table.

For the I-195 page, each instance of I-195 is treated as a separate route when you select any of the seven entries in the table, which is probably what you intended. But the MA and RI routes ought to be treated as a connected route.
Title: Re: New Mapview on tmtest
Post by: yakra on June 07, 2020, 11:09:11 pm
Ooh, nice!
http://tmtest.teresco.org/user/mapview.php?units=miles&u=yakra&cr=nh.nh153
If I zoom in until only a segment of PickPocRd <-> NH/ME is onscreen, the Visible Segments stats still = the Route Overall stats.
If I change to
http://tmtest.teresco.org/user/mapview.php?units=miles&u=yakra&cr=nh.nh153&v
Always Update Visible Routes is checked, and things proceed accordingly.
Looks like the presence/absence of the &v parameter controls whether Always Update Visible Routes is checked?
Title: Re: New Mapview on tmtest
Post by: Jim on June 07, 2020, 11:19:48 pm
I've replaced the links on the table of routes in systems.php to use cr= so you now get only the actual connected route listed in that table.

For the I-195 page, each instance of I-195 is treated as a separate route when you select any of the seven entries in the table, which is probably what you intended. But the MA and RI routes ought to be treated as a connected route.

When I click on the I-195 Rhode Island-Massachusetts link on http://tmtest.teresco.org/user/system.php?units=miles&u=terescoj&sys=usai, I get the whole connected route in mapview.
Title: Re: New Mapview on tmtest
Post by: Jim on June 07, 2020, 11:21:23 pm
Ooh, nice!
http://tmtest.teresco.org/user/mapview.php?units=miles&u=yakra&cr=nh.nh153
If I zoom in until only a segment of PickPocRd <-> NH/ME is onscreen, the Visible Segments stats still = the Route Overall stats.
If I change to
http://tmtest.teresco.org/user/mapview.php?units=miles&u=yakra&cr=nh.nh153&v
Always Update Visible Routes is checked, and things proceed accordingly.
Looks like the presence/absence of the &v parameter controls whether Always Update Visible Routes is checked?

Yes, that's used for initial setting of the checkbox, and if the checkbox is selected, the route/region/system/whatever would be first loaded, then remaining routes in the view.
Title: Re: New Mapview on tmtest
Post by: mapcat on June 08, 2020, 08:50:31 am
When I click on the I-195 Rhode Island-Massachusetts link on http://tmtest.teresco.org/user/system.php?units=miles&u=terescoj&sys=usai, I get the whole connected route in mapview.

Sorry, you can ignore this. I started at system.php and then must have switched over to mapview without thinking, and expected the routes listed in the route table to go to connected routes, but now see that that isn't what mapview is designed to do. So everything seems to be working as designed.
Title: Re: New Mapview on tmtest
Post by: michih on June 08, 2020, 12:32:31 pm
Is there a list which QS parameter we have and where they are supported?

I'm aware of rg, sys, rte, cr, country, u, units, lat, lon, zoom
Title: Re: New Mapview on tmtest
Post by: Jim on June 08, 2020, 12:42:55 pm
Is there a list which QS parameter we have and where they are supported?

I'm aware of rg, sys, rte, cr, country, u, units, lat, lon, zoom

Hopefully up-to-date at the top of the PHP file:

https://github.com/TravelMapping/Web/blob/mapview/user/mapview.php
Title: Re: New Mapview on tmtest
Post by: yakra on June 08, 2020, 03:29:05 pm
https://github.com/TravelMapping/Web/blob/7088765a52d782842cb9c1bea59d9b9cc86e6ad0/user/mapview.php#L19
should be lon, not lng.
Title: Re: New Mapview on tmtest
Post by: Jim on June 08, 2020, 05:34:00 pm
A legend has been added for Mapview segment coloring.  I'm considering ways to improve it now that I have the mechanism down.

Is it worth space in the control panel at the top to have a toggle to display/hide the legend?
Title: Re: New Mapview on tmtest
Post by: yakra on June 08, 2020, 05:48:49 pm
The legend is unobtrusive IMO.

Color by Concurrencies:
Thoughts on just using red for anything with 5+ routes, to have the same behavior as the HDX?
http://tmtest.teresco.org/user/mapview.php?units=miles&u=harvey&rg=IN
Using white for 9 concurrencies seems odd; was this intended? I don't see white segments on the map (looking mainly at the Indy beltway).
Adding some quick-n-dirty temp code to siteupdate to check for 9+ route concurrencies, I see only IN I-465 46 47 and pals. This is displayed in gray.

Highway System Colors:
Thoughts on including both the clinched & unclinched colors in the legend?
Title: Re: New Mapview on tmtest
Post by: michih on June 09, 2020, 02:42:29 am
A legend has been added for Mapview segment coloring.  I'm considering ways to improve it now that I have the mechanism down.

OMG, it looks like...... I could click and change the color... or click and uncheck (remove) the system from mapview..... I should not think about this.... ;)

Color by travel count: None, 1, 2, 7, 12, 18 is fine but a segment with 17 travels has the color indicated for 18. Maybe None, 1, 2, 3..7, 8..12, 13..18 etc?

Does "visible routes: plain" make sense? The black/grey style looks odd (maybe it would make sense if we count change the colors).


Is it worth space in the control panel at the top to have a toggle to display/hide the legend?

yes!

Highway System Colors:
Thoughts on including both the clinched & unclinched colors in the legend?

Seconded! Just next to it, maybe even without any pixel space in-between.
Title: Re: New Mapview on tmtest
Post by: Jim on June 09, 2020, 04:54:10 pm
Color by travel count: None, 1, 2, 7, 12, 18 is fine but a segment with 17 travels has the color indicated for 18. Maybe None, 1, 2, 3..7, 8..12, 13..18 etc?

Every value has a different color (up to 240 max travelers).  I need to look into a gradient scale instead of the fixed boxes.

Quote
Does "visible routes: plain" make sense? The black/grey style looks odd (maybe it would make sense if we count change the colors).

Do you mean in the legend, or at all?
Title: Re: New Mapview on tmtest
Post by: Jim on June 09, 2020, 08:14:21 pm
The tmtest version has a few improvements since I put this on the main site:

- toggle legend on/off
- enough colors for current max number of concurrencies
- a gradient scale in the legend for number of travelers (when there are at least two segments with different numbers of travelers > 2)

Title: Re: New Mapview on tmtest
Post by: kjslaughter on June 09, 2020, 11:08:26 pm
Wow!  I love all the new colors on the tables, the scroll-able maps and then today seeing all the counts by user and the ability to hover to see the users is just amazing. I can see how I'm the TM pioneer on a LOT of Georgia state highways as I've explored courthouses.

Great job!!
Title: Re: New Mapview on tmtest
Post by: michih on June 10, 2020, 01:03:05 pm
Does "visible routes: plain" make sense? The black/grey style looks odd (maybe it would make sense if we count change the colors).

Do you mean in the legend, or at all?

At all. I mean, the colors look like this: https://user-images.githubusercontent.com/13720877/84198040-f31c5200-aa70-11ea-80bf-15ef2f45be20.png which is..... a little bit ugly.

But it's fine if anyone needs it....
Title: Re: New Mapview on tmtest
Post by: michih on June 10, 2020, 01:12:07 pm
Color by travel count: None, 1, 2, 7, 12, 18 is fine but a segment with 17 travels has the color indicated for 18. Maybe None, 1, 2, 3..7, 8..12, 13..18 etc?
Every value has a different color (up to 240 max travelers).  I need to look into a gradient scale instead of the fixed boxes.

I just talked about the legend. I'd like to see the actual range, e.g. 3..7 instead of the upper limit only because it could be the upper or lower limit.

If the actual limits are
None, 1, 2, 3..7, 8..12, 13..18 you could indicate
None, 1, 2, 3..7, 8..12, 13..18 in the legend or
None, 1, 2, <= 7,  <= 12,  <= 18 or
None, 1, 2, < 8 , < 13, < 19

OTOH, you use a gradient scale now..... but still, I need to figure out that it is the upper limit. I think it is not intuitive
Title: Re: New Mapview on tmtest
Post by: Jim on June 10, 2020, 01:34:29 pm
If the actual limits are
None, 1, 2, 3..7, 8..12, 13..18 you could indicate
None, 1, 2, 3..7, 8..12, 13..18 in the legend or
None, 1, 2, <= 7,  <= 12,  <= 18 or
None, 1, 2, < 8 , < 13, < 19

OTOH, you use a gradient scale now..... but still, I need to figure out that it is the upper limit. I think it is not intuitive

There are no actual limits, since it's a somewhat continuous gradient.  I'm open to ideas how better to indicate that the two numbers displayed are the exact bounds, or if some kind of intermediate numbers make sense.
Title: Re: New Mapview on tmtest
Post by: michih on June 10, 2020, 02:12:42 pm
There are no actual limits, since it's a somewhat continuous gradient.  I'm open to ideas how better to indicate that the two numbers displayed are the exact bounds, or if some kind of intermediate numbers make sense.

Maybe "Limit" is the wrong word but you indicate colors and specify a number to it. To exactly this color.

Maybe it's not relevant though..... if the legend is intuitive to everyone but me.
Title: Re: New Mapview on tmtest
Post by: Jim on June 10, 2020, 04:01:01 pm
Right, "limits" are not the correct word, I think "bounds" would be.

Let's say the range is 5 to 76 travelers.  The current implementation chooses a color for each segment based on its number of travelers, using one of the 77 colors along the way around the color wheel from red (which would be 0, though that is a special case to draw in gray) to blue for 76.  The only ones I think it makes sense to label are the smallest in use (among those >= 2) and largest, so they're positioned to be lined up with the top and bottom.

I suppose a way to incorporate more information would be to have 4 labels next to the 4 squares that together make up the gradient, each with something like 5-22, 23-40, 41-58, and 59-76.  It seems that would be more confusing, though.
Title: Re: New Mapview on tmtest
Post by: michih on June 11, 2020, 06:22:32 am
I like the current legend implementation very much! Quite perfect! :)
Title: Re: New Mapview on tmtest
Post by: Jim on June 12, 2020, 08:49:25 pm
Another wish list item of mine is coming along, enabled by the https conversion.  I've added a button to the tmtest Mapview version (be sure to specify https://, as I didn't bother with redirection on the test site) where you can query the current location and use it to set your Mapview location.  It's of course dependent on your using a device that knows where it is, and you having given permission to TM to query it.
Title: Re: New Mapview on tmtest
Post by: Jim on June 12, 2020, 08:51:56 pm
My next step is to get a "you are here" kind of marker onto maps when that info is available.  So don't be surprised if I'm breaking tmtest's version on and off as I work on that.
Title: Re: New Mapview on tmtest
Post by: Jim on June 12, 2020, 09:42:15 pm
My next step is to get a "you are here" kind of marker onto maps when that info is available.  So don't be surprised if I'm breaking tmtest's version on and off as I work on that.

That was an easy addition, and it also now on tmtest.  For now, if the information is available from the browser, a blue marker is displayed at your current location.  A short walk around the block with my iPad verified that the marker's location follows.  Looking forward to try it out on the car or better yet on an actual road trip.
Title: Re: New Mapview on tmtest
Post by: michih on June 13, 2020, 02:35:36 am
I open https://tmtest.teresco.org/user/mapview.php?u=michih&v and get a browser (FF) message that tmtest.teresco.org tries to get my location. I press "allow" but nothing happens. When I press the "Press to query Current Location and Set Coords" button* again, the browser message appears again. When I open the FF settings - location permission settings - there is a list of websites which have tried to get the location. The list is empty. I use FF private mode. Does anyone else have this problem?


*I think that a short "Get Current Location" would be enough. Same with "Press to Search and Set Coords", this could be "Search Location" or whatever. btw: When I type in something stupid, e.g. "ewfssr4r" there could be a message "invalid location" or "location not found" or whatever message.
Title: Re: New Mapview on tmtest
Post by: Jim on June 13, 2020, 08:45:26 am
I open https://tmtest.teresco.org/user/mapview.php?u=michih&v and get a browser (FF) message that tmtest.teresco.org tries to get my location. I press "allow" but nothing happens. When I press the "Press to query Current Location and Set Coords" button* again, the browser message appears again. When I open the FF settings - location permission settings - there is a list of websites which have tried to get the location. The list is empty. I use FF private mode. Does anyone else have this problem?

I had a similar problem initially in Chrome on my Mac, and for whatever reason, a force reload wasn't enough.  I had to clear out some browser info (I picked a few options, don't remember which, and don't know which helped) before I could get it to start receiving the location events properly.  On Firefox, it worked first try, as it did in Chrome on my iPad.

The Leaflet interface for getting the location information is very simple, so I think most problems will be with browser permissions.  It's possible I can do something to help on the coding side, but there's not much to change.
Title: Re: New Mapview on tmtest
Post by: vdeane on June 13, 2020, 03:21:50 pm
I've been having issues on that front too - it throws an unknown error acquiring geolocation.  That said, it's not just a TMtest problem, nor is it even just a Chrome problem (Firefox does it too), so I'm guessing it's an ISP problem (I've been having these issues ever since the pandemic increased internet demand).  Interestingly, Chrome will report current location as somewhere in Kansas - sometimes - (I'm guessing the geographic center of the lower 48; this happens for the display marker, but not for setting coordinates, which just fails and uses the default) when it can't get it, but Firefox just leaves the map location blank.

Also, I have to manually type the https for tmtest - otherwise, I get a "not found" or "not secure" error.
Title: Re: New Mapview on tmtest
Post by: Jim on June 13, 2020, 03:48:54 pm
I was actually pretty surprised that my home network through Spectrum pinpoints my location down to the marker being over my house.  To me, this is more of a mobile device feature anyway.

Related question.  What's more obtrusive: the marker being on every map to show your current location if it's detected, even when you don't really care about it, or a checkbox on each page that has maps to enable/disable the feature?
Title: Re: New Mapview on tmtest
Post by: michih on June 13, 2020, 04:06:32 pm
To me, this is more of a mobile device feature anyway.

I think it's also a nice feature on the notebook when you are in a hotel on a roadtrip.

I tried it on my phone. The message that the website wants to get my location appears earlier. Already when I load the page, not when I press the button. And it does also not work. I get a "timeout error" now.
Title: Re: New Mapview on tmtest
Post by: vdeane on June 13, 2020, 04:21:49 pm
For the legend, it might be useful to show full system names rather than abbreviations.  Granted, that would make the legend larger and a bit more obtrusive... but there's a checkbox to turn that on and off anyways.
Title: Re: New Mapview on tmtest
Post by: yakra on June 13, 2020, 04:31:20 pm
For the legend, it might be useful to show full system names rather than abbreviations.  Granted, that would make the legend larger and a bit more obtrusive... but there's a checkbox to turn that on and off anyways.
Thought about that too. Maybe use the system code on the legend, with the full name in a tooltip upon mouseover?

I was actually pretty surprised that my home network through Spectrum pinpoints my location down to the marker being over my house.  To me, this is more of a mobile device feature anyway.
Yikes...  ???

Related question.  What's more obtrusive: the marker being on every map to show your current location if it's detected, even when you don't really care about it, or a checkbox on each page that has maps to enable/disable the feature?
I'd prefer having the checkbox.
Title: Re: New Mapview on tmtest
Post by: Jim on June 13, 2020, 04:36:56 pm
Currently on tmtest, any map that gets created is set to query location periodically (just using default frequency, whatever that is, which might be device-dependent).  So if you haven't already enabled location for your browser for the site, it should ask then.  I don't know why it would ask a second time when you try to get the position from Mapview's popup unless you don't click the "remember this" type of option.

So has anyone other than me been able to choose your location in the Mapview popup, or see your location on all the maps?  I've added some console log messages, so if anyone is willing to try it out and can let me know if you're seeing some messages in the console, it might help figure out if I'm doing something incorrectly here.  This is what I see in Firefox on successful use of geolocation in Mapview, with the last message repeated each time it queries my location again:

Code: [Select]
Enabling map.locate tmjsfuncs.js:316:13
mapLocationFound: created marker at LatLng(42.956189, -74.199605) tmjsfuncs.js:294:10
mapLocationFound: updated marker to LatLng(42.956189, -74.199605) tmjsfuncs.js:298:10
mapLocationFound: updated marker to LatLng(42.956183, -74.199612) tmjsfuncs.js:298:10
mapLocationFound: updated marker to LatLng(42.956191, -74.199624) tmjsfuncs.js:298:10
Title: Re: New Mapview on tmtest
Post by: oscar on June 13, 2020, 04:56:50 pm
I was actually pretty surprised that my home network through Spectrum pinpoints my location down to the marker being over my house.  To me, this is more of a mobile device feature anyway.

Related question.  What's more obtrusive: the marker being on every map to show your current location if it's detected, even when you don't really care about it, or a checkbox on each page that has maps to enable/disable the feature?

I'd prefer the latter, so I can disable the feature. I rarely run Mapview for the region where I am at the moment, more often to look ahead to a region I plan to travel at a later date, or where I'm maintaining route files.
Title: Re: New Mapview on tmtest
Post by: Jim on June 13, 2020, 10:37:14 pm
I've got a toggle in there now, will save its value in a cookie after I fix the problem with cookies noted in https://github.com/TravelMapping/Web/issues/486 .
Title: Re: New Mapview on tmtest
Post by: vdeane on June 13, 2020, 11:48:55 pm
I've got a toggle in there now, will save its value in a cookie after I fix the problem with cookies noted in https://github.com/TravelMapping/Web/issues/486 .
Would it be possible to move the button to select the map style down a bit?  For me, part of it is cut off by the top bar (now that I've noticed it, I see it did this even before the current location checkbox was implemented, but only when mousing over to select a style, not all the time).
Title: Re: New Mapview on tmtest
Post by: michih on June 14, 2020, 03:01:24 am
It works. Love it! :) However, the marker is about 8 meter of from the router and about 10 meter from my notebook :D
Title: Re: New Mapview on tmtest
Post by: Jim on June 14, 2020, 10:51:33 pm
There's now a checkbox on Mapview and the top of the HB to enable/disable the marker for your current location, and the setting is remembered in a cookie.

I'm hoping to get this along with the fix, which should be transparent to users, to set cookies properly for secure connections, up onto the main site early Monday so I can move on to the next tasks.
Title: Re: New Mapview on tmtest
Post by: SSOWorld on June 14, 2020, 11:28:07 pm
Any way to change default location? I'm not saying the Albany area's bad, but...
Title: Re: New Mapview on tmtest
Post by: oscar on June 14, 2020, 11:47:47 pm
Any way to change default location? I'm not saying the Albany area's bad, but...

I have my Mapview bookmark defaulting to show California, though it does start off centered on Siena College before taking me to California or some other region.
Title: Re: New Mapview on tmtest
Post by: yakra on June 15, 2020, 01:55:43 am
Is it feasible to start off at the location in the cookie, then reset the map to whatever other location, with no cookie -> Siena?
Title: Re: New Mapview on tmtest
Post by: michih on June 15, 2020, 04:27:05 am
Any way to change default location? I'm not saying the Albany area's bad, but...

I think that there must be any default. Should it be at 0°/0°?

Is it feasible to start off at the location in the cookie, then reset the map to whatever other location, with no cookie -> Siena?

A "last coords" cookie (don't like!) or a cookie just for the personal default (don't like!)?

I'd leave it as-is....
Title: Re: New Mapview on tmtest
Post by: si404 on June 15, 2020, 05:11:53 am
I think that there must be any default. Should it be at 0°/0°?
Being presented with just the Gulf of Guinea and nothing else on the map would definitely have people type in where they want, rather than scroll!

Sienna's fine, though the option to set a different default also works for me.
Title: Re: New Mapview on tmtest
Post by: Jim on June 15, 2020, 08:47:33 am
The cookie is only to remember the checkbox setting to enable or disable the current location marker.  Since the only time the Siena location (centered right on my office, BTW, in case anyone wants to stop by, post-pandemic) is used it in Mapview before a location is selected or when bad parameters are presented, I haven't been too worried about it.  Maybe the most useful thing would be to make the default location in the popup fill in by default with the currently-detected location on initial load, when that info is available.  I'm also open to a "default start location" kind of thing in a cookie that could be implemented as part of the control panel as proposed in https://github.com/TravelMapping/Web/issues/351 .
Title: Re: New Mapview on tmtest
Post by: Jim on June 15, 2020, 05:01:11 pm
The tmtest version of Mapview now offers selection to load by region and/or system in the popup that comes up on a generic page load or when you hit the "Jump" button.  If you choose at least one region or system, the coords and zoom level will be ignored, and auto-loading of all visible segments is deselected.

I don't think there's a great way to support rte= or cr= through the popup.  country= can be done, but looks to require quite a bit more work for what I suspect is little benefit over having to select all regions (use shift to select multiple) or by just reloading the page with country= in the QS params.

Title: Re: New Mapview on tmtest
Post by: michih on June 16, 2020, 01:43:07 pm
I don't think there's a great way to support rte= or cr= through the popup.

I never got why you wanted to support this. Except... if you want to use it as HB..... (https://forum.travelmapping.net/index.php?topic=3651.msg19373#msg19373) I think that this could be really nice!

country= can be done, but looks to require quite a bit more work for what I suspect is little benefit over having to select all regions (use shift to select multiple) or by just reloading the page with country= in the QS params.

To be honest, I tested the multi-selection but.... don't like it. I'd like to have a selection by country (and by continent) - in general*. What's the additional effort? We have "countries.csv", "continents.csv", "regions_countries_continents.csv" and "regions.csv". Is country and continent info not in DB?

*Also and especially for stats!
Title: Re: New Mapview on tmtest
Post by: Jim on June 16, 2020, 01:54:35 pm
I'd like to have a selection by country (and by continent) - in general*. What's the additional effort? We have "countries.csv", "continents.csv", "regions_countries_continents.csv" and "regions.csv". Is country and continent info not in DB?

*Also and especially for stats!

The info is all there in the DB, but not the JS/PHP/SQL to make it all happen.  I don't think it makes a whole lot of sense to have it in Mapview given how much data most would load in, but I think it makes total sense to have stats pages by country and continent in addition to the current region and system.  I'll make an Issue.  Added to Issue: https://github.com/TravelMapping/Web/issues/360