<p></p>
<p dir="auto"><a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/users/Vel-12/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/Vel-12">@Vel-12</a> this PR contains two commits, the second commit is fixing a mistake that you made in the first commit.</p>
<p dir="auto">From <a href="https://github.com/openstreetmap/openstreetmap-website/blob/master/CONTRIBUTING.md">CONTRIBUTING.md</a></p>
<blockquote>
<p dir="auto">The commit history is important. This is a large codebase, developed over many years by many developers. We frequently need to read the commit history (e.g. using git blame) to figure out what is going on. So small, understandable, and relevant commits are important for other developers looking back at your work in future.</p>
</blockquote>
<blockquote>
<p dir="auto">Avoid including "fixup" commits. If you have added a fixup commit (for example to fix a rubocop warning, or because you changed your own new code) please combine the fixup commit into the commit that introduced the problem. git rebase -i is very useful for this.</p>
</blockquote>
<p dir="auto">I would also like to see more of an explanation in your pull request, to show that by changing the direction of form controls that you aren't introducing any side effects. See also the continuing discussion at <a aria-label="Issue #3910" class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="1557854323" data-permission-text="Title is private" data-url="https://github.com/openstreetmap/openstreetmap-website/issues/3910" data-hovercard-type="issue" data-hovercard-url="/openstreetmap/openstreetmap-website/issues/3910/hovercard" href="https://github.com/openstreetmap/openstreetmap-website/issues/3910">#3910</a>.</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/3922#issuecomment-1424014123">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLJXNC7HOOLOKKCQY53WWTF4XANCNFSM6AAAAAAUWIDMHI">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLO2RUYBHP2WJN4ZG6LWWTF4XA5CNFSM6AAAAAAUWIDMHKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSU4C5SW.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/3922/c1424014123</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/3922#issuecomment-1424014123",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/3922#issuecomment-1424014123",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>