New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add mapillary to node detail view #559
Conversation
By default the Mapillary content is invisible and only gets shown if it can provide content. Note the call to resize after applying the .visible class to the parent container: This is necessary to ensure that it gets rendered properly. Without it would be in an invalid state
@@ -6,7 +6,7 @@ | |||
config.cache_classes = true | |||
|
|||
# Full error reports are disabled and caching is turned on | |||
config.consider_all_requests_local = false | |||
config.consider_all_requests_local = true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this done?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah good catch! I only enabled that to debug some things. This should be set to false
obviously.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine, please see comment though regarding a question.
This was set to true for debugging purposes and was only accidentally committed.
Resolves #480
This Pull Request adds Mapillary Street View for the Node Detail View additionally to Google Street View.
It is shown only if there's content available for the given latitude and longitude, otherwise it stays hidden.