diff --git a/app/assets/stylesheets/components/tick-cross.scss b/app/assets/stylesheets/components/tick-cross.scss index 9d9ce2489..ef04b0a93 100644 --- a/app/assets/stylesheets/components/tick-cross.scss +++ b/app/assets/stylesheets/components/tick-cross.scss @@ -36,44 +36,10 @@ } - &-edit-link { - - text-align: right; - position: absolute; - top: -25px; - right: -135px; - - a { - - &:before { - content: ""; - display: block; - - position: absolute; - - top: -1 * govuk-spacing(3) - 1; - right: 0; - bottom: -1 * govuk-spacing(3); - left: -1 * govuk-spacing(3); - - background: transparent; - } - - &:active:before, - &:focus:before { - border-color: $yellow; - border-style: solid; - border-width: 15px 3px 15px 15px; - right: -3px; - } - - } - - } - &-hint { color: #6F777B; padding-top: 5px; + margin: 0; } } diff --git a/app/assets/stylesheets/views/users.scss b/app/assets/stylesheets/views/users.scss index 4abef1993..a7b126c4a 100644 --- a/app/assets/stylesheets/views/users.scss +++ b/app/assets/stylesheets/views/users.scss @@ -7,9 +7,8 @@ $item-top-padding: govuk-spacing(3); &-item { - padding: $item-top-padding 150px govuk-spacing(3) 0; + padding: govuk-spacing(3) 0px; border-top: 1px solid $border-colour; - position: relative; &-heading { @@ -29,13 +28,61 @@ $item-top-padding: govuk-spacing(3); border-bottom: 1px solid $border-colour; } + & :last-child { + margin-bottom: 0; + } + + &-blank { + + margin-bottom: govuk-spacing(3); + + @include govuk-media-query($from: tablet) { + margin-bottom: 0; + } + + } + } &-edit-link { - text-align: right; - position: absolute; - top: $item-top-padding; - right: 0px; + + position: relative; + display: block; + margin: govuk-spacing(3) 0 govuk-spacing(3); + + @include govuk-media-query($from: tablet) { + float: right; + margin: 0; + } + + &:before { + content: ""; + display: block; + + position: absolute; + + top: -1 * govuk-spacing(3) - 1; + right: 0; + bottom: -1 * govuk-spacing(3); + left: -1 * govuk-spacing(3); + + background: transparent; + } + + &:active:before, + &:focus:before { + border-color: $yellow; + border-style: solid; + border-width: 15px 3px; + right: -3px; + left: -3px; + + @include govuk-media-query($from: tablet) { + border-width: 15px 3px 15px 15px; + left: -15px; + } + } + } } diff --git a/app/templates/views/manage-users.html b/app/templates/views/manage-users.html index 663d8ccb5..f8c9d858b 100644 --- a/app/templates/views/manage-users.html +++ b/app/templates/views/manage-users.html @@ -22,33 +22,35 @@
{% for user in users %}
-

- {%- if user.name -%} - {{ user.name }}  - {%- endif -%} - - {%- if user.status == 'pending' -%} - {{ user.email_address }} (invited) - {%- elif user.status == 'cancelled' -%} - {{ user.email_address }} (cancelled invite) - {%- elif user.id == current_user.id -%} - (you) - {% else %} - {{ user.email_address }} - {% endif %} - -

- +
+
{% endfor %} diff --git a/tests/app/main/views/organisations/test_organisation_invites.py b/tests/app/main/views/organisations/test_organisation_invites.py index d32748310..9a80301aa 100644 --- a/tests/app/main/views/organisations/test_organisation_invites.py +++ b/tests/app/main/views/organisations/test_organisation_invites.py @@ -28,7 +28,7 @@ def test_view_team_members( ) == 'Test User {}'.format(i + 1) assert normalize_spaces( - page.select('.tick-cross-list-edit-link')[0].text + page.select('.user-list-edit-link')[0].text ) == 'Cancel invitation'