Skip to content

Commit

Permalink
Update checkbox text (#287)
Browse files Browse the repository at this point in the history
  • Loading branch information
berroar authored Sep 17, 2020
1 parent 778f4d7 commit 8259a01
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion app/translations/messages.pot
Original file line number Diff line number Diff line change
Expand Up @@ -928,7 +928,7 @@ msgid "Manually enter address"
msgstr ""

#: templates/partials/answers/checkbox.html:8
msgid "Select all that apply"
msgid "Select at least one answer"
msgstr ""

#: templates/partials/answers/date.html:23
Expand Down
2 changes: 1 addition & 1 deletion templates/partials/answers/checkbox.html
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
{% elif answer.options|length == 1 %}
{% set checkbox_label = '' %}
{% else %}
{% set checkbox_label = _("Select all that apply") %}
{% set checkbox_label = _("Select at least one answer") %}
{% endif %}

{{ onsCheckboxes({
Expand Down
2 changes: 1 addition & 1 deletion tests/functional/spec/checkbox.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ describe('Checkbox with "other" option', () => {
});

it("Given a label has not been provided in the schema for a checkbox answer, When the checkbox answer is displayed, Then the default label should be visible", () => {
expect($("body").getText()).to.have.string("Select all that apply");
expect($("body").getText()).to.have.string("Select at least one answer");
});

it("Given a label has been set in the schema for a checkbox answer, When the checkbox answer is displayed, Then the label should be visible", () => {
Expand Down

0 comments on commit 8259a01

Please sign in to comment.