<p></p>
<p><b>@gravitystorm</b> requested changes on this pull request.</p>

<p dir="auto">I'm generally happy with the PR, and neutral on most of the changes, but I've flagged a few that need further attention.</p><hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3536#discussion_r859796289">config/locales/en.yml</a>:</p>
<pre style='color:#555'>> @@ -661,12 +661,12 @@ en:
           driving_school: "Driving School"
           embassy: "Embassy"
           events_venue: "Events Venue"
-          fast_food: "Fast Food"
+          fast_food: "Fast Food Restaurant"
</pre>
<p dir="auto">This should be removed, since amenity=fast_food also covers food trucks and takeaway only outlets.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3536#discussion_r859799594">config/locales/en.yml</a>:</p>
<pre style='color:#555'>> @@ -841,7 +841,7 @@ en:
           bridleway: "Bridleway"
           bus_guideway: "Guided Bus Lane"
           bus_stop: "Bus Stop"
-          construction: "Highway under Construction"
+          construction: "Road under Construction"
</pre>
<p dir="auto">highway=construction also applies to paths, steps and other non-road things. I know that "highway" has a general meaning too, outside of OSM, that doesn't exactly match what we mean within OSM, but the simplest way forward is to keep the "highway" terminology here since it's accurate within the OSM context.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3536#discussion_r859800194">config/locales/en.yml</a>:</p>
<pre style='color:#555'>> @@ -931,9 +931,9 @@ en:
           brownfield: "Brownfield Land"
           cemetery: "Cemetery"
           commercial: "Commercial Area"
-          conservation: "Conservation"
-          construction: "Construction"
-          farm: "Farm"
+          conservation: "Conservation Area"
+          construction: "Construction Area"
+          farm: "Farmland"
</pre>
<p dir="auto">This key is deprecated and should be removed rather than changed. There's only a few instances of it remaining in the database in 2022 anyway.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3536#discussion_r859803190">config/locales/en.yml</a>:</p>
<pre style='color:#555'>> @@ -1228,8 +1228,8 @@ en:
           deli: "Deli"
           department_store: "Department Store"
           discount: "Discount Items Shop"
-          doityourself: "Do-It-Yourself"
-          dry_cleaning: "Dry Cleaning"
+          doityourself: "Do-It-Yourself Shop"
+          dry_cleaning: "Dry Cleaner"
</pre>
<p dir="auto">The "Dry cleaning" term is the one exclusively used in the OSM wiki, so I think it's fine to keep the original</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/3536#pullrequestreview-954898791">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLKCJF3IJNEMPELUOWLVHE6Q3ANCNFSM5UHEEIIA">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLIVUARMX4S36DT35IDVHE6Q3A5CNFSM5UHEEIIKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOHDVJSZY.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/3536/review/954898791</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/3536#pullrequestreview-954898791",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3536#pullrequestreview-954898791",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>