Skip to content

Commit 4e7f1df

Browse files
committed
Change content and back link for new group page
Change the content, title, and back link for the new group page to match the new designs.
1 parent 0bb97cb commit 4e7f1df

3 files changed

Lines changed: 14 additions & 12 deletions

File tree

app/views/groups/new.html.erb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
<% set_page_title(title_with_error_prefix(t(".title"), @group.errors.any?)) %>
2-
<% content_for :back_link, govuk_back_link_to(groups_path,t("back_link.groups")) %>
1+
<% set_page_title(title_with_error_prefix(t("page_titles.new_group"), @group.errors.any?)) %>
2+
<% content_for :back_link, govuk_back_link_to(confirm_new_groups_path,t(".back_link")) %>
33

44
<%= form_with(model: @group) do |f| %>
55
<div class="govuk-grid-row">
@@ -8,7 +8,7 @@
88
<%= f.govuk_error_summary %>
99
<% end %>
1010

11-
<h1 class="govuk-heading-l"><%= t(".title") %></h1>
11+
<h1 class="govuk-heading-l"><%= t("page_titles.new_group") %></h1>
1212

1313
<%= t(".body_html") %>
1414

config/locales/en.yml

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -742,21 +742,18 @@ en:
742742
label: Move this group to another organisation
743743
prompt: Select an organisation
744744
new:
745+
back_link: Back to create a new group
745746
body_html: |
746-
<p>Each form needs to be created in a ‘group’. Once you have a group, you can add other people to create and edit forms in it.</p>
747-
748-
<p class="govuk-inset-text">When a group is first created it will be a ‘trial’ group. That means you cannot make any forms in the group live. You’ll be able to request for a group to be upgraded to an ‘active’ group if you need to make forms live.</p>
749-
750-
<p>Give your group a meaningful name that will reflect the forms you’ll create in the group, and make sense to the people that will be added to it. For example:</p>
751-
747+
<p>Give your group a meaningful name that will reflect the forms you’ll
748+
create in the group, and make sense to the people that will be added to
749+
it. For example:</p>
752750
<ul class="govuk-list govuk-list--bullet">
753751
<li>Licensing forms</li>
754752
<li>GOV.UK content team</li>
755753
<li>Juggling service forms</li>
754+
<li>Test forms</li>
756755
</ul>
757-
758756
<p>You can change the name later if you need to.</p>
759-
title: Create a new group for forms
760757
review_upgrade:
761758
body_html: |
762759
<p>%{upgrade_requester_name} has asked to upgrade this group to an ‘active’ group.</p>
@@ -1496,6 +1493,7 @@ en:
14961493
mou_signatures: MOUs and agreements
14971494
move_form: Move this form to a different group
14981495
name_settings: Ask for a person’s name
1496+
new_group: Give your group a name
14991497
new_page: Edit question
15001498
new_secondary_skip: 'Route for any other answer: set questions to skip'
15011499
non_crown_agreement_confirmation: You’ve agreed to the GOV.UK Forms agreement

spec/views/groups/new.html.erb_spec.rb

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
end
1010

1111
it "contains the page heading" do
12-
expect(rendered).to have_css("h1", text: I18n.t("groups.new.title"))
12+
expect(rendered).to have_css("h1", text: I18n.t("page_titles.new_group"))
1313
end
1414

1515
it "renders the new group form" do
@@ -25,4 +25,8 @@
2525
it "includes the body text" do
2626
expect(rendered).to include(I18n.t("groups.new.body_html"))
2727
end
28+
29+
it "has a back link" do
30+
expect(view.content_for(:back_link)).to have_link(I18n.t("groups.new.back_link"), href: confirm_new_groups_path)
31+
end
2832
end

0 commit comments

Comments
 (0)