@financial-times/n-myft-ui 30.0.2 → 30.1.1

package/build-state/npm-shrinkwrap.json

@@ -101,7 +101,6 @@
         "@financial-times/o-overlay": "^4.0.0",
         "@financial-times/o-spacing": "^3.0.0",
         "@financial-times/o-tooltip": "^5.2.4",
-        "@financial-times/o-topper": "^5.2.3",
         "n-ui-foundations": "^9.0.0"
       }
     },
@@ -2851,12 +2850,6 @@
         "node": "12.x"
       }
     },
-    "node_modules/@financial-times/n-map-content-to-topper": {
-      "version": "1.6.0",
-      "resolved": "https://registry.npmjs.org/@financial-times/n-map-content-to-topper/-/n-map-content-to-topper-1.6.0.tgz",
-      "integrity": "sha512-7AezWXpfEEZkz21Taw47h9SoM6Ycg7vrKZ3SOVpo1f0vqRbK7VpnvLOTjhTGOkMdPi5z/5kvjN5ag5P4yvpRMg==",
-      "peer": true
-    },
     "node_modules/@financial-times/n-notification": {
       "version": "8.2.2",
       "resolved": "https://registry.npmjs.org/@financial-times/n-notification/-/n-notification-8.2.2.tgz",
@@ -3117,24 +3110,6 @@
         "@financial-times/o-visual-effects": "^4.0.1"
       }
     },
-    "node_modules/@financial-times/o-topper": {
-      "version": "5.2.3",
-      "resolved": "https://registry.npmjs.org/@financial-times/o-topper/-/o-topper-5.2.3.tgz",
-      "integrity": "sha512-4CoeEp5nG7ecmZuJMr7DlEs0w5mPS3jikHxOlVZqMP18qWlwXC6BgbPz1ZslxBt/+SnJmHlNN2VIQevPQLNtEQ==",
-      "peer": true,
-      "engines": {
-        "npm": "^7 || ^8"
-      },
-      "peerDependencies": {
-        "@financial-times/n-map-content-to-topper": "^1.5.0",
-        "@financial-times/o-colors": "^6.0.1",
-        "@financial-times/o-editorial-typography": "^2.0.1",
-        "@financial-times/o-grid": "^6.0.0",
-        "@financial-times/o-icons": "^7.0.1",
-        "@financial-times/o-spacing": "^3.0.0",
-        "@financial-times/o-typography": "^7.0.1"
-      }
-    },
     "node_modules/@financial-times/o-typography": {
       "version": "7.3.2",
       "resolved": "https://registry.npmjs.org/@financial-times/o-typography/-/o-typography-7.3.2.tgz",
@@ -26123,12 +26098,6 @@
         "winston": "^2.4.0"
       }
     },
-    "@financial-times/n-map-content-to-topper": {
-      "version": "1.6.0",
-      "resolved": "https://registry.npmjs.org/@financial-times/n-map-content-to-topper/-/n-map-content-to-topper-1.6.0.tgz",
-      "integrity": "sha512-7AezWXpfEEZkz21Taw47h9SoM6Ycg7vrKZ3SOVpo1f0vqRbK7VpnvLOTjhTGOkMdPi5z/5kvjN5ag5P4yvpRMg==",
-      "peer": true
-    },
     "@financial-times/n-notification": {
       "version": "8.2.2",
       "resolved": "https://registry.npmjs.org/@financial-times/n-notification/-/n-notification-8.2.2.tgz",
@@ -26268,13 +26237,6 @@
         "ftdomdelegate": "^4.0.6"
       }
     },
