@openui5/sap.ui.rta 1.136.4 → 1.139.0

package/src/sap/ui/rta/util/guidedTour/GuidedTour.js

@@ -27,7 +27,7 @@ sap.ui.define([
 	 * @class Constructor for a new sap.ui.rta.util.guidedTour.GuidedTour
 	 * @extends sap.ui.base.ManagedObject
 	 * @author SAP SE
-	 * @version 1.136.4
+	 * @version 1.139.0
 	 * @constructor
 	 * @since 1.136
 	 * @private
@@ -99,6 +99,13 @@ sap.ui.define([
 		}
 	}
 
+	function restorePopoverModality() {
+		// If the affected control was a Popover, we need to restore the modal state
+		if (this._oAffectedControl?.isA("sap.m.Popover")) {
+			this._oAffectedControl.setModal(this._oAffectedControlOriginalModalState);
+		}
+	}
+
 	GuidedTour.prototype.autoStart = async function(oTourContent) {
 		const sResponse = await Utils.showMessageBox(
 			"information",
@@ -152,16 +159,23 @@ sap.ui.define([
 		try {
 			for (const sActionSelector of aActionSelectors) {
 				const oActionControl = await waitForElementVisibility.call(this, sActionSelector);
-				oActionControl.firePress();
+				await oActionControl.firePress();
 			}
 
-			const oAffectedControl = oStepConfig.waitForElement
+			this._oAffectedControl = oStepConfig.waitForElement
 				? await waitForElementVisibility.call(this, sSelector)
 				: Element.getElementById(sSelector);
 
 			// Open the popover if the element is visible
-			if (oAffectedControl && oAffectedControl.getVisible()) {
-				this._oPopover.openBy(oAffectedControl);
+			if (this._oAffectedControl && this._oAffectedControl.getVisible()) {
+				// If the affected control is a Popover, we need to open it modal
+				// because if the affected control is not modal, it could be closed
+				// this would also close the Guided Tour Popover
+				if (this._oAffectedControl.isA("sap.m.Popover")) {
+					this._oAffectedControlOriginalModalState = this._oAffectedControl.getModal();
+					this._oAffectedControl.setModal(true);
+				}
+				this._oPopover.openBy(this._oAffectedControl);
 			} else {
 				skipStep.call(this);
 			}
@@ -184,6 +198,7 @@ sap.ui.define([
 	};
 
 	GuidedTour.prototype.onNextPress = function() {
+		restorePopoverModality.call(this);
 		this.setProperty("forwardDirection", true);
 		this._oPopover.close();
 		const nCurrentStep = this.getCurrentStep();
@@ -200,6 +215,7 @@ sap.ui.define([
 	};
 
 	GuidedTour.prototype.onPreviousPress = function() {
+		restorePopoverModality.call(this);
 		this.setProperty("forwardDirection", false);
 		this._oPopover.close();
 		const nCurrentStep = this.getCurrentStep();
@@ -225,6 +241,7 @@ sap.ui.define([
 	};
 
 	GuidedTour.prototype.onClosePress = async function() {
+		restorePopoverModality.call(this);
 		this._oPopover.close();
 		this.fireTourClosed();
 		this.destroy();

package/src/sap/ui/rta/util/guidedTour/TourMarker.fragment.xml

@@ -36,7 +36,7 @@
 			<OverflowToolbar>
 				<ToolbarSpacer/>
 				<Button
-					id="previewsButton"
+					id="previousButton"
 					text="{i18n>BTN_GUIDED_TOUR_PREV}"
 					visible="{= !${/isFirstStep}}"
 					press=".onPreviousPress" />

package/src/sap/ui/rta/util/guidedTour/content/GeneralTour.js

@@ -129,7 +129,7 @@ sap.ui.define([
 				title: oTextResources.getText("TIT_TOUR_GENERAL_STEP_VERSIONS_DROPDOWN_TITLE"),
 				description: oTextResources.getText("TXT_TOUR_GENERAL_STEP_VERSIONS_DROPDOWN_DESCRIPTION"),
 				listContent: [],
-				markerSelector: "sapUIRta_toolbar_fragment--sapUiRta_versionHistoryDialog--originalVersionList",
+				markerSelector: "sapUIRta_toolbar_fragment--sapUiRta_versionHistoryDialog--popover",
 				waitForElement: true,
 				actionSelectors: ["sapUIRta_toolbar_fragment--sapUiRta_versionButton"]
 			},

package/src/sap/ui/rta/util/hasStableId.js

@@ -16,15 +16,15 @@ sap.ui.define([
 	"use strict";
 
 	function isCloneFromAggregationBinding(oControl) {
-		var {sParentAggregationName} = oControl;
-		var oParent = oControl.getParent();
+		const {sParentAggregationName} = oControl;
+		const oParent = oControl.getParent();
 
 		if (oParent && sParentAggregationName) {
-			var oBindingInfo = oParent.getBindingInfo(sParentAggregationName);
+			const oBindingInfo = oParent.getBindingInfo(sParentAggregationName);
 			if (oBindingInfo) {
 				if (
 					oBindingInfo.template
-					&& oControl instanceof oBindingInfo.template.getMetadata().getClass()
+					&& oControl.isA(oBindingInfo.template.getMetadata().getClass())
 				) {
 					return oBindingInfo.template;
 				}
@@ -47,31 +47,23 @@ sap.ui.define([
 		}
 
 		if (typeof oElementOverlay.data("hasStableId") !== "boolean") {
-			var aStableElements = oElementOverlay.getDesignTimeMetadata().getStableElements(oElementOverlay);
-			var bUnstable = false;
+			const aStableElements = oElementOverlay.getDesignTimeMetadata().getStableElements(oElementOverlay);
+			let bUnstable = false;
 
 			if (aStableElements.length > 0) {
 				if (isCloneFromAggregationBinding(oElementOverlay.getElement())) {
 					bUnstable = aStableElements.some(function(vStableElement) {
-						var vControl;
-						var oAppComponent;
-						var bUnstable = false;
-
-						if (isPlainObject(vStableElement)) {
-							vControl = vStableElement.id;
-							oAppComponent = vStableElement.appComponent;
-						} else {
-							vControl = vStableElement;
-						}
+						const vControl = isPlainObject(vStableElement) ? vStableElement.id : vStableElement;
+						const oAppComponent = isPlainObject(vStableElement) && vStableElement.appComponent;
 
 						// 1. First we check control as is, as getStableElements() may already return a control from the template
 						bUnstable = !FlUtils.checkControlId(vControl, oAppComponent);
 
 						// 2. If it's unstable, we try to detect the corresponding control in the template manually
 						if (bUnstable) {
-							var oControl = ElementUtil.getElementInstance(vControl);
+							const oControl = ElementUtil.getElementInstance(vControl);
 							if (ElementUtil.getElementInstance(vControl)) {
-								var mLocationInTemplate = ElementUtil.getAggregationInformation(oControl);
+								const mLocationInTemplate = ElementUtil.getAggregationInformation(oControl);
 								bUnstable = !FlUtils.checkControlId(ElementUtil.extractTemplateId(mLocationInTemplate), oAppComponent);
 							}
 						}
@@ -80,7 +72,7 @@ sap.ui.define([
 					});
 				} else {
 					bUnstable = aStableElements.some(function(vStableElement) {
-						var vControl = vStableElement.id || vStableElement;
+						const vControl = vStableElement.id || vStableElement;
 						return !FlUtils.checkControlId(vControl, vStableElement.appComponent);
 					});
 				}

package/src/sap/ui/rta/util/showMessageBox.js

@@ -17,20 +17,20 @@ sap.ui.define([
 ) {
 	"use strict";
 
-	var sLinkRegex = "\\[(.*?)\\]\\((.*?)\\)";
+	const sLinkRegex = "\\[(.*?)\\]\\((.*?)\\)";
 
 	function hasLinks(sMessage) {
-		var vResult = getLinks(sMessage);
+		const vResult = getLinks(sMessage);
 		return Array.isArray(vResult) && vResult.length > 0;
 	}
 
 	function isLink(sText) {
-		var oRegex = new RegExp(`^${sLinkRegex}$`);
+		const oRegex = new RegExp(`^${sLinkRegex}$`);
 		return Array.isArray(oRegex.exec(sText));
 	}
 
 	function extractLink(sLink) {
-		var aMatch = new RegExp(sLinkRegex).exec(sLink);
+		const aMatch = new RegExp(sLinkRegex).exec(sLink);
 		return {
 			text: aMatch[1],
 			href: aMatch[2]
@@ -38,16 +38,16 @@ sap.ui.define([
 	}
 
 	function getLinks(sMessage) {
-		var oRegExp = new RegExp(sLinkRegex, "g");
+		const oRegExp = new RegExp(sLinkRegex, "g");
 		return sMessage.match(oRegExp);
 	}
 
 	function convertIntoControls(aSymbols) {
-		var oLayout = new HorizontalLayout({
+		const oLayout = new HorizontalLayout({
 			allowWrapping: true,
 			content: aSymbols.map(function(sSymbol) {
 				if (isLink(sSymbol)) {
-					var mLink = extractLink(sSymbol);
+					const mLink = extractLink(sSymbol);
 					return new Link({
 						text: mLink.text,
 						href: mLink.href,
@@ -69,19 +69,19 @@ sap.ui.define([
 	}
 
 	function getSymbols(sMessage) {
-		var aSymbols = [sMessage];
-		var aLinks = getLinks(sMessage);
+		const aSymbols = [sMessage];
+		const aLinks = getLinks(sMessage);
 
 		aLinks.forEach(function(sLink) {
-			var i = 0;
+			let i = 0;
 			while (i < aSymbols.length) {
-				var sSymbol = aSymbols[i];
+				const sSymbol = aSymbols[i];
 
 				if (isLink(sSymbol)) {
 					i++;
 				} else {
-					var aParts = sSymbol.split(sLink);
-					var aInsert = [];
+					const aParts = sSymbol.split(sLink);
+					const aInsert = [];
 
 					aParts.forEach(function(sPart, iIndex, aOriginal) { // eslint-disable-line no-loop-func
 						aInsert.push(sPart);
@@ -111,10 +111,10 @@ sap.ui.define([
 	 * @param {string} [sMessageType="show"] - Decides the type of the MessageBox that should be shown with (see different types at {@link sap.m.MessageBox})
 	 */
 	return function showMessageBox(sMessage, mOptions, sMessageType) {
-		var vMessage;
+		let vMessage;
 
 		if (hasLinks(sMessage)) {
-			var aSymbols = getSymbols(sMessage);
+			const aSymbols = getSymbols(sMessage);
 			vMessage = convertIntoControls(aSymbols);
 		} else {
 			vMessage = sMessage;

package/src/sap/ui/rta/util/validateFlexEnabled.js

@@ -24,13 +24,14 @@ sap.ui.define([
 	DtUtil
 ) {
 	"use strict";
+
 	return function(oRta) {
-		var mMessageBoxShow = {};
-		var aPendingOverlaysToValidate = [];
-		var oComponent = FlUtils.getAppComponentForControl(oRta.getRootControlInstance());
+		const mMessageBoxShow = {};
+		let aPendingOverlaysToValidate = [];
+		const oComponent = FlUtils.getAppComponentForControl(oRta.getRootControlInstance());
 
-		function _displayMessage(oRta, oComponent, sText, sIconType, sTitle) {
-			var sComponentId = oComponent.getId();
+		function displayMessage(oRta, oComponent, sText, sIconType, sTitle) {
+			const sComponentId = oComponent.getId();
 			if (!mMessageBoxShow[sComponentId]) {
 				showMessageBox(
 					oRta._getTextResources().getText(sText),
@@ -41,16 +42,16 @@ sap.ui.define([
 					},
 					"show"
 				);
-				_setMessageBoxShow(sComponentId, true);
+				setMessageBoxShow(sComponentId, true);
 			}
 		}
 
-		function _setMessageBoxShow(sComponentId, bValue) {
+		function setMessageBoxShow(sComponentId, bValue) {
 			mMessageBoxShow[sComponentId] = bValue;
 		}
 
-		function _isValidApp(oComponent) {
-			var oManifest = oComponent.getManifest();
+		function isValidApp(oComponent) {
+			const oManifest = oComponent.getManifest();
 
 			return (
 				ObjectPath.get(["sap.app", "id"], oManifest) !== "sap.ui.documentation.sdk"
@@ -59,7 +60,7 @@ sap.ui.define([
 			);
 		}
 
-		function _isTestEnvironment() {
+		function isTestEnvironment() {
 			return (
 				"QUnit" in window
 				|| (
@@ -69,28 +70,28 @@ sap.ui.define([
 			);
 		}
 
-		function _isControlAvailable(oControl) {
+		function isControlAvailable(oControl) {
 			return oControl && !oControl._bIsBeingDestroyed;
 		}
 
-		function _handleModeChanged(oEvent) {
-			var sNewMode = oEvent.getParameters().mode;
+		function handleModeChanged(oEvent) {
+			const sNewMode = oEvent.getParameters().mode;
 			if (sNewMode === "adaptation") {
-				var oRta = oEvent.getSource();
-				var oComponent = FlUtils.getAppComponentForControl(oRta.getRootControlInstance());
-				aPendingOverlaysToValidate = aPendingOverlaysToValidate.filter(_isControlAvailable);
-				_handleUnstableIds(oRta, oComponent, aPendingOverlaysToValidate);
+				const oRta = oEvent.getSource();
+				const oComponent = FlUtils.getAppComponentForControl(oRta.getRootControlInstance());
+				aPendingOverlaysToValidate = aPendingOverlaysToValidate.filter(isControlAvailable);
+				handleUnstableIds(oRta, oComponent, aPendingOverlaysToValidate);
 				aPendingOverlaysToValidate = [];
 			}
 		}
 
-		function _validateCreatedOverlay(oEvent, oRta) {
-			var oElementOverlayCreated = oEvent.getParameters().elementOverlay;
+		function validateCreatedOverlay(oEvent, oRta) {
+			const oElementOverlayCreated = oEvent.getParameters().elementOverlay;
 			if (oRta.getMode() === "adaptation") {
-				var oComponent = FlUtils.getAppComponentForControl(oRta.getRootControlInstance());
+				const oComponent = FlUtils.getAppComponentForControl(oRta.getRootControlInstance());
 				DtUtil.waitForSynced(oRta._oDesignTime, function(oOverlay) {
-					if (_isControlAvailable(oOverlay)) {
-						_handleUnstableIds(oRta, oComponent, [oOverlay]);
+					if (isControlAvailable(oOverlay)) {
+						handleUnstableIds(oRta, oComponent, [oOverlay]);
 					}
 				})(oElementOverlayCreated);
 			} else {
@@ -98,33 +99,33 @@ sap.ui.define([
 			}
 		}
 
-		function _handleUnstableIds(oRta, oComponent, aElementOverlays) {
-			var aUnstableOverlays = validateStableIds(aElementOverlays, oComponent);
+		function handleUnstableIds(oRta, oComponent, aElementOverlays) {
+			const aUnstableOverlays = validateStableIds(aElementOverlays, oComponent);
 
 			if (aUnstableOverlays.length) {
 				aUnstableOverlays.forEach(function(oElementOverlay) {
 					Log.error("Control ID was generated dynamically by SAPUI5. To support SAPUI5 flexibility, a stable control ID is needed to assign the changes to.", oElementOverlay.getElement().getId());
 				});
-				_displayMessage(oRta, oComponent, "MSG_UNSTABLE_ID_FOUND", "ERROR", "HEADER_ERROR");
+				displayMessage(oRta, oComponent, "MSG_UNSTABLE_ID_FOUND", "ERROR", "HEADER_ERROR");
 			}
 		}
 
 		oRta.attachEventOnce("stop", function() {
-			_setMessageBoxShow(oComponent.getId(), false);
+			setMessageBoxShow(oComponent.getId(), false);
 		});
 
-		if (oComponent && _isValidApp(oComponent)) {
-			var oManifest = oComponent.getManifest();
-			var vFlexEnabled = ObjectPath.get(["sap.ui5", "flexEnabled"], oManifest);
+		if (oComponent && isValidApp(oComponent)) {
+			const oManifest = oComponent.getManifest();
+			const vFlexEnabled = ObjectPath.get(["sap.ui5", "flexEnabled"], oManifest);
 
 			if (typeof vFlexEnabled !== "boolean") {
-				if (!_isTestEnvironment()) {
-					_displayMessage(oRta, oComponent, "MSG_NO_FLEX_ENABLED_FLAG", "WARNING", "HEADER_WARNING");
+				if (!isTestEnvironment()) {
+					displayMessage(oRta, oComponent, "MSG_NO_FLEX_ENABLED_FLAG", "WARNING", "HEADER_WARNING");
 				}
 			} else { // flexEnabled === true
-				oRta.attachEvent("modeChanged", _handleModeChanged);
-				oRta._oDesignTime.attachEvent("elementOverlayCreated", oRta, _validateCreatedOverlay);
-				_handleUnstableIds(oRta, oComponent, oRta._oDesignTime.getElementOverlays());
+				oRta.attachEvent("modeChanged", handleModeChanged);
+				oRta._oDesignTime.attachEvent("elementOverlayCreated", oRta, validateCreatedOverlay);
+				handleUnstableIds(oRta, oComponent, oRta._oDesignTime.getElementOverlays());
 			}
 		}
 	};

package/src/sap/ui/rta/util/validateStableIds.js

@@ -7,40 +7,36 @@
 sap.ui.define([
 	"sap/base/util/ObjectPath",
 	"sap/ui/rta/util/hasStableId",
-	"sap/ui/core/mvc/View",
-	"sap/ui/core/Fragment",
 	"sap/ui/dt/OverlayUtil",
 	"sap/base/util/isPlainObject"
 ], function(
 	ObjectPath,
 	hasStableId,
-	View,
-	Fragment,
 	OverlayUtil,
 	isPlainObject
 ) {
 	"use strict";
 
 	function isFioriElementsApp(oComponent) {
-		var mManifest = oComponent.getManifest();
+		const mManifest = oComponent.getManifest();
 
-		var isV4 = !!ObjectPath.get(["sap.ui5", "dependencies", "libs", "sap.fe.templates"], mManifest);
-		var isV2 = !!ObjectPath.get(["sap.ui.generic.app"], mManifest);
-		var isOVP = !!ObjectPath.get(["sap.ovp"], mManifest);
+		const isV4 = !!ObjectPath.get(["sap.ui5", "dependencies", "libs", "sap.fe.templates"], mManifest);
+		const isV2 = !!ObjectPath.get(["sap.ui.generic.app"], mManifest);
+		const isOVP = !!ObjectPath.get(["sap.ovp"], mManifest);
 		return isV2 || isV4 || isOVP;
 	}
 
 	function getFioriElementsExtensions(oComponent) {
-		var oManifest = oComponent.getManifest();
+		const oManifest = oComponent.getManifest();
 
-		var mViewExtensions = ObjectPath.get(["sap.ui5", "extends", "extensions", "sap.ui.viewExtensions"], oManifest);
+		const mViewExtensions = ObjectPath.get(["sap.ui5", "extends", "extensions", "sap.ui.viewExtensions"], oManifest);
 
-		var aViewExtensions = [];
+		const aViewExtensions = [];
 
 		if (isPlainObject(mViewExtensions)) {
 			Object.keys(mViewExtensions).forEach(function(sViewExtensionGroupName) {
 				if (sViewExtensionGroupName.startsWith("sap.suite.ui.generic.template")) {
-					var mViewExtensionGroup = mViewExtensions[sViewExtensionGroupName];
+					const mViewExtensionGroup = mViewExtensions[sViewExtensionGroupName];
 
 					Object.keys(mViewExtensionGroup).forEach(function(sViewExtensionName) {
 						aViewExtensions.push(mViewExtensionGroup[sViewExtensionName]);
@@ -53,20 +49,19 @@ sap.ui.define([
 	}
 
 	function getExtensionOverlays(aExtensionList, aElementOverlays) {
-		var aExtensionOverlays = [];
+		const aExtensionOverlays = [];
 
-		for (var i = 0, l = aElementOverlays.length; i < l; i++) {
-			var oElementOverlay = aElementOverlays[i];
-			var oElement = oElementOverlay.getElement();
-			var bIsExtensionOverlay = aExtensionList.some(function(mViewExtension) { // eslint-disable-line no-loop-func
-				var ClassDeclaration = sap.ui.require(mViewExtension.className.replace(/\./g, "/"));
-				var sExtensionName;
-				var sElementName;
+		for (let i = 0, l = aElementOverlays.length; i < l; i++) {
+			const oElementOverlay = aElementOverlays[i];
+			const oElement = oElementOverlay.getElement();
+			const bIsExtensionOverlay = aExtensionList.some(function(mViewExtension) { // eslint-disable-line no-loop-func
+				let sExtensionName;
+				let sElementName;
 
-				if (oElement instanceof View) {
+				if (oElement.isA("sap.ui.core.mvc.View")) {
 					sElementName = oElement.getViewName();
 					sExtensionName = mViewExtension.viewName;
-				} else if (oElement instanceof Fragment) {
+				} else if (oElement.isA("sap.ui.core.Fragment")) {
 					sElementName = oElement.getFragmentName();
 				} else {
 					// viewName/fragmentName are essential for proper element detection
@@ -74,7 +69,7 @@ sap.ui.define([
 				}
 
 				return (
-					oElement instanceof ClassDeclaration
+					oElement.isA(mViewExtension.className)
 					&& sElementName === sExtensionName
 				);
 			});
@@ -89,7 +84,7 @@ sap.ui.define([
 			}
 		}
 
-		var aRelevantOverlays = [];
+		const aRelevantOverlays = [];
 
 		aExtensionOverlays.forEach(function(oElementOverlay) {
 			OverlayUtil.iterateOverlayElementTree(oElementOverlay, function(oElementOverlay) {
@@ -101,12 +96,10 @@ sap.ui.define([
 	}
 
 	return function(aElementOverlays, oComponent) {
-		var aResult = [];
-
-		var aRelevantOverlays = aElementOverlays.slice(0);
+		let aRelevantOverlays = aElementOverlays.slice(0);
 
 		if (isFioriElementsApp(oComponent)) {
-			var aExtensionList = getFioriElementsExtensions(oComponent);
+			const aExtensionList = getFioriElementsExtensions(oComponent);
 
 			aRelevantOverlays = (
 				aExtensionList.length
@@ -115,13 +108,11 @@ sap.ui.define([
 			);
 		}
 
-		aResult = aRelevantOverlays.filter(function(oElementOverlay) {
+		return aRelevantOverlays.filter(function(oElementOverlay) {
 			return (
 				!oElementOverlay.getDesignTimeMetadata().markedAsNotAdaptable()
 				&& !hasStableId(oElementOverlay)
 			);
 		});
-
-		return aResult;
 	};
 });

package/src/sap/ui/rta/util/validateText.js

@@ -12,8 +12,8 @@ sap.ui.define([
 ) {
 	"use strict";
 
-	var sEmptyTextKey = "\xa0";
-	var oValidators = {
+	const sEmptyTextKey = "\xa0";
+	const oValidators = {
 		noEmptyText: {
 			validatorFunction(sNewText) {
 				return sNewText !== sEmptyTextKey;
@@ -26,8 +26,8 @@ sap.ui.define([
 		if (sOldText === sNewText) {
 			throw Error("sameTextError");
 		}
-		var oBindingParserResult;
-		var bError;
+		let oBindingParserResult;
+		let bError;
 
 		try {
 			oBindingParserResult = BindingParser.complexParser(sNewText, undefined, true);
@@ -42,11 +42,11 @@ sap.ui.define([
 
 	return function(sNewText, sOldText, oAction) {
 		checkPreconditionsAndThrowError(sNewText, sOldText);
-		var sErrorText;
-		var aValidators = oAction && oAction.validators || [];
+		let sErrorText;
+		const aValidators = oAction && oAction.validators || [];
 
 		aValidators.some(function(vValidator) {
-			var oValidator;
+			let oValidator;
 
 			if (
 				typeof vValidator === "string" && oValidators[vValidator]

package/src/sap/ui/rta/util/whatsNew/WhatsNew.js

@@ -12,7 +12,6 @@ sap.ui.define([
 	"sap/ui/fl/write/api/FeaturesAPI",
 	"sap/ui/model/json/JSONModel",
 	"sap/ui/model/resource/ResourceModel",
-	"sap/ui/rta/util/ReloadManager",
 	"sap/ui/rta/util/whatsNew/WhatsNewUtils"
 ], function(
 	mLibrary,
@@ -22,7 +21,6 @@ sap.ui.define([
 	FeaturesAPI,
 	JSONModel,
 	ResourceModel,
-	ReloadManager,
 	WhatsNewUtils
 ) {
 	"use strict";
@@ -33,7 +31,7 @@ sap.ui.define([
 	 * @class Constructor for a new sap.ui.rta.util.whatsNew.WhatsNew
 	 * @extends sap.ui.base.ManagedObject
 	 * @author SAP SE
-	 * @version 1.136.4
+	 * @version 1.139.0
 	 * @constructor
 	 * @since 1.129
 	 * @private
@@ -51,27 +49,44 @@ sap.ui.define([
 		}
 	});
 
+	function updateLearnMoreButton(oCarousel) {
+		const sActivePageId = oCarousel.getActivePage();
+		const aPages = oCarousel.getPages();
+		const oLearnMoreBtn = Element.getElementById("sapUiRtaWhatsNewDialog_LearnMore");
+
+		// During initialization of the carousel the ActivePageId is not set yet, so we default to the index of the first page
+		const iCurrentIndex = sActivePageId
+			? aPages.findIndex(function(oPage) {
+				return oPage.getId() === sActivePageId;
+			})
+			: 0;
+
+		if (this.aUnseenFeatures[iCurrentIndex] && this.aUnseenFeatures[iCurrentIndex].documentationUrls) {
+			oLearnMoreBtn.setEnabled(true);
+		} else {
+			oLearnMoreBtn.setEnabled(false);
+		}
+	}
+
 	WhatsNew.prototype.setDontShowAgainFeatureIds = async function(aDontShowAgainFeatureIds) {
 		this.setProperty("dontShowAgainFeatureIds", aDontShowAgainFeatureIds);
 		this.aUnseenFeatures = await WhatsNewUtils.getFilteredFeatures(aDontShowAgainFeatureIds, this.getLayer());
 	};
 
-	WhatsNew.prototype.initializeWhatsNewDialog = async function(aExcludeFeatureIds = []) {
-		const aDontShowAgainFeatureIds = await FeaturesAPI.getSeenFeatureIds({ layer: this.getLayer() });
+	WhatsNew.prototype.initializeWhatsNewDialog = async function(aDontShowAgainFeatureIds, aExcludeFeatureIds = []) {
 		await this.setDontShowAgainFeatureIds(aDontShowAgainFeatureIds);
 		if (this.aUnseenFeatures.length === 0 || this.getLayer() !== "CUSTOMER") {
 			return;
 		}
 		const oWhatsNewDialogModel = new JSONModel();
-		const aVisibleFeatures = this.aUnseenFeatures.filter((oFeature) => !aExcludeFeatureIds.includes(oFeature.featureId));
-		if (aVisibleFeatures.length === 0) {
+		this.aUnseenFeatures = this.aUnseenFeatures.filter((oFeature) => !aExcludeFeatureIds.includes(oFeature.featureId));
+		if (this.aUnseenFeatures.length === 0) {
 			return;
 		}
-		oWhatsNewDialogModel.setData({ featureCollection: aVisibleFeatures });
+		oWhatsNewDialogModel.setData({ featureCollection: this.aUnseenFeatures });
 		if (!this.oWhatsNewDialog)	{
 			await this.createWhatsNewDialog(oWhatsNewDialogModel);
 		}
-		ReloadManager.setDontShowWhatsNewAfterReload();
 		this.oWhatsNewDialog.open();
 	};
 
@@ -83,6 +98,16 @@ sap.ui.define([
 		});
 		this.oWhatsNewDialog.setModel(oRTAResourceModel, "i18n");
 		this.oWhatsNewDialog.setModel(oWhatsNewDialogModel, "whatsNewModel");
+
+		// Dynamically enable/disable the Learn More button based on the current carousel page
+		const oCarousel = Element.getElementById("sapWhatsNewDialogCarousel");
+		if (oCarousel) {
+			oCarousel.attachEvent("pageChanged", () => {
+				updateLearnMoreButton.call(this, oCarousel);
+			});
+			// Initial state
+			updateLearnMoreButton.call(this, oCarousel);
+		}
 	};
 
 	WhatsNew.prototype.closeWhatsNewDialog = function() {

package/src/sap/ui/rta/util/whatsNew/WhatsNewOverview.js

@@ -23,14 +23,10 @@ sap.ui.define([
 	let oWhatsNewOverviewDialog;
 	let aWhatsNewOverviewContent;
 
-	async function getWhatsNewOverviewContent() {
-		const aFeatures = await WhatsNewUtils.getFilteredFeatures([]);
-		return [...aFeatures].reverse();
-	}
-
 	WhatsNewOverview.openWhatsNewOverviewDialog = async function() {
 		const oWhatsNewDialogModel = new JSONModel();
-		aWhatsNewOverviewContent = await getWhatsNewOverviewContent();
+		// Get the unseen features from WhatsNewUtils; newest features are the first in the array
+		aWhatsNewOverviewContent = await WhatsNewUtils.getFilteredFeatures([]);
 		oWhatsNewDialogModel.setData({ featureCollection: aWhatsNewOverviewContent });
 		oWhatsNewDialogModel.setProperty("overviewActive", true);
 		if (!oWhatsNewOverviewDialog) {

package/src/sap/ui/rta/util/whatsNew/WhatsNewOverviewDialog.fragment.xml

@@ -61,6 +61,7 @@
 					<ToolbarSpacer />
 					<Button
 						id="sapUiRtaWhatsNewOverviewDialog_LearnMore"
+						enabled="{= ${whatsNewModel>documentationUrls} !== null}"
 						visible="{= ${whatsNewModel>overviewActive} === false}"
 						text="{i18n>BTN_LEARN_MORE_WHATS_NEW_DIALOG}"
 						press=".onLearnMorePress"/>