Travel Mapping

User Discussions => Welcome & Notices => Topic started by: Jim on June 11, 2016, 10:41:36 pm

Title: Please test new region page
Post by: Jim on June 11, 2016, 10:41:36 pm
Today/tonight's task was the user/region.php page.

For example: http://tmtest.teresco.org/user/region.php?u=terescoj&rg=DE (http://tmtest.teresco.org/user/region.php?u=terescoj&rg=DE)

Only minor functionality changes here, as almost everything was behind the scenes.  The main thing is that the two text boxes for user and region are now drop-downs populated from the DB.

Next up is system.php, which needs almost identical changes to what I just did for region.php.  Not sure when I'll get to that, busy couple days coming up.  After that, only the shields/clinched.php page needs updating to complete my initial reorg/refactoring/cleanup/enhancement phase.
Title: Re: Please test new region page
Post by: michih on June 12, 2016, 03:03:17 am
The "(Full Map)" link is in the headline of the 3rd table. I'd prefer having the link close to the map. In addition, the 2nd bracket is included by accident.

Generally, I think the design could be improved.
1st line: Show markers
2nd line: User: (michih) Region: (Belgium) <Update Map and Stats>
3rd line: blank
4th line: Traveler Stats for michih in Belgium:

Suggestion:
1st line: Traveler Stats: User: (michih) Region: (Belgium) <Update Map and Stats> Show markers Full Map

or:

1st line: Traveler Stats: User: (michih) Region: (Belgium) <Update Map and Stats>
2nd line: Show markers Full Map

-----------------------

http://tmtest.teresco.org/user/region.php?u=michih&rg=FRA

andcg is displayed when selecting France. Andorra is a sovereign state though!
I guess it's a highway data error although I couldn't find any.
Title: Re: Please test new region page
Post by: si404 on June 12, 2016, 03:46:57 am
Quote from: michih on June 12, 2016, 03:03:17 am
andcg is displayed when selecting France. Andorra is a sovereign state though!
I guess it's a highway data error although I couldn't find any.
There's a andcg (well, the main tunnel route into Andorra that doesn't have a CG number) route in France to connect it to the French network: http://tmtest.teresco.org/hb/?r=fra.tunenv

As such, the system will appear.
Title: Re: Please test new region page
Post by: michih on June 12, 2016, 08:18:13 am
Quote from: si404 on June 12, 2016, 03:46:57 am
There's a andcg (well, the main tunnel route into Andorra that doesn't have a CG number) route in France to connect it to the French network: http://tmtest.teresco.org/hb/?r=fra.tunenv
As such, the system will appear.

OSM indicates that it's CG2 on Andorran territory. I guess it's minimum correct about the tunnel and I think it should be part of Andorra region only. Just my 2c.
Title: Re: Please test new region page
Post by: Jim on June 12, 2016, 08:32:47 am
Quote from: michih on June 12, 2016, 03:03:17 am
The "(Full Map)" link is in the headline of the 3rd table. I'd prefer having the link close to the map. In addition, the 2nd bracket is included by accident.

Not sure what you mean by "2nd bracket.".  Could you be more specific?
Title: Re: Please test new region page
Post by: michih on June 12, 2016, 09:44:35 am
Quote from: Jim on June 12, 2016, 08:32:47 am
Quote from: michih on June 12, 2016, 03:03:17 am
The "(Full Map)" link is in the headline of the 3rd table. I'd prefer having the link close to the map. In addition, the 2nd bracket is included by accident.

Not sure what you mean by "2nd bracket.".  Could you be more specific?

region.php line 478:
)</a> --> </a>)

Code: [Select]
<tr><th colspan="5">Stats by Route: (
<?php echo "<a href=\"/user/mapview.php?u=".$user."&rg=".$region."\">" ?>
Full Map)</a></th></tr>-->
Code: [Select]
<tr><th colspan="5">Stats by Route: (
<?php echo "<a href=\"/user/mapview.php?u=".$user."&rg=".$region."\">" ?>
Full Map</a>)</th></tr>
Title: Re: Please test new region page
Post by: si404 on June 12, 2016, 01:58:22 pm
Quote from: michih on June 12, 2016, 08:18:13 am
OSM indicates that it's CG2 on Andorran territory.
You're quite right. I shall change it.