Add requested changes

README.md

@@ -38,14 +38,13 @@ The latest javascript file can be found in github releases https://github.com/nh
 ...
 
 <!-- where you want the form to appear -->
-<div id="nhsuk-user-feedback-form" data-form-endpoint="https://example.com/endpoint/" data-enable-text-response="false"></div>
+<div id="nhsuk-user-feedback-form" data-form-endpoint="https://example.com/endpoint/"></div>
 ```
 
 #### Attributes
 
 `data-form-endpoint` - (required) An HTTP data store endpoint to POST data to. Include the trailing slash
-`data-enable-text-response` - (optional) If "true", text responses will be enabled. If "false" or left out,
-text responses will not be disabled.
+`data-enable-text-response` - (optional) Include to enable text responses on the feedback form.
 
 ## API
 

src/confirmation/index.js

@@ -7,19 +7,16 @@ export default class ConfirmationScreen extends Screen {
   }
 
   render() {
-    const textResponseLabel = '<p>We do not check feedback every day and cannot respond to comments.</p>';
-    const blankLabel = '';
-
-    let label;
+    let message;
 
     /* If data-enable-text-response attribute is "false" show a blank label */
     if (this.getEnableTextResponse() === false) {
-      label = blankLabel;
+      message = '';
     } else {
-      label = textResponseLabel;
+      message = '<p>We do not check feedback every day and cannot respond to comments.</p>';
     }
 
-    const optionalHtml = html.replace('{{ label }}', label);
+    const optionalHtml = html.replace('{{ optionalTextResponseMessage }}', message);
 
     const node = this.updateHtml(optionalHtml);
     const header = node.querySelector('.nhsuk-user-feedback-form-header');

src/confirmation/template.html

@@ -1,5 +1,5 @@
 <div class="nhsuk-panel nhsuk-u-reading-width">
   <h2 class="nhsuk-heading-s nhsuk-user-feedback-form-header">Thank you for your feedback.</h2>
-   {{ label }}
+   {{ optionalTextResponseMessage }}
   <p>Find out how to <a href="/contact-us/">contact the NHS</a> if you need to speak to someone.</p>
 </div>

src/index.js

@@ -9,10 +9,7 @@ import PostData from './post-data';
 import './nhsuk-feedback-form.css';
 
 function getEnableTextResponse(container) {
-  if (container.getAttribute('data-enable-text-response') === 'true') {
-    return true;
-  }
-  return false;
+  return container.hasAttribute('data-enable-text-response');
 }
 
 /**

tests/example/disabled-text-response.html

@@ -17,7 +17,7 @@
     <div class="nhsuk-width-container">
       <main class="nhsuk-main-wrapper">
         <h1>User feedback form:</h1>
-        <div id="nhsuk-user-feedback-form" data-form-endpoint="http://localhost:8080/my-endpoint/" data-enable-text-response="false"></div>
+        <div id="nhsuk-user-feedback-form" data-form-endpoint="http://localhost:8080/my-endpoint/"></div>
 
         <!--
         If you have the user-feedback-store running locally and want to test it, the endpoint setting should probably be this:

tests/example/index.html

@@ -17,7 +17,7 @@
     <div class="nhsuk-width-container">
       <main class="nhsuk-main-wrapper">
         <h1>User feedback form:</h1>
-        <div id="nhsuk-user-feedback-form" data-form-endpoint="http://localhost:8080/my-endpoint/" data-enable-text-response="true"></div>
+        <div id="nhsuk-user-feedback-form" data-form-endpoint="http://localhost:8080/my-endpoint/"></div>
 
         <!--
         If you have the user-feedback-store running locally and want to test it, the endpoint setting should probably be this: