@financial-times/n-myft-ui 33.0.0 → 33.1.0

package/components/jsx/preferences-modal/index.js

@@ -11,10 +11,10 @@ const renderError = ({ message, preferencesModal }) => {
 
 const removeError = ({ preferencesModal }) => {
 	renderError({
-		message: "",
+		message: '',
 		preferencesModal,
 	});
-}
+};
 
 /**
  * This preference modal is part of a test
@@ -60,7 +60,7 @@ const toggleCheckboxStatus = ({ instantAlertsCheckbox, isChecked }) => {
 		instantAlertsCheckbox.dataset.trackable = 'pop-up-box|set-instant-alert-on';
 		instantAlertsCheckbox.checked = false;
 	}
-}
+};
 
 const tracking = (conceptId) => {
 	const trackingData = {
@@ -72,7 +72,7 @@ const tracking = (conceptId) => {
 			name: 'pop-up-box',
 			id: '72de123e-5082-11ee-be56-0242ac120002',
 		}
-	}
+	};
 
 	const trackingEvent = new CustomEvent('oTracking.event', {
 		detail: trackingData,
@@ -218,7 +218,7 @@ const toggleInstantAlertsPreference = async ({ event, conceptId, preferencesModa
 	instantAlertsCheckbox.removeAttribute('disabled');
 };
 
-const setCheckboxForAlertConceptToOff =  ({ event, preferencesModal }) => {
+const setCheckboxForAlertConceptToOff = ({ event, preferencesModal }) => {
 	const conceptId = preferencesModal.dataset.conceptId;
 	const instantAlertsCheckbox = preferencesModal.querySelector('[data-component-id="myft-preferences-modal-checkbox"]');
 
@@ -231,7 +231,7 @@ const setCheckboxForAlertConceptToOff =  ({ event, preferencesModal }) => {
 		instantAlertsCheckbox,
 		isChecked: false,
 	});
-}
+};
 
 export default () => {
 	/**

package/karma.conf.js

@@ -155,7 +155,7 @@ module.exports = function (karma) {
 			safari: {
 				base: 'BrowserStack',
 				os: 'OS X',
-				os_version : 'High Sierra',
+				os_version : 'Monterey',
 				browser: 'Safari',
 				browser_version: 'latest'
 			}

package/myft/ui/lists.js

@@ -500,6 +500,12 @@ function openCreateListAndAddArticleOverlay (contentId, config) {
 function initialEventListeners () {
 
 	document.body.addEventListener('myft.user.saved.content.add', event => {
+		//Only show the create list overlay if the saved article is the main article, this is useful if there are other article listed in the article page
+		//For example having a related articles that can also be saved
+		const currentContentId = document.querySelector('[data-content-id]').dataset.contentId;
+		if (event && event.detail && currentContentId !== event.detail.subject) {
+			return;
+		}
 		event.stopPropagation();
 		const contentId = event.detail.subject;
 		const configSet = event.currentTarget.querySelector('[data-myft-ui-save-config]');
@@ -512,6 +518,12 @@ function initialEventListeners () {
 	});
 
 	document.body.addEventListener('myft.user.saved.content.remove', event => {
+		//Only show unsave notification if the saved article is the main article, this is useful if there are other article listed in the article page
+		//For example having a related articles that can also be saved
+		const currentContentId = document.querySelector('[data-content-id]').dataset.contentId;
+		if (event && event.detail && currentContentId !== event.detail.subject) {
+			return;
+		}
 		event.stopPropagation();
 		const contentId = event.detail.subject;
 		return showUnsavedNotification(contentId);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@financial-times/n-myft-ui",
-  "version": "33.0.0",
+  "version": "33.1.0",
   "description": "Client side component for interaction with myft",
   "main": "server.js",
   "scripts": {
@@ -63,7 +63,7 @@
     "karma-chrome-launcher": "3.2.0",
     "karma-firefox-launcher": "^1.0.0",
     "karma-html-reporter": "^0.2.6",
-    "karma-mocha": "^1.3.0",
+    "karma-mocha": "^2.0.1",
     "karma-sinon": "^1.0.5",
     "karma-sinon-chai": "2.0.2",
     "karma-sourcemap-loader": "^0.3.7",
@@ -71,7 +71,7 @@
     "karma-webpack": "^4.0.2",
     "lintspaces-cli": "^0.7.0",
     "lolex": "5.1.2",
-    "mocha": "6.2.2",
+    "mocha": "^10.2.0",
     "mockery": "2.1.0",
     "node-fetch": "2.6.0",
     "nodemon": "^1.9.2",