Merge branch 'develop' into update-oauth-template
authorMark Felder <feld@FreeBSD.org>
Wed, 1 May 2019 17:32:17 +0000 (12:32 -0500)
committerMark Felder <feld@FreeBSD.org>
Wed, 1 May 2019 17:32:17 +0000 (12:32 -0500)
1  2 
lib/pleroma/web/templates/layout/app.html.eex
lib/pleroma/web/templates/o_auth/o_auth/show.html.eex

index 7d2d609d13f6780bd51f28d2c5948188b3d71c0b,3389c91cce11160ea89a8eec65f0100e7954f472..a26b2c3c4b60c32a119858a354bfd4c511844186
            margin-top: 0
          }
  
 -        .scopes-input {
 -          flex-direction: column;
 +        .scope {
 +          flex-basis: 0%;
          }
  
 -        .scope {
 -          flex-basis: 50%;
 +        .scope:before {
 +          content: "";
 +          margin-left: 0em;
 +          margin-right: 1em;
 +        }
 +
 +        .scope:first-child:before {
 +          margin-left: 1em;
 +          content: "✔";
 +        }
 +
 +        .scope:after {
 +          content: ",";
 +        }
 +
 +        .scope:last-child:after {
 +          content: "";
          }
        }
+       .form-row {
+         display: flex;
+       }
+       .form-row > label {
+         text-align: left;
+         line-height: 47px;
+         flex: 1;
+       }
+       .form-row > input {
+         flex: 2;
+       }
      </style>
    </head>
    <body>
index 45f2b5cc07b87ddabfd0c54ed2b8f3732a87aec7,3e360a52cc3bf75b575550c097254496dc9a9258..8b69c30335cba0b8e88ea522587c8c8383e9937c
@@@ -6,50 -6,19 +6,52 @@@
  <% end %>
  
  <h2>OAuth Authorization</h2>
 -
  <%= form_for @conn, o_auth_path(@conn, :authorize), [as: "authorization"], fn f -> %>
 -<div class="input">
 -  <%= label f, :name, "Name or email" %>
 -  <%= text_input f, :name %>
 -</div>
 -<div class="input">
 -  <%= label f, :password, "Password" %>
 -  <%= password_input f, :password %>
 -</div>
 +
 +<%= if @params["registration"] in ["true", true] do %>
 +  <h3>This is the first time you visit! Please enter your Pleroma handle.</h3>
 +  <p>Choose carefully! You won't be able to change this later. You will be able to change your display name, though.</p>
 +  <div class="input">
 +    <%= label f, :nickname, "Pleroma Handle" %>
 +    <%= text_input f, :nickname, placeholder: "lain" %>
 +  </div>
 +  <%= hidden_input f, :name, value: @params["name"] %>
 +  <%= hidden_input f, :password, value: @params["password"] %>
 +  <br>
 +
 +<% else %>
 +  <div class="input">
 +    <%= label f, :name, "Username" %>
 +    <%= text_input f, :name %>
 +  </div>
 +  <div class="input">
 +    <%= label f, :password, "Password" %>
 +    <%= password_input f, :password %>
 +  </div>
 +  <%= submit "Log In" %>
 +  <div class="scopes-input">
 +    <%= label f, :scope, "The following permissions will be granted" %>
 +    <div class="scopes">
 +      <%= for scope <- @available_scopes do %>
 +        <%# Note: using hidden input with `unchecked_value` in order to distinguish user's empty selection from `scope` param being omitted %>
 +        <%= if scope in @scopes do %>
 +          <div class="scope">
 +            <%= checkbox f, :"scope_#{scope}", value: scope in @scopes && scope, checked_value: scope, unchecked_value: "", name: "authorization[scope][]" %>
 +            <%= label f, :"scope_#{scope}", String.capitalize(scope) %>
 +            <%= if scope in @scopes && scope do %>
 +              <%= String.capitalize(scope) %>
 +            <% end %>
 +          </div>
 +        <% else %>
 +          <%= checkbox f, :"scope_#{scope}", value: scope in @scopes && scope, checked_value: scope, unchecked_value: "", name: "authorization[scope][]" %>
 +        <% end %>
 +      <% end %>
 +    </div>
 +  </div>
 +<% end %>
  
+ <%= render @view_module, "_scopes.html", Map.merge(assigns, %{form: f}) %>
  <%= hidden_input f, :client_id, value: @client_id %>
  <%= hidden_input f, :response_type, value: @response_type %>
  <%= hidden_input f, :redirect_uri, value: @redirect_uri %>