@financial-times/n-conversion-forms 23.0.0 → 23.0.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (7) hide show
  1. package/build-state/npm-shrinkwrap.json +2 -2
  2. package/components/accept-terms.jsx +42 -25
  3. package/components/message.jsx +1 -1
  4. package/dist/accept-terms.js +12 -7
  5. package/dist/message.js +1 -1
  6. package/main.scss +9 -1
  7. package/package.json +2 -2
package/build-state/npm-shrinkwrap.json CHANGED
@@ -14,7 +14,7 @@
14
14
  "classnames": "2.3.1",
15
15
  "fetchres": "1.7.2",
16
16
  "lodash.get": "4.4.2",
17
- "n-common-static-data": "financial-times/n-common-static-data#v1.6.1"
17
+ "n-common-static-data": "financial-times/n-common-static-data#v1.7.1"
18
18
  },
19
19
  "devDependencies": {
20
20
  "@babel/cli": "^7.10.5",
@@ -56319,7 +56319,7 @@
56319
56319
  },
56320
56320
  "n-common-static-data": {
56321
56321
  "version": "git+ssh://git@github.com/financial-times/n-common-static-data.git#f20f8476d37b1694134c4bc5ba72bfeb75ea2565",
56322
- "from": "n-common-static-data@financial-times/n-common-static-data#v1.6.1"
56322
+ "from": "n-common-static-data@financial-times/n-common-static-data#v1.7.1"
56323
56323
  },
