Merge pull request #2567 from alphagov/dont-stick-on-sms

Remove sticky textbox for one-off text messages
This commit is contained in:
Chris Hill-Scott
2018-12-07 10:55:35 +00:00
committed by GitHub
2 changed files with 31 additions and 1 deletions

View File

@@ -14,7 +14,10 @@
{{ page_title }}
</h1>
{% call form_wrapper(class="js-stick-at-top-when-scrolling", module="autofocus") %}
{% call form_wrapper(
class="js-stick-at-top-when-scrolling" if template.template_type != 'sms' else '',
module="autofocus"
) %}
<div class="grid-row">
<div class="column-two-thirds {% if form.placeholder_value.label.text == 'phone number' %}extra-tracking{% endif %}">
{{ textbox(

View File

@@ -1172,6 +1172,33 @@ def test_send_one_off_has_skip_link(
assert not skip_links
@pytest.mark.parametrize('template_mock, expected_sticky', [
(mock_get_service_template, False),
(mock_get_service_email_template, True),
(mock_get_service_letter_template, True),
])
def test_send_one_off_has_sticky_header_for_email_and_letter(
mocker,
client_request,
fake_uuid,
mock_has_no_jobs,
template_mock,
expected_sticky,
):
template_mock(mocker)
mocker.patch('app.main.views.send.get_page_count_for_letter', return_value=99)
page = client_request.get(
'main.send_one_off_step',
service_id=SERVICE_ONE_ID,
template_id=fake_uuid,
step_index=0,
_follow_redirects=True,
)
assert bool(page.select('.js-stick-at-top-when-scrolling')) == expected_sticky
@pytest.mark.parametrize('user', (
active_user_with_permissions,
active_caseworking_user,