diff --git a/scss/modules.scss b/scss/modules.scss index 71e14dcc92..bf0a462e47 100644 --- a/scss/modules.scss +++ b/scss/modules.scss @@ -86,9 +86,23 @@ a.mini-user:hover { font-style: italic; color: $gray; } - .auth-button, .account-delete { - float: right; - margin-top: 8px; + + .account-action { + padding-right: 8px; + vertical-align: middle; + text-align: right; + + .account-delete { + display: inline-block; + } + } + + tr.has-avatar > .account-details { + vertical-align: bottom; + } + + .email { + position: relative; } .toggle-bitcoin, .toggle-email { position: absolute; @@ -96,13 +110,11 @@ a.mini-user:hover { } .toggle-bitcoin { margin-top: 8px; + right: 8px; } .account-type ~ .auth-button, .toggle-email { margin-top: -1.25em; } - td > div { - position: relative; - } table { width: 100%; } @@ -156,9 +168,6 @@ a.mini-user:hover { } } } - tr.has-avatar > td { - vertical-align: bottom; - } div.account-type { font-size: 12px; color: $gray; diff --git a/templates/account-row.html b/templates/account-row.html index 17e87c2d64..81fdf398d4 100644 --- a/templates/account-row.html +++ b/templates/account-row.html @@ -14,11 +14,6 @@ {% if account == None %} None - {% if not user.ANON and user.participant == participant %} - {% call auth_button(platform.name, 'connect') %} - + Add - {% endcall %} - {% endif %} {% else %} {{ account.user_name|e }} @@ -26,15 +21,23 @@ ({{ account.display_name|e }}) {% endif %} - {% if user.participant == participant %} + {% endif %} + +
{{ platform.display_name }}
+ + + {% if user.participant == participant %} + {% if account == None %} + {% call auth_button(platform.name, 'connect') %} + + Add + {% endcall %} + {% else %}
{% endif %} {% endif %} - -
{{ platform.display_name }}
{% endmacro %} diff --git a/templates/connected-accounts.html b/templates/connected-accounts.html index 82a4c4b24b..77a6469f35 100644 --- a/templates/connected-accounts.html +++ b/templates/connected-accounts.html @@ -29,7 +29,7 @@

Other Giving Options

- + {% if not user.ANON and own_account %}
{% else %} @@ -70,7 +70,7 @@

Other Giving Options

{% endif %} {% if accounts[website.platforms.venmo.name] or own_account %} - {{ account_row(website.platforms.venmo, accounts, auth_button) }} + {% include "templates/venmo-account.html" %} {% endif %} diff --git a/templates/venmo-account.html b/templates/venmo-account.html new file mode 100644 index 0000000000..db7748289e --- /dev/null +++ b/templates/venmo-account.html @@ -0,0 +1,49 @@ +{% from 'templates/avatar-url.html' import avatar_url with context %} + +{% set account = accounts.get('venmo', None) %} + + +
+ {% if account != None %} + + {% endif %} + +
+ + + {% if account == None %} + None + {% else %} + {{ account.user_name|e }} + {% if account.display_name and account.display_name != account.user_name %} + ({{ account.display_name|e }}) + {% endif %} + + {% endif %} + +
Venmo
+ + + {% if account == None %} + {% if not user.ANON and user.participant == participant %} + {% call auth_button('venmo', 'connect') %} + + Add + {% endcall %} + {% endif %} + {% else %} + {% if user.participant == participant %} +
+
+ {% else %} +
+ +
+ {% endif %} + {% endif %} + + + + diff --git a/www/%username/account/index.html.spt b/www/%username/account/index.html.spt index 2ae34e2401..c83768b6fa 100644 --- a/www/%username/account/index.html.spt +++ b/www/%username/account/index.html.spt @@ -72,9 +72,14 @@ locked = False
Credit card
{% if participant.last_bill_result != "" %} None - + Add {% else %} Added + {% endif %} + + + {% if participant.last_bill_result != "" %} + + Add + {% else %} Edit {% endif %} @@ -100,9 +105,14 @@ locked = False
Bank account
{% if participant.last_ach_result != "" %} None - + Add {% else %} Added + {% endif %} + + + {% if participant.last_ach_result != "" %} + + Add + {% else %} Edit {% endif %}