@financial-times/n-myft-ui 23.1.1 → 24.0.1

package/components/collections/collections.html

@@ -10,17 +10,9 @@
 		{{#concepts}}
 			<li class="collection__concept">
 				{{#if ../liteStyle}}
-					{{>	n-myft-ui/components/follow-button/follow-button
-						variant="primary"
-						buttonText=name
-						collectionName=../collectionName
-					}}
+					{{{renderReactComponent localPath="components/follow-button/follow-button" variant="primary" buttonText=name flags=@root.flags collectionName=../collectionName}}}
 				{{else}}
-					{{>	n-myft-ui/components/follow-button/follow-button
-						variant="inverse"
-						buttonText=name
-						collectionName=../collectionName
-					}}
+					{{{renderReactComponent localPath="components/follow-button/follow-button" variant="inverse" buttonText=name flags=@root.flags collectionName=../collectionName}}}
 				{{/if}}
 			</li>
 		{{/concepts}}
@@ -50,7 +42,7 @@
 						{{~/unless~}}
 					{{~/concepts~}}"
 			/>
-			{{> n-myft-ui/components/csrf-token/input}}
+			{{{renderReactComponent localPath="components/csrf-token/input"}}}
 			<input
 				type="hidden"
 				name="name"

package/components/concept-list/concept-list.html

@@ -20,10 +20,7 @@
 						class="concept-list__concept">
 						{{prefLabel}}
 					</a>
-					{{> n-myft-ui/components/follow-button/follow-button
-						conceptId=id
-						name=prefLabel
-					}}
+					{{{renderReactComponent localPath="components/follow-button/follow-button" conceptId=id name=prefLabel flags=@root.flags}}}
 				</li>
 			{{/each}}
 		</ul>

package/components/csrf-token/__tests__/input.test.js

@@ -0,0 +1,23 @@
+import React from 'react';
+import CsrfToken from '../input';
+import { render } from '@testing-library/react';
+import '@testing-library/jest-dom';
+
+const props = {
+	cacheablePersonalisedUrl: false
+};
+
+describe('Csrf Token Input', () => {
+
+	test('It renders default button', async () => {
+		let { container } = render(<CsrfToken {...props} />);
+		expect(container.querySelector('[name=\'token\']')).toBeTruthy();
+	});
+
+	test('It renders csrf token attribute', async () => {
+		let { container } = render(<CsrfToken cacheablePersonalisedUrl={true} csrfToken={'test-token'} />);
+		expect(container.querySelector('[data-myft-csrf-token=\'test-token\']')).toBeTruthy();
+	});
+
+
+});

package/components/csrf-token/input.jsx

@@ -0,0 +1,26 @@
+import React from 'react';
+
+export default function CsrfToken ({ cacheablePersonalisedUrl, csrfToken }) {
+
+	let inputProps = {};
+
+	if (cacheablePersonalisedUrl) {
+		inputProps = {
+			...inputProps,
+			'data-myft-csrf-token': csrfToken
+		};
+	}
+
+	if(csrfToken) {
+		inputProps.value = csrfToken;
+	}
+
+	return (
+		<input
+			{...inputProps}
+			type="hidden"
+			name="token"
+		/>
+	);
+
+}

package/components/follow-button/__tests__/follow-button.test.js

@@ -0,0 +1,40 @@
+import React from 'react';
+import FollowButton from '../follow-button';
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom';
+
+const props = {
+	flags: {
+		myFtApi: true,
+		myFtApiWrite: true
+	},
+	conceptId: '0000-000000-00000-0000',
+	name: 'Follow button'
+};
+
+describe('Follow button', () => {
+
+	test('It renders default button', async () => {
+		render(<FollowButton {...props} />);
+		expect(screen.findByText('Add to myFT')).toBeTruthy();
+	});
+
+	test('It renders a variant', async () => {
+		const { container } = render(<FollowButton {...props} variant={'standard'} />);
+		expect(container.getElementsByClassName('n-myft-follow-button--standard')).toHaveLength(1);
+	});
+
+	test('It renders follow button form', async () => {
+		const { container } = render(<FollowButton {...props} variant={'standard'} />);
+		expect(container.querySelector(`form[action='/myft/add/${props.conceptId}']`)).toBeTruthy();
+	});
+
+	test('Button state changes when attributes change', () => {
+		render(<FollowButton {...props}
+			variant={'standard'}
+			setFollowButtonStateToSelected={true}
+			cacheablePersonalisedUrl={true} />);
+		expect(screen.findByText('Added')).toBeTruthy();
+	});
+
+});

package/components/follow-button/follow-button.jsx

@@ -0,0 +1,174 @@
+import React from 'react';
+import CsrfToken from '../csrf-token/input';
+
+function generateFormProps (props) {
+	let generatedProps = {};
+
+	const {
+		collectionName,
+		followPlusDigestEmail,
+		conceptId,
+		setFollowButtonStateToSelected,
+		cacheablePersonalisedUrl
+	} = props;
+
+	if (collectionName) {
+		generatedProps['data-myft-tracking'] = `collectionName=${collectionName}`;
+	}
+
+	if(followPlusDigestEmail) {
+		generatedProps['action'] = `/__myft/api/core/follow-plus-digest-email/${conceptId}?method=put`;
+		generatedProps['data-myft-ui-variant'] = 'followPlusDigestEmail';
+	} else {
+		if(setFollowButtonStateToSelected && cacheablePersonalisedUrl) {
+			generatedProps['action'] = `/myft/remove/${conceptId}`;
+			generatedProps['data-js-action'] = `/__myft/api/core/followed/concept/${conceptId}?method=delete`;
+		} else {
+			generatedProps['action'] = `/myft/add/${conceptId}`;
+			generatedProps['data-js-action'] = `/__myft/api/core/followed/concept/${conceptId}?method=put`;
+		}
+	}
+
+	return generatedProps;
+
+}
+
+function generateButtonProps (props) {
+
+	const {
+		cacheablePersonalisedUrl,
+		setFollowButtonStateToSelected,
+		name,
+		buttonText,
+		variant,
+		conceptId,
+		alternateText,
+		followPlusDigestEmail
+	} = props;
+
+	let generatedProps = {
+		'data-concept-id': conceptId,
+		'n-myft-follow-button': 'true',
+		'data-trackable': 'follow',
+		type: 'submit'
+	};
+
+	if (cacheablePersonalisedUrl && setFollowButtonStateToSelected) {
+		generatedProps['aria-label'] = `Remove ${name} from myFT`;
+		generatedProps['title'] = `Remove ${name} from myFT`
+		generatedProps['data-alternate-label'] = `Add ${name} to myFT`;
+		generatedProps['aria-pressed'] = true;
+
+		if(alternateText) {
+			generatedProps['data-alternate-text'] = alternateText;
+		} else {
+			if(buttonText) {
+				generatedProps['data-alternate-text'] = buttonText;
+			} else {
+				generatedProps['data-alternate-text'] = 'Add to myFT';
+			}
+		}
+	} else {
+		generatedProps['aria-pressed'] = false;
+		generatedProps['aria-label'] = `Add ${name} to myFT`;
+		generatedProps['title'] = `Add ${name} to myFT`;
+		generatedProps['data-alternate-label'] = `Remove ${name} from myFT`;
+		if (alternateText) {
+			generatedProps['data-alternate-text'] = alternateText;
+		} else {
+			if (buttonText) {
+				generatedProps['data-alternate-text'] = buttonText;
+			} else {
+				generatedProps['data-alternate-text'] = 'Added';
+			}
+		}
+	}
+
+	if(variant) {
+		generatedProps[`n-myft-follow-button--${variant}`] = 'true';
+	}
+
+	if(followPlusDigestEmail) {
+		generatedProps['data-trackable-context-messaging'] = 'add-to-myft-plus-digest-button';
+	}
+
+	return generatedProps;
+}
+
+function getButtonText (props) {
+
+	const {
+		buttonText,
+		setFollowButtonStateToSelected,
+		cacheablePersonalisedUrl
+	} = props;
+	let outputText;
+
+	if(buttonText) {
+		outputText = buttonText;
+	} else {
+		if(setFollowButtonStateToSelected && cacheablePersonalisedUrl) {
+			outputText = 'Added';
+		} else {
+			outputText = 'Add to myFT';
+		}
+	}
+
+	return outputText;
+}
+
+/**
+ *
+ * @param {Object} props
+ * @param {string} props.name
+ * @param {Object} props.flags
+ * @param {string} props.extraClasses
+ * @param {string} props.conceptId
+ * @param {string} props.variant
+ * @param {string} props.buttonText
+ * @param {*} props.setFollowButtonStateToSelected
+ * @param {string} props.cacheablePersonalisedUrl
+ * @param {string} props.alternateText
+ * @param {*} props.followPlusDigestEmail
+ * @param {string} props.collectionName
+ */
+export default function FollowButton (props) {
+
+	const {
+		name,
+		flags,
+		extraClasses,
+		conceptId,
+		variant,
+	} = props;
+
+	const formProps = generateFormProps(props);
+	const buttonProps = generateButtonProps(props);
+
+	const getVariantClass = (variant) => variant ? `n-myft-follow-button--${variant}` : '';
+
+	return (
+		<>
+			{flags.myFtApiWrite && <form
+				className={`n-myft-ui n-myft-ui--follow ${extraClasses || ''}`}
+				method="GET"
+				data-myft-ui="follow"
+				data-concept-id={conceptId}
+				{...formProps}>
+				<CsrfToken cacheablePersonalisedUrl={props.cacheablePersonalisedUrl} csrfToken={props.csrfToken} />
+				<div
+					className="n-myft-ui__announcement o-normalise-visually-hidden"
+					aria-live="assertive"
+					data-pressed-text={`Now following ${name}.`}
+					data-unpressed-text={`No longer following ${name}.`}
+				></div>
+				<button
+					{...buttonProps}
+					className={[`n-myft-follow-button ${getVariantClass(variant)}`]}>
+					{getButtonText(props)}
+				</button>
+			</form>}
+		</>
+	);
+
+}

package/components/instant-alert/instant-alert.html

@@ -5,7 +5,7 @@
 			data-concept-id="{{conceptId}}"
 			action="/myft/add/{{conceptId}}?instant=true"
 			data-js-action="/__myft/api/core/followed/concept/{{conceptId}}?method=put">
-	{{> n-myft-ui/components/csrf-token/input}}
+	{{{renderReactComponent localPath="components/csrf-token/input"}}}
 	<input type="hidden" value="{{name}}" name="name">
 	{{#if directType}}
 	<input type="hidden" value="{{directType}}" name="directType">

package/components/pin-button/pin-button.html

@@ -2,7 +2,7 @@
 <span class="myft-pin-divider"></span>
 <div class="myft-pin-button-wrapper">
 	<form method="post" action="/__myft/api/core/prioritised/concept/{{id}}?method={{#if prioritised}}delete{{else}}put{{/if}}" data-myft-prioritise>
-		{{> n-myft-ui/components/csrf-token/input }}
+		{{{renderReactComponent localPath="components/csrf-token/input"}}}
 		<input type="hidden" value="{{name}}" name="name"> {{#if directType}}
 		<input type="hidden" value="{{directType}}" name="directType"> {{else}}
 		<input type="hidden" value="http://www.ft.com/ontology/concept/Concept" name="directType"> {{/if}}

package/components/save-for-later/save-for-later.html

@@ -4,7 +4,7 @@
 	data-myft-ui="saved"
 	action="/myft/save/{{contentId}}"
 	data-js-action="/__myft/api/core/saved/content/{{contentId}}?method=put">
-	{{> n-myft-ui/components/csrf-token/input}}
+	{{{renderReactComponent localPath="components/csrf-token/input"}}}
 	<div
 		class="n-myft-ui__announcement o-normalise-visually-hidden"
 		aria-live="assertive"
@@ -17,10 +17,14 @@
 		data-trackable="{{#if trackableId}}{{trackableId}}{{else}}save-for-later{{/if}}"
 		{{#if isSaved}}
 			{{!-- The value of alternate label needs to be the opposite of label / the current saved state. This allows the client side JS to toggle the labels on state changes --}}
+			title="{{#if title}}{{title}} is{{/if}} Saved to myFT"
+			aria-label="{{#if title}}{{title}} is{{/if}} Saved to myFT"
 			data-alternate-label="{{#if title}}Save {{title}} to myFT for later{{else}}Save this article to myFT for later{{/if}}"
 			aria-pressed="true"
 		{{else}}
-			data-alternate-label="Saved to myFT"
+			title="{{#if title}}Save {{title}} to myFT for later{{else}}Save this article to myFT for later{{/if}}"
+			aria-label="{{#if title}}Save {{title}} to myFT for later{{else}}Save this article to myFT for later{{/if}}"
+			data-alternate-label="{{#if title}}{{title}} is{{/if}} Saved to myFT"
 			aria-pressed="false"
 		{{/if}}
 		{{#unlessEquals appIsStreamPage true}}
@@ -40,13 +44,6 @@
 			{{/if}}
 		{{/if}}
 		data-content-id="{{contentId}}" {{! duplicated here for tracking}}
-		{{#if isSaved}}
-			aria-label="{{#if prefixText}}{{prefixText}}{{/if}} {{#if title}}{{title}} is{{/if}} Saved to myFT"
-			title="{{#if title}}{{title}} is{{/if}} Saved to myFT"
-		{{else}}
-			aria-label="{{#if prefixText}}{{prefixText}}{{/if}} {{#if title}}Save {{title}} to myFT for later{{else}}Save this article to myFT for later{{/if}}"
-			title="{{#if title}}Save {{title}} to myFT for later{{else}}Save this article to myFT for later{{/if}}"
-		{{/if}}
 		>
 			{{#if saveButtonWithIcon}}
 				<span class="save-button-with-icon-copy" data-variant-label>{{#if buttonText~}}

package/components/unread-articles-indicator/date-fns.js

@@ -1,5 +1,5 @@
 // date-fns from v2 doesn't accept String arguments anymore.
-// the detail => https://github.com/date-fns/date-fns/blob/master/CHANGELOG.md#200---2019-08-20
+// the detail => https://github.com/date-fns/date-fns/blob/HEAD/CHANGELOG.md#200---2019-08-20
 // By adding validation for dates before their functions allows us to know it when unexpected value passed.
 
 import isTodayOriginal from 'date-fns/src/isToday';
@@ -11,22 +11,15 @@ import parseISO from 'date-fns/src/parseISO';
 
 const isValid = (date) => {
 	if (!isValidOriginal(date)) {
-		console.error('Invalid date passed', [date]); //eslint-disable-line
+		console.error("Invalid date passed", [date]); //eslint-disable-line
 	}
 	return date;
 };
 
 const isToday = (date) => isTodayOriginal(isValid(date));
-const isAfter = (date, dateToCompare) => isAfterOriginal(isValid(date), isValid(dateToCompare));
+const isAfter = (date, dateToCompare) =>
+	isAfterOriginal(isValid(date), isValid(dateToCompare));
 const addMinutes = (date, amount) => addMinutesOriginal(isValid(date), amount);
 const startOfDay = (date) => startOfDayOriginal(isValid(date));
 
-
-export {
-	isToday,
-	isAfter,
-	addMinutes,
-	startOfDay,
-	isValid,
-	parseISO,
-};
+export { isToday, isAfter, addMinutes, startOfDay, isValid, parseISO };

package/demos/app.js

@@ -1,9 +1,16 @@
-const express = require('@financial-times/n-internal-tool');
+require('sucrase/register');
+const nExpress = require('@financial-times/n-express');
 const chalk = require('chalk');
 const errorHighlight = chalk.bold.red;
 const highlight = chalk.bold.green;
+const { PageKitReactJSX } = require('@financial-times/dotcom-server-react-jsx');
+const fs = require('fs');
+const path = require('path');
+const handlebars = require('handlebars');
+const { PageKitHandlebars, helpers } = require('@financial-times/dotcom-server-handlebars');
 
-const xHandlebars = require('@financial-times/x-handlebars');
+const demoJSX = require('./templates/demo').default;
+const demoLayoutSource = fs.readFileSync(path.join(__dirname, './templates/demo-layout.html'),'utf8').toString();
 
 const fixtures = {
 	followButton: require('./fixtures/follow-button'),
@@ -14,29 +21,38 @@ const fixtures = {
 	instantAlert: require('./fixtures/instant-alert')
 };
 
-const app = module.exports = express({
+const app = module.exports = nExpress({
 	name: 'public',
 	systemCode: 'n-myft-ui-demo',
 	withFlags: true,
-	withHandlebars: true,
-	withNavigation: false,
+	withConsent: false,
+	withServiceMetrics: false,
 	withAnonMiddleware: false,
 	hasHeadCss: false,
-	layoutsDir: 'demos/templates',
-	viewsDirectory: '/demos/templates',
 	partialsDirectory: process.cwd(),
 	directory: process.cwd(),
 	demo: true,
-	s3o: false,
+	withBackendAuthentication: false,
+});
+
+app.set('views', path.join(__dirname, '/templates'));
+app.set('view engine', '.html');
+
+app.engine('.html', new PageKitHandlebars({
+	cache: false,
+	handlebars,
 	helpers: {
-		x: xHandlebars()
+		...helpers
 	}
-});
+}).engine);
+
+app.use('/public', nExpress.static(path.join(__dirname, '../public'), { redirect: false }));
+
+const jsxRenderer = (new PageKitReactJSX({ includeDoctype: false }));
 
 app.get('/', (req, res) => {
 	res.render('demo', Object.assign({
 		title: 'n-myft-ui demo',
-		layout: 'demo-layout',
 		flags: {
 			myFtApi: true,
 			myFtApiWrite: true
@@ -44,18 +60,22 @@ app.get('/', (req, res) => {
 	}, fixtures));
 });
 
-app.get('/digest-on-follow', (req, res) => {
-	res.render('digest-on-follow', Object.assign({
-		title: 'n-myft-ui digest on follow',
-		layout: 'demo-layout',
+app.get('/demo-jsx', async (req, res) => {
+	let demo = await jsxRenderer.render(demoJSX, Object.assign({
+		title: 'n-myft-ui demo',
 		flags: {
 			myFtApi: true,
-			myFtApiWrite: true,
-		},
-		appIsStreamPage: false
-	}, fixtures.followButtonPlusDigest));
+			myFtApiWrite: true
+		}
+	}, fixtures));
+
+	let template = handlebars.compile(demoLayoutSource);
+	let result = template({body: demo});
+
+	res.send(result);
 });
 
+
 function runPa11yTests () {
 	const spawn = require('child_process').spawn;
 	const pa11y = spawn('pa11y-ci');

package/demos/templates/demo-layout.html

@@ -14,7 +14,7 @@
 		<div class="o-grid-row">
 			<ul>
 				<li><a href="/">Basic</a></li>
-				<li><a href="/digest-on-follow">Follow plus digest button</a></li>
+				<li><a href="/demo-jsx">JSX demo</a></li>
 			</ul>
 		</div>
 	</div>