darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-10 01:55:41 -04:00
Code Issues Packages Projects Releases Wiki Activity

Say ‘Change’ not ‘Edit’ postage

Browse Source 
To match the call to action on the template page.

Karl said:

> I feel good about it. You're not really editing it so much as choosing between 2 options. It feels more human.
>
> I think it only started out as edit to match all the other sections of the letter – it certainly wasn't given any real consideration.
This commit is contained in:
Chris Hill-Scott
2019-02-06 16:37:16 +00:00
parent 5e6d48e50f
commit e45024affd
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/templates/views/templates/edit-template-postage.html
View File

@@ -4,13 +4,13 @@
{% from "components/form.html" import form_wrapper %} {% from "components/form.html" import form_wrapper %}
{% block service_page_title %} {% block service_page_title %}
Edit postage Change postage
{% endblock %} {% endblock %}
{% block maincolumn_content %} {% block maincolumn_content %}
<h1 class="heading-large"> <h1 class="heading-large">
Edit postage Change postage
</h1> </h1>
{% call form_wrapper() %} {% call form_wrapper() %}
<div class="grid-row"> <div class="grid-row">

2
tests/app/main/views/test_templates.py
View File

@@ -493,7 +493,7 @@ def test_edit_letter_template_postage_page_displays_correctly(
template_id=fake_uuid, template_id=fake_uuid,
) )
assert page.select_one('h1').text.strip() == 'Edit postage' assert page.select_one('h1').text.strip() == 'Change postage'
assert page.select('input[checked]')[0].attrs["value"] == 'second' assert page.select('input[checked]')[0].attrs["value"] == 'second'