Skip to content
This repository has been archived by the owner on May 27, 2024. It is now read-only.

Route polylines don't show when visiting direct link to route layers #401

Open
barbeau opened this issue Apr 25, 2017 · 3 comments
Open
Assignees
Labels

Comments

@barbeau
Copy link
Member

barbeau commented Apr 25, 2017

Summary:

If you go to:
https://maps.usf.edu/#layers/usf_A,usf_B,usf_C,usf_D,usf_E,usf_F

...you should see all USF buses and route polylines on the map.

For some reason, the polylines are currently missing:

image

It should look like:

image

Steps to reproduce:

Go to https://maps.usf.edu/#layers/usf_A,usf_B,usf_C,usf_D,usf_E,usf_F

Expected behavior:

You should see route polylines on the map

Observed behavior:

No route polylines are shown

Workaround - go to https://maps.usf.edu/#layers/ and manually check the layers

Platform:

Windows 7 Enterprise w/ Chrome Version 57.0.2987.133 (64-bit)

@barbeau barbeau added the bug label Apr 25, 2017
@jmfield2
Copy link
Contributor

jmfield2 commented Apr 25, 2017

Looks like it might be a race condition waiting for the route geometries to load from the server.. It also throws a javascript error at:

for(v=0; v < this.vehicles.length; v++) {

because the vehicles weren't defined - although they do eventually load it seems since those refresh every 5 seconds. But the geometries are only loaded on initialize: and so, somehow they don't seem to load or be assigned to the correct (this) context..

@barbeau
Copy link
Member Author

barbeau commented Apr 26, 2017

Thanks @jmfield2!

@barbeau
Copy link
Member Author

barbeau commented Jul 12, 2017

From @liquidsunshine, regarding priority of some of these related issues for using the app to show information in the CUTR lobby on a TV display:

Here’s how I would prioritize them, based on CUTR’s needs:

  1. Route polylines don't show when visiting direct link to route layers #401 (a bug)
  2. Make map zoom and center configurable via URL parameters #403 (allows us to automatically center on CUTR)
  3. Add URL parameter to turn off "Don't use while driving" #404 (allows us to automatically bypass popup)
  4. See if we can support an additional map zoom level #405 (offers a better view of CUTR)
  5. Make client-side refresh rate configurable via URL parameter #402 (possibly improves refresh performance)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants