@tomhughes commented on this pull request.


> @@ -139,6 +139,16 @@
       <div class='text-body-secondary'>
         <small>
           <dl class="list-inline">
+            <% if @user.home_location_name && 
@user.home_location_name.strip.present? %>

This could just be `@user.home_location_name&.strip.present?`?

> @@ -0,0 +1,65 @@
+OSM.HomeLocationNameEndpoint = function Endpoint(latInput, lonInput, 
locationNameInput) {

I don't really get what `Endpoint` means in this name? Would somethinglike 
`HomeLocationGeocoder` be a better name maybe?

> @@ -0,0 +1,65 @@
+OSM.HomeLocationNameEndpoint = function Endpoint(latInput, lonInput, 
locationNameInput) {
+  const endpoint = {
+    dirty: false,

What does `dirty` mean here? What is the flag mean to do? This implementing the 
autofill logic described in 
https://github.com/openstreetmap/openstreetmap-website/pull/5302#discussion_r1897207842
 I think? So maybe `autofillName` or something would be a better name?

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

Message ID: 
<openstreetmap/openstreetmap-website/pull/5302/review/2708616...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to