@oracle/oraclejet 14.0.5 → 14.1.0

package/dist/js/libs/oj/debug/ojlegend.js

@@ -475,6 +475,7 @@ var __oj_legend_section_metadata =
    * @property {string=} title The title of the legend section.
    * @property {"center"|"end"|"start"} [titleHalign="start"] The horizontal alignment of the section title. If the section is collapsible or nested, only start alignment is supported.
    * @property {Object=} titleStyle The CSS style object defining the style of the section title. The following style properties are supported: color, cursor, fontFamily, fontSize, fontStyle, fontWeight, textDecoration.
+   * @ojdeprecated {since: '14.1.0', description: 'This attribute is deprecated. Applications should use the expanded API on oj-legend instead', target: 'property', for: 'expanded' }
    * @ojsignature [{target: "Type", value: "K", for: "id"},
    *               {target: "Type", value: "Array.<oj.ojLegend.Item<K>>", for: "items", jsdocOverride: true},
    *               {target: "Type", value: "Array.<oj.ojLegend.Section<K>>", for: "sections", jsdocOverride: true},
@@ -504,6 +505,11 @@ var __oj_legend_section_metadata =
    * @property {Object=} svgStyle The inline style to apply to the legend item. The style class and inline style will override any other styling specified through the options. For tooltips and hover interactivity, it's recommended to also pass a representative color to the color attribute. Only SVG CSS style properties are supported.
    * @property {"image"|"line"|"lineWithMarker"|"marker"} [symbolType="marker"] The type of legend symbol to display.
    * @property {string} text The legend item text.
+   * @ojdeprecated [{since: '14.1.0', description: 'This is not recommended in the Redwood design system.', target: 'property', for: 'svgStyle'},
+   *                {since: '14.1.0', description: 'This is not recommended in the Redwood design system.', target: 'property', for: 'svgClassName'},
+   *                {since: '14.1.0', description: 'This is not recommended in the Redwood design system.', target: 'property', for: 'markerSvgClassName'},
+   *                {since: '14.1.0', description: 'This is not recommended in the Redwood design system.', target: 'property', for: 'markerSvgStyle'},
+   *                {since: '14.1.0', description: 'This api is deprecated. Use hidden-categories in oj-legend instead.', target: 'property', for: 'categoryVisibility'}]
    * @ojsignature [{target: "Type", value: "K", for: "id"},
    *               {target: "Type", value: "Partial<CSSStyleDeclaration>", for: "svgStyle", jsdocOverride: true},
    *               {target: "Type", value: "Partial<CSSStyleDeclaration>", for: "markerSvgStyle", jsdocOverride: true},
@@ -511,7 +517,6 @@ var __oj_legend_section_metadata =
    */
 
   // METHOD TYPEDEFS
-
   /**
    * @typedef {Object} oj.ojLegend.SectionContext
    * @ojtsignore
@@ -705,7 +710,7 @@ var __oj_legend_section_metadata =
    *               }
    *              ]
    *
-   * @ojpropertylayout {propertyGroup: "common", items: ["orientation", "halign", "valign", "hoverBehavior", "hoverBehaviorDelay", "style"]}
+   * @ojpropertylayout {propertyGroup: "common", items: ["orientation", "halign", "valign", "hoverBehavior", "style"]}
    * @ojpropertylayout {propertyGroup: "data", items: ["data"]}
    * @ojvbdefaultcolumns 2
    * @ojvbmincolumns 1
@@ -966,6 +971,7 @@ var __oj_legend_section_metadata =
        * @ojunits milliseconds
        * @ojmin 0
        * @default 200
+       * @ojdeprecated {since: '14.1.0', description: 'This is not recommended in the Redwood design system.'}
        *
        * @example <caption>Initialize the Legend with the <code class="prettyprint">hover-behavior-delay</code> attribute specified:</caption>
        * &lt;oj-legend hover-behavior-delay="150">&lt;/oj-legend>
@@ -1653,6 +1659,7 @@ var __oj_legend_section_metadata =
    * @instance
    * @type {string=}
    * @default ""
+   * @ojdeprecated {since: '14.1.0', description: 'This is not recommended in the Redwood design system.'}
    */
   /**
    * The inline style to apply to the legend item symbol. This inline style and <code> svg-class-name </code> will override any other styling specified through the options except for <code>marker-svg-style</code> and <code>marker-svg-class-name</code>.
@@ -1665,6 +1672,7 @@ var __oj_legend_section_metadata =
    * @instance
    * @type {Object=}
    * @ojsignature {target: "Type", value: "Partial<CSSStyleDeclaration>", jsdocOverride: true}
+   * @ojdeprecated {since: '14.1.0', description: 'This is not recommended in the Redwood design system.'}
    */
   /**
    * The CSS style class to apply to the marker of the legend item symbol. This style class and <code> marker-svg-style </code> will override any other styling specified for the marker. For tooltips and hover interactivity, it's recommended to also pass a representative color to the markerColor attribute.
@@ -1675,6 +1683,7 @@ var __oj_legend_section_metadata =
    * @instance
    * @type {string=}
    * @default ""
+   * @ojdeprecated {since: '14.1.0', description: 'This is not recommended in the Redwood design system.'}
    */
   /**
    * The inline style to apply to the marker of the legend item symbol. This inline style and <code> marker-svg-class-name </code> will override any other styling specified for the marker. For tooltips and hover interactivity, it's recommended to also pass a representative color to the markerColor attribute.
@@ -1686,6 +1695,7 @@ var __oj_legend_section_metadata =
    * @instance
    * @type {Object=}
    * @ojsignature {target: "Type", value: "Partial<CSSStyleDeclaration>", jsdocOverride: true}
+   * @ojdeprecated {since: '14.1.0', description: 'This is not recommended in the Redwood design system.'}
    */
   /**
    * The shape of the marker. Only applies if symbolType is "marker" or "lineWithMarker". Can take the name of a built-in shape or the SVG path commands for a custom shape. Does not apply if a custom image is specified.
@@ -1716,6 +1726,7 @@ var __oj_legend_section_metadata =
    * @ojvalue {string} "hidden" Legend item will have a hollow symbol.
    * @ojvalue {string} "visible" Legend item will be filled.
    * @default "visible"
+   * @ojdeprecated {since: '14.1.0', description: 'This attribute is deprecated. Use hidden-categories on oj-legend instead'}
    */
   /**
    *  Whether drilling is enabled on the legend item. Drillable objects will show a pointer cursor on hover and fire <code class="prettyprint">ojDrill</code> event on click. To enable drilling for all legend items at once, use the drilling attribute in the top level.

package/dist/js/libs/oj/debug/ojlogger.js

@@ -25,22 +25,33 @@ define(['exports'], function (exports) { 'use strict';
    *
    * <p>If the logging options are changed at a later point, the Logger will use the modified options for the subsequent log operations.
    *
+   * <p>The logging level can be overridden via sessionStorage.setItem() call for the current browser session.
+   * Use 'ojet.logLevel' as the key with one of the following values: 'none' (least verbose), 'error', 'warning', 'info', 'log' (most verbose).
+   * Set the value in the browser console and refresh the browser in order for the value to take effect.
+   * </p>
+   * <h3> Session storage usage : </h3>
+   * <pre class="prettyprint">
+   * <code>
+   * // override logging level for the current session
+   * sessionStorage.setItem('ojet.logLevel', 'log');
+   * </code></pre>
+   *
    * <p>All the logging methods support string formatting, accept variable number of arguments and accept a function as a parameter.
    * When a callback function is specified as a parameter the function will be called if the log level is sufficient.
    *
-   * <h3> Usage : </h3>
+   * <h3> Logger usage : </h3>
    * <pre class="prettyprint">
    * <code>
    * //optional calls, see defaults
-   * oj.Logger.option("level",  oj.Logger.LEVEL_INFO);
-   * oj.Logger.option("writer",  customWriter);  //an object that implements the following methods: log(), info(), warn(), error()
+   * Logger.option("level",  Logger.LEVEL_INFO);
+   * Logger.option("writer",  customWriter);  //an object that implements the following methods: log(), info(), warn(), error()
    *
    * // logging a message
-   * oj.Logger.info("My log level is %d", oj.Logger.option("level"));  // string formatting
-   * oj.Logger.warn("Beware of bugs", "in the above code");            // multiple parameters
+   * Logger.info("My log level is %d", Logger.option("level"));  // string formatting
+   * Logger.warn("Beware of bugs", "in the above code");            // multiple parameters
    *
    * // using a callback function as a parameter
-   * oj.Logger.info(function(){
+   * Logger.info(function(){
    *    var foo = "This ";
    *    var bar = "is ";
    *    var zing = "a function";
@@ -107,6 +118,47 @@ define(['exports'], function (exports) { 'use strict';
   Logger._defaultOptions = { level: Logger.LEVEL_ERROR, writer: null };
   Logger._options = Logger._defaultOptions;
 
+  /*
+   * Helper method that retrieves and parses session storage setting
+   */
+  const _getSessionStorage = () => {
+    const sessionValue =
+      typeof window !== 'undefined' && window.sessionStorage !== undefined
+        ? sessionStorage.getItem('ojet.logLevel')
+        : undefined;
+    let logLevel;
+    switch (sessionValue) {
+      case 'none':
+        logLevel = Logger.LEVEL_NONE;
+        break;
+      case 'error':
+        logLevel = Logger.LEVEL_ERROR;
+        break;
+      case 'warning':
+        logLevel = Logger.LEVEL_WARN;
+        break;
+      case 'info':
+        logLevel = Logger.LEVEL_INFO;
+        break;
+      case 'log':
+        logLevel = Logger.LEVEL_LOG;
+        break;
+      default:
+        logLevel = undefined;
+    }
+    return logLevel;
+  };
+
+  /* SessionStorage setting */
+  const _sessionStorage = _getSessionStorage();
+
+  /*
+   * Helper method - returns current log level from session storage or options
+   */
+  const _getLogLevel = () => {
+    return _sessionStorage || Logger._options.level;
+  };
+
   /* public members*/
   /**
    * Writes an error message.
@@ -210,12 +262,23 @@ define(['exports'], function (exports) { 'use strict';
     if (arguments.length === 0) {
       keys = Object.keys(Logger._options);
       for (i = 0; i < keys.length; i++) {
-        ret[keys[i]] = Logger._options[keys[i]];
+        ret[keys[i]] = key === 'level' ? _getLogLevel() : Logger._options[keys[i]];
       }
       return ret;
     }
     if (typeof key === 'string' && value === undefined) {
-      return Logger._options[key] === undefined ? null : Logger._options[key];
+      let optValue;
+      switch (key) {
+        case 'level':
+          optValue = _getLogLevel();
+          break;
+        case 'writer':
+          optValue = Logger._options.writer;
+          break;
+        default:
+          optValue = null;
+      }
+      return optValue;
     }
 
     // setters

package/dist/js/libs/oj/debug/ojmenu.js

@@ -5,7 +5,7 @@
  * as shown at https://oss.oracle.com/licenses/upl/
  * @ignore
  */
-define(['jquery', 'ojs/ojjquery-hammer', 'ojs/ojpopupcore', 'ojs/ojoption', 'ojs/ojcore-base', 'hammerjs', 'ojs/ojcontext', 'ojs/ojthemeutils', 'ojs/ojcomponentcore', 'ojs/ojanimation', 'ojs/ojlogger', 'ojs/ojconfig', 'ojs/ojcustomelement-utils'], function ($, ojjqueryHammer, ojpopupcore, ojoption, oj, Hammer, Context, ThemeUtils, Components, AnimationUtils, Logger, Config, ojcustomelementUtils) { 'use strict';
+define(['jquery', 'ojs/ojjquery-hammer', 'ojs/ojpopupcore', 'ojs/ojoption', 'ojs/ojcore-base', 'hammerjs', 'ojs/ojcontext', 'ojs/ojthemeutils', 'ojs/ojcomponentcore', 'ojs/ojanimation', 'ojs/ojlogger', 'ojs/ojconfig', 'ojs/ojcustomelement-utils', 'ojs/ojcustomelement-registry'], function ($, ojjqueryHammer, ojpopupcore, ojoption, oj, Hammer, Context, ThemeUtils, Components, AnimationUtils, Logger, Config, ojcustomelementUtils, ojcustomelementRegistry) { 'use strict';
 
   $ = $ && Object.prototype.hasOwnProperty.call($, 'default') ? $['default'] : $;
   oj = oj && Object.prototype.hasOwnProperty.call(oj, 'default') ? oj['default'] : oj;
@@ -4319,7 +4319,7 @@ define(['jquery', 'ojs/ojjquery-hammer', 'ojs/ojpopupcore', 'ojs/ojoption', 'ojs
       ojMenu: {
         openOptions: Components.createDynamicPropertyGetter(function (context) {
           var position;
-          if (ojcustomelementUtils.CustomElementUtils.isElementRegistered(context.element.tagName)) {
+          if (ojcustomelementRegistry.isElementRegistered(context.element.tagName)) {
             if (context.containers.indexOf('ojMenu') >= 0) {
               position = {
                 my: { horizontal: 'start', vertical: 'top' },

package/dist/js/libs/oj/debug/ojmessage.js

@@ -1209,6 +1209,12 @@ define(['require', 'exports', 'ojs/ojcore', 'ojs/ojknockout', 'ojs/ojbutton', 'o
     }
 
     var severity = this._computeSeverity();
+
+    // If severity is none, return undefined as we do not want to show the category
+    if (severity === 'none') {
+      return undefined;
+    }
+
     var translations = this._properties.translations;
 
     var translatedCategory =
@@ -1934,6 +1940,9 @@ var __oj_message_metadata =
             "info": {
               "type": "string"
             },
+            "none": {
+              "type": "string"
+            },
             "warning": {
               "type": "string"
             }

package/dist/js/libs/oj/debug/ojmessages.js

@@ -1128,17 +1128,23 @@ define(['ojs/ojcore-base', 'jquery', 'knockout', 'ojs/ojanimation', 'ojs/ojconte
   };
 
   MessagesViewModel.prototype._updateLiveRegionAndContainer = function (message) {
-    var translations = Translations.getComponentTranslations('oj-ojMessage').categories;
+    const translations = Translations.getComponentTranslations('oj-ojMessage').categories;
 
     // oj.Message has 'fatal' severity which is no different from 'error', oj-message does not support
     //  'fatal' for this reason. Map 'fatal' to 'error' just to be compatible with cases where the
     //  message stream could come from existing oj.Message sources.
-    var severity = message.severity === 'fatal' ? 'error' : message.severity;
-    var category = !message.category ? translations[severity] : message.category;
-    var options = { category: category, summary: message.summary, detail: message.detail };
-
-    var liveRegion = this._getLiveRegion();
-    var text = this._getTranslationsDefault('ariaLiveRegion.newMessage', options);
+    const severity = (message.severity === 'fatal' ? 'error' : message.severity) || 'none';
+    const category = !message.category ? translations[severity] : message.category;
+    // check if the message has detail text or not (as it is an optional property)
+    // and determine which translation resource needs to be used.
+    const options = {
+      category: category,
+      summary: message.summary,
+      detail: message.detail || this._getTranslationsDefault('ariaLiveRegion.noDetail')
+    };
+
+    const liveRegion = this._getLiveRegion();
+    const text = this._getTranslationsDefault('ariaLiveRegion.newMessage', options);
     liveRegion.announce(text);
 
     this._refresh(); // re-evaluate the position as the overlay size can change.
@@ -2045,6 +2051,9 @@ var __oj_messages_metadata =
             },
             "newMessage": {
               "type": "string"
+            },
+            "noDetail": {
+              "type": "string"
             }
           }
         },

package/dist/js/libs/oj/debug/ojnavigationlist.js

@@ -1396,6 +1396,13 @@ var __oj_tab_bar_metadata =
           return;
         }
         if ($target.hasClass(this.getNavListRemoveIcon()) && this.isTabBar()) {
+          // click on the 'X' should not change selection, but it should still change currentItem
+          this.HandleClickActive(item, event);
+          var contentElement = this.getItemContentElement(item);
+          if (contentElement.length > 0) {
+            // make sure it gets focus otherwise focus goes to the body as 'X' is no longer in DOM
+            contentElement[0].focus();
+          }
           this._handleRemove(event, item);
           return;
         }

package/dist/js/libs/oj/debug/ojnbox-toolkit.js

@@ -4494,8 +4494,10 @@ define(['exports', 'ojs/ojdvt-toolkit', 'ojs/ojtranslation'], function (exports,
                 : ' oj-avatar-bg-' + bgColor + ' oj-nbox-node-initials-' + bgColor)
           );
           if (hasSource) {
+            // for ios use a smaller cropped image JET-53563
+            var isIos = dvt.Agent.os === 'ios';
             var iconContainer = new dvt.Container(nbox.getCtx());
-            var iconBackgroundSize = icon['backgroundSize'];
+            var iconBackgroundSize = isIos ? icon['backgroundIosSize'] : icon['backgroundSize'];
             var backgroundMarker = new dvt.ImageMarker(
               nbox.getCtx(),
               iconX,
@@ -4510,7 +4512,11 @@ define(['exports', 'ojs/ojdvt-toolkit', 'ojs/ojtranslation'], function (exports,
             );
             iconContainer.addChild(iconMarker);
             iconContainer.addChild(backgroundMarker);
-            backgroundMarker.setClassName('oj-nbox-node-initials-background-pattern');
+            backgroundMarker.setClassName(
+              isIos
+                ? 'oj-nbox-node-initials-background-pattern oj-nbox-ios'
+                : 'oj-nbox-node-initials-background-pattern'
+            );
             iconMarker = iconContainer;
           }
 

package/dist/js/libs/oj/debug/ojnbox.js

@@ -2101,6 +2101,10 @@ var __oj_n_box_node_metadata =
         path: 'styleDefaults/nodeDefaults/iconDefaults/backgroundSize',
         property: 'width'
       };
+      styleClasses['oj-nbox-node-initials-background oj-nbox-ios'] = {
+        path: 'styleDefaults/nodeDefaults/iconDefaults/backgroundIosSize',
+        property: 'width'
+      };
       styleClasses['oj-nbox-node-categorylabel'] = {
         path: 'styleDefaults/_categoryNodeDefaults/labelStyle',
         property: 'TEXT'

package/dist/js/libs/oj/debug/ojpopup.js

@@ -182,8 +182,8 @@ define(['ojs/ojpopupcore', 'ojs/ojcore-base', 'jquery', 'ojs/ojcontext', 'ojs/oj
      *   <li>"alertdialog" defines type of dialog that contains an alert message, where initial focus
      *       goes to an element within the dialog.</li>
      * </ul>
-     * The popup also adds the <code class="prettyprint">aria-describedby="popup-id"</code> attribute
-     * to the assocaited launcher while the popup is open.
+     * The non-modal (modeless) popup also adds the <code class="prettyprint">aria-describedby="popup-id"</code>
+     * attribute to the assocaited launcher while the popup is open.
      * </p>
      *
      * On platforms that support voice over mode (VO), the popup injects anchor tags "skip links"
@@ -192,12 +192,12 @@ define(['ojs/ojpopupcore', 'ojs/ojcore-base', 'jquery', 'ojs/ojcontext', 'ojs/oj
      *  <ul>
      *    <li>A close link, {@link oj.ojPopup#translations.ariaCloseSkipLink}, is injected as a
      *        sibling to the popup's content. Activation of this link will close the popup.</li>
-     *    <li>For cases where the popup doesn't steal focus when it's open, a content navigation
+     *    <li>For cases where the non-modal popup doesn't steal focus when it's open, a content navigation
      *        skip link, {@link oj.ojPopup#translations.ariaFocusSkipLink}, is injected as a
      *        sibling to the launcher (first required argument of the open method). If the launcher
      *        selector targets a sub-element of the launcher, the skip link could be injected under
      *        the launcher, which can be problematic for oj-button as skip link activation will
-     *        also activate the associated launcher.</li>
+     *        also activate the associated launcher. The focus skip link is not injected for modal popups.</li>
      *  </ul>
      *
      * <p>One point often overlooked is making the gestures that launch a popup accessible.
@@ -1043,7 +1043,9 @@ define(['ojs/ojpopupcore', 'ojs/ojcore-base', 'jquery', 'ojs/ojcontext', 'ojs/oj
         this._setPosition(position);
 
         this._setAutoDismiss(options.autoDismiss);
-        this._addDescribedBy();
+        if (options.modality !== 'modal') {
+          this._addDescribedBy();
+        }
 
         if (!this._IsCustomElement() || !element[0].hasAttribute('role')) {
           element.attr('role', options.role);
@@ -2239,38 +2241,46 @@ define(['ojs/ojpopupcore', 'ojs/ojcore-base', 'jquery', 'ojs/ojcontext', 'ojs/oj
        * @private
        */
       _initVoiceOverAssist: function () {
+        var isModal = this.options.modality === 'modal';
         var isVOSupported =
           oj.AgentUtils.getAgentInfo().os === oj.AgentUtils.OS.IOS ||
           oj.AgentUtils.getAgentInfo().os === oj.AgentUtils.OS.ANDROID;
-        var liveRegion = this._liveRegion;
-        if (!liveRegion) {
-          liveRegion = new ojpopupcore.PopupLiveRegion();
-          this._liveRegion = liveRegion;
-        }
-
         var message;
-        var initialFocus = this._deriveInitialFocus();
-        if (isVOSupported) {
-          message = this.getTranslatedString(
-            initialFocus === 'none'
-              ? 'ariaLiveRegionInitialFocusNoneTouch'
-              : 'ariaLiveRegionInitialFocusFirstFocusableTouch'
-          );
-        } else {
-          message = this.getTranslatedString(
-            initialFocus === 'none'
-              ? 'ariaLiveRegionInitialFocusNone'
-              : 'ariaLiveRegionInitialFocusFirstFocusable'
-          );
+
+        if (!isModal) {
+          // for non-modal popups install the live region for keybard/VO navigation
+          var liveRegion = this._liveRegion;
+          if (!liveRegion) {
+            liveRegion = new ojpopupcore.PopupLiveRegion();
+            this._liveRegion = liveRegion;
+          }
+          var initialFocus = this._deriveInitialFocus();
+          if (isVOSupported) {
+            message = this.getTranslatedString(
+              initialFocus === 'none'
+                ? 'ariaLiveRegionInitialFocusNoneTouch'
+                : 'ariaLiveRegionInitialFocusFirstFocusableTouch'
+            );
+          } else {
+            message = this.getTranslatedString(
+              initialFocus === 'none'
+                ? 'ariaLiveRegionInitialFocusNone'
+                : 'ariaLiveRegionInitialFocusFirstFocusable'
+            );
+          }
+          liveRegion.announce(message);
         }
-        liveRegion.announce(message);
 
         if (isVOSupported) {
-          var focusSkipLinkId = this._getSubId('focusSkipLink');
-          var launcher = this._launcher;
-          var callback = this._intialFocus.bind(this, true);
-          message = this.getTranslatedString('ariaFocusSkipLink');
-          this._focusSkipLink = new ojpopupcore.PopupSkipLink(launcher, message, callback, focusSkipLinkId);
+          var callback;
+
+          if (!isModal) {
+            var focusSkipLinkId = this._getSubId('focusSkipLink');
+            var launcher = this._launcher;
+            callback = this._intialFocus.bind(this, true);
+            message = this.getTranslatedString('ariaFocusSkipLink');
+            this._focusSkipLink = new ojpopupcore.PopupSkipLink(launcher, message, callback, focusSkipLinkId);
+          }
 
           var content = this._content;
           var closeSkipLinkId = this._getSubId('closeSkipLink');
@@ -2286,8 +2296,10 @@ define(['ojs/ojpopupcore', 'ojs/ojcore-base', 'jquery', 'ojs/ojcontext', 'ojs/oj
        */
       _destroyVoiceOverAssist: function () {
         var liveRegion = this._liveRegion;
-        liveRegion.destroy();
-        delete this._liveRegion;
+        if (liveRegion) {
+          liveRegion.destroy();
+          delete this._liveRegion;
+        }
 
         var focusSkipLink = this._focusSkipLink;
         if (focusSkipLink) {

package/dist/js/libs/oj/debug/ojpopupcore.js

@@ -1495,9 +1495,9 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
 
     ZOrderUtils.preOrderVisit(layer, ZOrderUtils._closeDescendantPopupsCallback);
 
+    ZOrderUtils._removeOverlayFromAncestorLayer(layer);
     ZOrderUtils._resetAriaHiddenOnBackround(layer);
     ZOrderUtils._restoreBodyOverflow();
-    ZOrderUtils._removeOverlayFromAncestorLayer(layer);
 
     layer.removeData(ZOrderUtils._EVENTS_DATA);
     layer.removeData(ZOrderUtils._MODALITY_DATA);
@@ -1551,8 +1551,11 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
   };
 
   ZOrderUtils._restoreBodyOverflow = function () {
-    const body = document.body;
-    body.classList.remove('oj-component-modal-open');
+    // if there are no more modals left, enable scrolling on body
+    if (!ZOrderUtils.hasModalDialogOpen()) {
+      const body = document.body;
+      body.classList.remove('oj-component-modal-open');
+    }
   };
 
   ZOrderUtils._removeFocusWithinFromOverlayedContent = function () {
@@ -3406,6 +3409,11 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
           }
         });
 
+        // JET-56620: normalize position left/top to integer values to avoid unnecessary
+        // repositioning caused by potential floating number oscillations
+        position.left = Math.round(position.left);
+        position.top = Math.round(position.top);
+
         if (options.using) {
           // Adds feedback as second argument to using callback, if present
           using = function (props) {