-    "@financial-times/o-topper": {
-      "version": "5.2.3",
-      "resolved": "https://registry.npmjs.org/@financial-times/o-topper/-/o-topper-5.2.3.tgz",
-      "integrity": "sha512-4CoeEp5nG7ecmZuJMr7DlEs0w5mPS3jikHxOlVZqMP18qWlwXC6BgbPz1ZslxBt/+SnJmHlNN2VIQevPQLNtEQ==",
-      "peer": true,
-      "requires": {}
-    },
     "@financial-times/o-typography": {
       "version": "7.3.2",
       "resolved": "https://registry.npmjs.org/@financial-times/o-typography/-/o-typography-7.3.2.tgz",

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

@@ -5,7 +5,7 @@
 	action="/myft/save/{{contentId}}"
 	data-js-action="/__myft/api/core/saved/content/{{contentId}}?method=put"
 	{{#if @root.flags.manageArticleLists}}data-myft-ui-save-new="manageArticleLists"{{/if}}
-	{{#if @root.flags.manageArticleLists}}data-myft-ui-save-new-config="{{#if @root.flags.myftListPublicPrivateToggle}}showPublicToggle{{/if}},{{#if modal}}modal{{/if}}"{{/if}}>
+	{{#if @root.flags.manageArticleLists}}data-myft-ui-save-new-config="{{#if modal}}modal{{/if}}"{{/if}}>
 	{{> n-myft-ui/components/csrf-token/input}}
 	<div
 		class="n-myft-ui__announcement o-normalise-visually-hidden"

package/myft/ui/lists.js

@@ -291,7 +291,7 @@ function initialEventListeners () {
 			// These will be used to build a sanity check dashboard, and will be removed after we get clean-up this test.
 			document.body.dispatchEvent(new CustomEvent('oTracking.event', {
 				detail: {
-					category: 'publicToggle',
+					category: 'lists',
 					action: 'savedArticle',
 					article_id: contentId,
 					teamName: 'customer-products-us-growth',

package/myft/ui/save-article-to-list-variant.js

@@ -14,7 +14,7 @@ let scrolledOnOpen;
 let listOverlayBottom;
 
 export default async function openSaveArticleToListVariant (contentId, options = {}) {
-	const { name, showPublicToggle = false, modal = false } = options;
+	const { name, modal = false } = options;
 
 	function createList (newList, cb) {
 		if(!newList || !newList.name) {
@@ -112,7 +112,7 @@ export default async function openSaveArticleToListVariant (contentId, options =
 
 	function openFormHandler () {
 		hideListElement();
-		const formElement = FormElement(createList, showPublicToggle, attachDescription, onFormListCreated, onFormCancel, modal);
+		const formElement = FormElement(createList, attachDescription, onFormListCreated, onFormCancel, modal);
 		const overlayContent = document.querySelector('.o-overlay__content');
 		removeDescription();
 		overlayContent.insertAdjacentElement('beforeend', formElement);
@@ -166,7 +166,7 @@ function getResizeHandler (target) {
 	};
 }
 
-function FormElement (createList, showPublicToggle, attachDescription, onListCreated, onCancel, modal=false) {
+function FormElement (createList, attachDescription, onListCreated, onCancel, modal=false) {
 	const formString = `
 	<form class="myft-ui-create-list-variant-form">
 		<label class="myft-ui-create-list-variant-form-name o-forms-field">
@@ -176,24 +176,21 @@ function FormElement (createList, showPublicToggle, attachDescription, onListCre
 			</span>
 		</label>
 
-		${showPublicToggle ?
-		`<div class="myft-ui-create-list-variant-form-public o-forms-field" role="group">
-				<span class="o-forms-input o-forms-input--toggle">
-					<label>
-						<input class="myft-ui-create-list-variant-form-toggle" type="checkbox" name="is-public" value="public" checked data-trackable="private-link" text="private">
-						<span class="myft-ui-create-list-variant-form-toggle-label o-forms-input__label">
-							<span class="o-forms-input__label__main">
-								Public
-							</span>
-							<span id="myft-ui-create-list-variant-form-public-description" class="o-forms-input__label__prompt">
-								Your profession & list will be visible to others
-							</span>
+		<div class="myft-ui-create-list-variant-form-public o-forms-field" role="group">
+			<span class="o-forms-input o-forms-input--toggle">
+				<label>
+					<input class="myft-ui-create-list-variant-form-toggle" type="checkbox" name="is-public" value="public" checked data-trackable="private-link" text="private">
+					<span class="myft-ui-create-list-variant-form-toggle-label o-forms-input__label">
+						<span class="o-forms-input__label__main">
+							Public
 						</span>
-					</label>
-				</span>
-			</div>` :
-		''
-}
+						<span id="myft-ui-create-list-variant-form-public-description" class="o-forms-input__label__prompt">
+							Your profession & list will be visible to others
+						</span>
+					</span>
+				</label>
+			</span>
+		</div>
 
 		<div class="myft-ui-create-list-variant-form-buttons">
 			<button class="o-buttons o-buttons--primary o-buttons--inverse o-buttons--big" type="button" data-trackable="cancel-link" text="cancel">
@@ -233,7 +230,6 @@ function FormElement (createList, showPublicToggle, attachDescription, onListCre
 	function handleCancelClick (event) {
 		event.preventDefault();
 		event.stopPropagation();
-		triggerCancelEvent();
 		formElement.remove();
 		if (!lists.length) attachDescription();
 		onCancel();
@@ -242,9 +238,7 @@ function FormElement (createList, showPublicToggle, attachDescription, onListCre
 	formElement.querySelector('button[type="submit"]').addEventListener('click', handleSubmit);
 	formElement.querySelector('button[type="button"]').addEventListener('click', handleCancelClick);
 
-	if (showPublicToggle) {
-		addPublicToggleListener(formElement);
-	}
+	addPublicToggleListener(formElement);
 
 	return formElement;
 }
@@ -253,7 +247,6 @@ function addPublicToggleListener (formElement) {
 	function onPublicToggleClick (event) {
 		event.target.setAttribute('data-trackable', event.target.checked ? 'private-link' : 'public-link');
 		event.target.setAttribute('text', event.target.checked ? 'private' : 'public');
-		triggerPublicToggleEvent(event.target.checked);
 	}
 
 	formElement.querySelector('input[name="is-public"]').addEventListener('click', onPublicToggleClick);
@@ -488,20 +481,6 @@ function triggerCreateListEvent (contentId, listId) {
 	}));
 }
 
-// Temporary event on the public toggle feature.
-// These will be used to build a sanity check dashboard, and will be removed after we get clean-up this test.
-function triggerPublicToggleEvent (isPublic) {
-	document.body.dispatchEvent(new CustomEvent('oTracking.event', {
-		detail: {
-			category: 'publicToggle',
-			action: `${isPublic ? 'setPublic' : 'setPrivate'}`,
-			teamName: 'customer-products-us-growth',
-			amplitudeExploratory: true
-		},
-		bubbles: true
-	}));
-}
-
 // Temporary event on the public toggle feature.
 // These will be used to build a sanity check dashboard, and will be removed after we get clean-up this test.
 function triggerAddToNewListEvent () {
@@ -516,20 +495,6 @@ function triggerAddToNewListEvent () {
 	}));
 }
 
-// Temporary event on the public toggle feature.
-// These will be used to build a sanity check dashboard, and will be removed after we get clean-up this test.
-function triggerCancelEvent () {
-	document.body.dispatchEvent(new CustomEvent('oTracking.event', {
-		detail: {
-			category: 'publicToggle',
-			action: 'cancel ',
-			teamName: 'customer-products-us-growth',
-			amplitudeExploratory: true
-		},
-		bubbles: true
-	}));
-}
-
 //Temporary event to determine whether users need to scroll to add to a list
 function checkScrollToAdd () {
 	//if the bottom of the overlay was not showing and scrolling has occurred since it was opened

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@financial-times/n-myft-ui",
-  "version": "30.0.2",
+  "version": "30.1.1",
   "description": "Client side component for interaction with myft",
   "main": "server.js",
   "scripts": {
@@ -96,7 +96,6 @@
     "@financial-times/o-overlay": "^4.0.0",
     "@financial-times/o-spacing": "^3.0.0",
     "@financial-times/o-tooltip": "^5.2.4",
-    "@financial-times/o-topper": "^5.2.3",
     "n-ui-foundations": "^9.0.0"
   },
   "dependencies": {