[openstreetmap/openstreetmap-website] Re-arrange login and signup screens as discussed in #4128 (PR #4455)

Milan Cvetkovic notifications at github.com
Mon Mar 4 18:02:17 UTC 2024


@milan-cvetkovic commented on this pull request.



> +.nav-z-100 {
+  z-index: 100;
+}

Actually, it looks like z-index does not make any difference here. Thanks!

>  
-      <%= f.password_field :pass_crypt, :tabindex => 6 %>
-      <%= f.password_field :pass_crypt_confirmation, :tabindex => 7 %>
+      <p class="mb-3 text-muted fs-6"><%= t(".by_signing_up_html",
+                                            :tou_link => link_to(t(".tou"),
+                                                                 "https://wiki.osmfoundation.org/wiki/Terms_of_Use",
+                                                                 :target => :new),
+                                            :privacy_policy_link => link_to(t(".privacy_policy"),
+                                                                            t(".privacy_policy_url"),
+                                                                            :title => t(".privacy_policy_title"),
+                                                                            :target => :new),
+                                            :contributor_terms_link => link_to(t(".contributor_terms"),
+                                                                               t(".contributor_terms_url"),
+                                                                               :target => :new)) %></p>
+      <%= f.form_group do %>
+        <%= f.check_box :consider_pd,

Actually, I believe consider_pd is not necessary to go forward. There were comments earlier about this checkbox, there was a suggestion to remove it entirely: https://github.com/openstreetmap/openstreetmap-website/pull/4455#issuecomment-1899125291

You might be referring to TOU and privacy policy, both of them accepted implicitly by signing up, as agreed by LWG in https://github.com/openstreetmap/openstreetmap-website/pull/4455#issuecomment-1881708829.

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

Message ID: <openstreetmap/openstreetmap-website/pull/4455/review/1914646735 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240304/1a9222aa/attachment-0001.htm>


More information about the rails-dev mailing list