@hlfan commented on this pull request.


> @@ -52,6 +51,8 @@ def secondary_nav_items
       [about_path, t("layouts.header.about")]
     ]
 
+    items.insert(2, [traces_path, t("layouts.header.gps_traces")]) unless 
Settings.traces_disabled

I think we can go the with traces default path here too:
```suggestion
    items.delete_at(2) if Settings.traces_disabled
```

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7156#pullrequestreview-4582914865
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/7156/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to