What occured

1. Signed-in
2. Successfully added a photo: https://www.cyclestreets.net/location/158096/
3. Went to add another photo, and overlay slid down advising to sign in.
4. Signed in again and added another photo
5. Subsequent photos were added without the need to sign in again.

So the problem appears to be something to do with forgetting the first sign in.

However - the above was retried using the private browsing mode of Firefox and it did stay signed-in. So this problem may be hard to reproduce consistently.

Platform

Firefox Daylight 30.0 (3279) / iPhone 7

Photo

Scene on following this photo of the day link:
https://twitter.com/cyclestreets/status/1342017166267314178?s=21

Map

Closing the overlay shows map around current location, not County Down where the photo is located .

Where is it?

One reason users follow Photo of the day from Twitter, or follow a photo map location link more generally is to determine where the image is located.

Suggestion

Turn off “use current location mode” when following location link, allowing user to close overlay and explore area of photo.

The blog and donate iframes should be loaded only on demand.

This will need them to be written in by JS, rather than use the Iframe tag directly in the HTML.

If you load a route from a URL, i.e. /journey/xxxx/, and then drag a point, a JS error occurs, and route loading will not happen.

The photo flip animation is nice, but the X button at the top is not clickable and the text of the links at the bottom are reversed:

MacOS/Chrome.

Ideally we should compile dependencies with Babel, so there is a single .js file, though this adds some development/deployment complexity.

E.g. see https://stackoverflow.com/questions/39471896/how-to-compile-all-included-files-into-one-using-babel

Although it's possible to add the Photomap as a full layer, it would be nice to have a box in the journey result page to toggle photos-en-route.

I think this would up the rate of photo submissions, as it would expose the functionality more.

E.g. see thread at:
https://twitter.com/gazza_d/status/1345785157949804549

The popups, e.g. Photomap, cannot be closed by keyboard.

The X button should respond to the Escape key.

Photomap / since

Select any year in the Since field to get this screenshot from iphone 7 / Firefox at https://m.cyclestreets.net/photomap:since=NaN/#17.25/52.199272/0.147761

Desktop

Selecting the same link on a laptop doesn't immediately give the failure, but selecting a since date does reproduce the problem on subsequent occasions.

The Photomap browse performance is poor, because all the images in the icons are downloaded (at 1000px resolution) even if not clicked on.

This may actually be an issue in the layerviewer library. It may be necessary to add support for a callback.

The current one at /images/og-image-twitter.png needs to be more polished.

Although the site is not desktop-optimised, we should at least provide basic zoom functionality in the form of +/- buttons when not in touch mode.

(This was raised in feedback 14104.)

Samsung tablet:

"The linear speed selector doesn't seem to respond to touch..it does 'select' but not move.
I've have tried 'drag' and single select.
Not sure about the speed labels being off to one side so I tried selecting each one but no response"

https://twitter.com/Vafuncullo/status/1345329862224080896

Videos currently don't play.

E.g. /photomap/24948/ which is a video, MP4 as per others.

createBetaSwitch in the LayerViewer needs cookie memory, set to e.g. 7 days.

If using a desktop machine, the form doesn't submit with the return key.

Although the site is not intended for desktop, this should be supported, as disabling a standard form mechanism indicates poor accessibility.

The map style panel has a highlight round one style, but this does not always match the currently-shown style.

E.g. this screenshot shows Mapbox Streets background but the panel has OpenCycleMap highlighted.

Photomap items with longer captions, like

/photomap/8492/

need to be scrollable (or expandable in some way).

An imported photo shows the user rather than the credit (which is the field that contains the Twitter attribution).

https://twitter.com/LiorSteinberg/status/1343837052933177344

Currently if a user loads a previously-planned route which does not have one or more strategy variations planned, this produces a popup for each such variation:

Sorry, the route for Quietest route could not be loaded: None of the desired plans were found.

(The latter part of this sentence is obviously badly-worded, incidentally.)

NB This scenario will always happen for instance with the old mobile site, which planned three separate routes rather than three at once.

We should replace this popup with writing into the route panel result.