56324
56324
  "nan": {
56325
56325
  "version": "2.15.0",
package/components/accept-terms.jsx CHANGED
@@ -22,6 +22,7 @@ export function AcceptTerms ({
22
22
  isPrintProduct = false,
23
23
  specialTerms = null,
24
24
  isSingleTerm = false,
25
+ isNewDigitalBuyFlowConsent = false,
25
26
  }) {
26
27
  const divProps = {
27
28
  id: 'acceptTermsField',
@@ -50,34 +51,49 @@ export function AcceptTerms ({
50
51
 
51
52
  const authFirstStepTerms = (
52
53
  <>
53
- <ul className="o-typography-list ncf__accept-terms-list">
54
- <li>
55
- <span className="terms-auth-first-step">
56
- For more information about how we use your data, please refer to our <a
57
- className="ncf__link--external"
58
- href="http://help.ft.com/help/legal-privacy/terms-conditions/"
59
- target='_blank'
60
- rel="noopener noreferrer"
61
- data-trackable="terms-and-conditions"
62
- >
63
- privacy
64
- </a> and&nbsp;
65
- <a
66
- className="ncf__link--external"
67
- href="http://help.ft.com/help/legal-privacy/terms-conditions/"
68
- target='_blank'
69
- rel="noopener noreferrer"
70
- data-trackable="terms-and-conditions"
71
- >
72
- cookie
73
- </a> policies.
74
- </span>
75
- </li>
76
- </ul>
54
+ {
55
+ // TODO: clean up the code in https://financialtimes.atlassian.net/browse/ACQ-1593
56
+ // isNewDigitalBuyFlowConsent flag for new A/B testing of consents auth first buy flow
57
+ // after the test it will be one of the conditions for the below flow
58
+ }
59
+ {isNewDigitalBuyFlowConsent && (
60
+ <hr className=" ncf ncf__divider-horizontal" />
61
+ )}
62
+ {!isNewDigitalBuyFlowConsent && (
63
+ <ul className="o-typography-list ncf__accept-terms-list">
64
+ <li>
65
+ <span className="terms-auth-first-step">
66
+ For more information about how we use your data, please refer to
67
+ our{' '}
68
+ <a
69
+ className="ncf__link--external"
70
+ href="http://help.ft.com/help/legal-privacy/terms-conditions/"
71
+ target="_blank"
72
+ rel="noopener noreferrer"
73
+ data-trackable="terms-and-conditions"
74
+ >
75
+ privacy
76
+ </a>{' '}
77
+ and&nbsp;
78
+ <a
79
+ className="ncf__link--external"
80
+ href="http://help.ft.com/help/legal-privacy/terms-conditions/"
81
+ target="_blank"
82
+ rel="noopener noreferrer"
83
+ data-trackable="terms-and-conditions"
84
+ >
85
+ cookie
86
+ </a>{' '}
87
+ policies.
88
+ </span>
89
+ </li>
90
+ </ul>
91
+ )}
77
92
  <label className={labelClassName} htmlFor="termsAcceptance">
78
93
  <input {...inputProps} />
79
94
  <span className="o-forms-input__label terms-auth-first-step">
80
- I confirm that I am {ageRestriction} years or older and agree to the full {' '}
95
+ I confirm that I am {ageRestriction} years or older and agree to the
96
+ full{' '}
81
97
  <a
82
98
  className="ncf__link--external"
83
99
  href="http://help.ft.com/help/legal-privacy/terms-conditions/"
@@ -391,4 +407,5 @@ AcceptTerms.propTypes = {
391
407
  isPrintProduct: PropTypes.bool,
392
408
  specialTerms: PropTypes.string,
393
409
  isSingleTerm: PropTypes.bool,
410
+ isNewDigitalBuyFlowConsent: PropTypes.bool,
394
411
  };
package/components/message.jsx CHANGED
@@ -87,7 +87,7 @@ const actionType = PropTypes.shape({
87
87
  link: PropTypes.string.isRequired,
88
88
  isSecondary: PropTypes.bool,
89
89
  text: PropTypes.string,
90
- target: '_self',
90
+ target: PropTypes.oneOf(['_self']),
91
91
  });
92
92
 
93
93
  Message.propTypes = {
package/dist/accept-terms.js CHANGED
@@ -55,7 +55,9 @@ function AcceptTerms(_ref) {
55
55
  _ref$specialTerms = _ref.specialTerms,
56
56
  specialTerms = _ref$specialTerms === void 0 ? null : _ref$specialTerms,
57
57
  _ref$isSingleTerm = _ref.isSingleTerm,
58
- isSingleTerm = _ref$isSingleTerm === void 0 ? false : _ref$isSingleTerm;
58
+ isSingleTerm = _ref$isSingleTerm === void 0 ? false : _ref$isSingleTerm,
59
+ _ref$isNewDigitalBuyF = _ref.isNewDigitalBuyFlowConsent,
60
+ isNewDigitalBuyFlowConsent = _ref$isNewDigitalBuyF === void 0 ? false : _ref$isNewDigitalBuyF;
59
61
 
60
62
  var divProps = _objectSpread(_objectSpread({
61
63
  id: 'acceptTermsField',
@@ -83,28 +85,30 @@ function AcceptTerms(_ref) {
83
85
  defaultChecked: true
84
86
  });
85
87
 
86
- var authFirstStepTerms = /*#__PURE__*/_react["default"].createElement(_react["default"].Fragment, null, /*#__PURE__*/_react["default"].createElement("ul", {
88
+ var authFirstStepTerms = /*#__PURE__*/_react["default"].createElement(_react["default"].Fragment, null, isNewDigitalBuyFlowConsent && /*#__PURE__*/_react["default"].createElement("hr", {
89
+ className: " ncf ncf__divider-horizontal"
90
+ }), !isNewDigitalBuyFlowConsent && /*#__PURE__*/_react["default"].createElement("ul", {
87
91
  className: "o-typography-list ncf__accept-terms-list"
88
92
  }, /*#__PURE__*/_react["default"].createElement("li", null, /*#__PURE__*/_react["default"].createElement("span", {
89
93
  className: "terms-auth-first-step"
90
- }, "For more information about how we use your data, please refer to our ", /*#__PURE__*/_react["default"].createElement("a", {
94
+ }, "For more information about how we use your data, please refer to our", ' ', /*#__PURE__*/_react["default"].createElement("a", {
91
95
  className: "ncf__link--external",
92
96
  href: "http://help.ft.com/help/legal-privacy/terms-conditions/",
93
97
  target: "_blank",
94
98
  rel: "noopener noreferrer",
95
99
  "data-trackable": "terms-and-conditions"
96
- }, "privacy"), " and\xA0", /*#__PURE__*/_react["default"].createElement("a", {
100
+ }, "privacy"), ' ', "and\xA0", /*#__PURE__*/_react["default"].createElement("a", {
97
101
  className: "ncf__link--external",
98
102
  href: "http://help.ft.com/help/legal-privacy/terms-conditions/",
99
103
  target: "_blank",
100
104
  rel: "noopener noreferrer",
101
105
  "data-trackable": "terms-and-conditions"
102
- }, "cookie"), " policies."))), /*#__PURE__*/_react["default"].createElement("label", {
106
+ }, "cookie"), ' ', "policies."))), /*#__PURE__*/_react["default"].createElement("label", {
103
107
  className: labelClassName,
104
108
  htmlFor: "termsAcceptance"
105
109
  }, /*#__PURE__*/_react["default"].createElement("input", inputProps), /*#__PURE__*/_react["default"].createElement("span", {
106
110
  className: "o-forms-input__label terms-auth-first-step"
107
- }, "I confirm that I am ", ageRestriction, " years or older and agree to the full ", ' ', /*#__PURE__*/_react["default"].createElement("a", {
111
+ }, "I confirm that I am ", ageRestriction, " years or older and agree to the full", ' ', /*#__PURE__*/_react["default"].createElement("a", {
108
112
  className: "ncf__link--external",
109
113
  href: "http://help.ft.com/help/legal-privacy/terms-conditions/",
110
114
  target: isEmbedded ? '_top' : '_blank',
@@ -243,5 +247,6 @@ AcceptTerms.propTypes = {
243
247
  transitionType: _propTypes["default"].string,
244
248
  isPrintProduct: _propTypes["default"].bool,
245
249
  specialTerms: _propTypes["default"].string,
246
- isSingleTerm: _propTypes["default"].bool
250
+ isSingleTerm: _propTypes["default"].bool,
251
+ isNewDigitalBuyFlowConsent: _propTypes["default"].bool
247
252
  };
package/dist/message.js CHANGED
@@ -89,7 +89,7 @@ var actionType = _propTypes["default"].shape({
89
89
  link: _propTypes["default"].string.isRequired,
90
90
  isSecondary: _propTypes["default"].bool,
91
91
  text: _propTypes["default"].string,
92
- target: '_self'
92
+ target: _propTypes["default"].oneOf(['_self'])
93
93
  });
94
94
 
95
95
  Message.propTypes = {
package/main.scss CHANGED
@@ -53,6 +53,14 @@
53
53
  .ncf {
54
54
  @include oTypographySans($scale: 0);
55
55
 
56
+ &__divider-horizontal {
57
+ margin: 0 0 16px;
58
+ height: 0;
59
+ left: 0;
60
+ top: 0;
61
+ border: 1px solid rgba(0, 0, 0, 0.15);
62
+ }
63
+
56
64
  &__wrapper {
57
65
  background: oColorsByName('white');
58
66
  padding: 20px;
@@ -101,7 +109,7 @@
101
109
 
102
110
  &__fieldset {
103
111
  border: 0;
104
- margin: 0 0 20px;
112
+ margin: 0 0 10px;
105
113
  min-inline-size: auto;
106
114
  padding: 0;
107
115
 
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@financial-times/n-conversion-forms",
3
- "version": "23.0.0",
3
+ "version": "23.0.3",
4
4
  "description": "Containing jsx components and styles for forms included on Accounts and Acqusition apps (next-signup, next-profile, next-retention, etc).",
5
5
  "main": "dist/index.js",
6
6
  "scripts": {
@@ -29,7 +29,7 @@
29
29
  "classnames": "2.3.1",
30
30
  "fetchres": "1.7.2",
31
31
  "lodash.get": "4.4.2",
32
- "n-common-static-data": "financial-times/n-common-static-data#v1.6.1"
32
+ "n-common-static-data": "financial-times/n-common-static-data#v1.7.1"
33
33
  },
34
34
  "devDependencies": {
35
35
  "@babel/cli": "^7.10.5",