[openstreetmap/openstreetmap-website] Introduce "reload" js controller method (PR #3774)
Tom Hughes
notifications at github.com
Mon Oct 31 13:18:12 UTC 2022
@tomhughes commented on this pull request.
> @@ -179,8 +187,20 @@ OSM.Router = function (map, rts) {
};
router.load = function () {
- var loadState = currentRoute.run("load", currentPath);
+ var loadState = loadOrReload();
I'm asking why put the logic in a function rather than just having it inline.
Are you saying that function is only called once rather than each time the load function is invoked?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/3774#discussion_r1009408679
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/3774/review/1161863231 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20221031/ec2c549d/attachment.htm>
More information about the rails-dev
mailing list