@oracle/oraclejet 16.0.6 → 16.1.0

package/dist/js/libs/oj/debug_esm/ojgantt-toolkit.js

@@ -2071,9 +2071,21 @@ class DvtGanttRowLabelContent {
         ? this._addExpandCollapseButton(this._content, rowObj)
         : this._content;
       this._gantt.getEventManager().associate(this._contentDisplayable, this, true);
+
+      this._contentDisplayableContainer = this._contentDisplayable;
+      // Hierarchical labels are already wrapped with a container by _addExpandCollapseButton
+      // Non hierarchical labels should also be wrapped with a container so that a role "row" can be set on it:
+      if (!this._gantt.isRowsHierarchical()) {
+        this._contentDisplayableContainer = new Container(this._gantt.getCtx());
+        this._contentDisplayableContainer.addChild(this._contentDisplayable);
+      }
+      // Note the "true" flag; we don't want deferred aria creation for the role because the parent with role "rowheader"
+      // and the ancestor role "grid" requires role "row" children to be present on initial render. Otherwise an axe-core violation is triggered.
+      // Performance is not really a concern because there are typically far more tasks than rows.
+      this._contentDisplayableContainer.setAriaRole('row', true);
     }
-    if (!this._contentDisplayable.getParent()) {
-      this._rowAxis.addLabelContent(this._contentDisplayable);
+    if (!this._contentDisplayableContainer.getParent()) {
+      this._rowAxis.addLabelContent(this._contentDisplayableContainer);
     }
 
     // TODO consider tweaking the logic to avoid this step when an explicit width is set on the row axis for potential performance improvements, especially for custom content
@@ -2103,10 +2115,7 @@ class DvtGanttRowLabelContent {
     this._height = contentDimensions.h;
 
     this._contentDisplayable.setAriaProperty('label', this.getAriaLabel());
-    // Note the "true" flag; we don't want deferred aria creation for the role because the parent with role "rowheader"
-    // and the ancestor role "grid" requires role "row" children to be present on initial render. Otherwise an axe-core violation is triggered.
-    // Performance is not really a concern because there are typically far more tasks than rows.
-    this._contentDisplayable.setAriaRole('row', true);
+
     // DvtOutputText by default sets aria-hidden to true. We need to ensure this is unset for accessibility.
     this._contentDisplayable.setAriaProperty('hidden', null);
   }
@@ -2352,6 +2361,14 @@ class DvtGanttRowLabelContent {
     return this._contentDisplayable;
   }
 
+  /**
+   * Gets the content displayable container
+   * @return {dvt.Container} The content displayable container
+   */
+  getDisplayableContainer() {
+    return this._contentDisplayableContainer;
+  }
+
   /**
    * Gets the content background
    * @return {dvt.Displayable} The content displayable
@@ -6302,6 +6319,25 @@ class DvtGanttKeyboardHandler extends TimeComponentKeyboardHandler {
     return retVal;
   }
 
+  /**
+   * @override
+   */
+  isSelectionEvent(event) {
+    var currentNavigable = this._eventManager.getFocus();
+    // JET-62641: only tasks are selectable,
+    // and rowlabel <-> task navigation should not be considered as selection events
+    return (
+      super.isSelectionEvent(event) &&
+      currentNavigable?.nodeType === 'task' &&
+      !(
+        this._gantt.isRowAxisEnabled() &&
+        event.altKey &&
+        (event.keyCode === KeyboardEvent.LEFT_ARROW ||
+          event.keyCode === KeyboardEvent.RIGHT_ARROW)
+      )
+    );
+  }
+
   /**
    * Calculate the distance (basically time duration in millis) between predecessor and successor
    * @param {DvtGanttDependencyNode} dep
@@ -11563,13 +11599,17 @@ class DvtGanttRowNode extends Container {
         labelDisplayable.setTextString(labelDisplayable.getUntruncatedTextString());
 
       // Truncate label to fit
-      TextUtils.fitText(
+      var container = labelContent.getDisplayableContainer();
+      var rendered = TextUtils.fitText(
         labelDisplayable,
         rowAxis.getWidth() - totalPadding,
         height,
-        labelDisplayable.getParent(),
+        container,
         1
       );
+      if (!rendered && container) {
+        container.removeFromParent();
+      }
 
       // Always start align labels
       if (isRTL) {
@@ -14499,7 +14539,7 @@ class DvtGanttAnimationManager {
     var context = this._gantt.getCtx();
 
     // Ensure clean slate
-    this._gantt.StopAnimation();
+    this._gantt.StopAnimation(true);
 
     this._animationMode = 'none';
 
@@ -14594,7 +14634,7 @@ class DvtGanttAnimationManager {
     var subPlayables, context;
 
     // Stop any unfinished animations
-    this._gantt.StopAnimation();
+    this._gantt.StopAnimation(true);
 
     context = this._gantt.getCtx();
 
@@ -16488,7 +16528,7 @@ class DvtGanttRowAxis extends Container {
   adjustPosition() {
     var isRTL = Agent.isRightToLeft(this._gantt.getCtx());
     var width = this._width;
-    var height = this._gantt.getCanvasSize() - this._gantt.getAxesHeight();
+    var height = this._gantt.getDatabodyHeight();
     var componentStartPadding = DvtGanttStyleUtils.getComponentPaddingStart(
       this._gantt.getOptions()
     );
@@ -16643,10 +16683,7 @@ class DvtGanttReferenceObjects extends Container {
     }
 
     this._y1 = y1 == null ? this._gantt.getDatabodyStart() : y1;
-    this._y2 =
-      y2 == null
-        ? this._gantt.getDatabodyStart() + this._gantt._canvasSize - this._gantt.getAxesHeight()
-        : y2;
+    this._y2 = y2 == null ? this._gantt.getDatabodyStart() + this._gantt.getDatabodyHeight() : y2;
 
     this._ref = new Path(this._gantt.getCtx(), this._calcCmds(this._y1, this._y2, 0));
     if (this._type === 'line' || this._type === 'timeCursor') {
@@ -16943,7 +16980,7 @@ const DvtGanttRenderer = {
     }
     if (gantt.isContentDirScrollbarOn()) {
       availSpaceWidth = gantt.Width - scrollbarPadding;
-      availSpaceHeight = gantt.getCanvasSize() - gantt.getAxesHeight();
+      availSpaceHeight = gantt.getDatabodyHeight();
       var scrollbarXOffset = Agent.isRightToLeft(gantt.getCtx())
         ? gantt.getScrollbarPadding()
         : 0;
@@ -16991,7 +17028,8 @@ const DvtGanttRenderer = {
       var bottomOffset = 0;
       if (gantt.getAxisPosition() === 'bottom') bottomOffset = gantt.getAxesHeight();
       gantt.contentDirScrollbar.setViewportRange(
-        databody.getTranslateY() - (gantt.getCanvasSize() - databodyStart - bottomOffset),
+        databody.getTranslateY() -
+          (Math.max(0, gantt.getCanvasSize() - bottomOffset) - databodyStart),
         databody.getTranslateY()
       );
     }
@@ -17495,7 +17533,7 @@ const DvtGanttRenderer = {
         gantt.getStartXOffset(),
         gantt.getStartYOffset() + axesHeight * (gantt.getAxisPosition() === 'top'),
         gantt.getCanvasLength(),
-        gantt.getCanvasSize() - axesHeight
+        gantt.getDatabodyHeight()
       ),
       gantt.EventManager,
       null
@@ -17627,7 +17665,7 @@ const DvtGanttRenderer = {
             pos,
             gantt.getDatabodyStart(),
             pos,
-            gantt.getDatabodyStart() + gantt._canvasSize - gantt.getAxesHeight()
+            gantt.getDatabodyStart() + gantt.getDatabodyHeight()
           );
           gridLine.setPixelHinting(true);
           gridLine.setMouseEnabled(false);
@@ -18307,7 +18345,7 @@ class Gantt extends TimeComponent {
    * @return {Object} An object with yMin and yMax properties
    */
   getViewportYBounds(translateY) {
-    var viewportHeight = this._backgroundHeight - this.getAxesHeight();
+    var viewportHeight = Math.max(0, this._backgroundHeight - this.getAxesHeight());
     var yMin = 0;
     if (translateY != null) {
       yMin = this._translateYToScrollPositionY(translateY);
@@ -18689,10 +18727,12 @@ class Gantt extends TimeComponent {
     this._backgroundHeight = this.Height;
 
     if (this.isTimeDirScrollbarOn())
-      this._backgroundHeight =
+      this._backgroundHeight = Math.max(
+        0,
         this._backgroundHeight -
-        CSSStyle.toNumber(this.timeDirScrollbarStyles.getHeight()) -
-        3 * scrollbarPadding;
+          CSSStyle.toNumber(this.timeDirScrollbarStyles.getHeight()) -
+          3 * scrollbarPadding
+      );
     if (this.isContentDirScrollbarOn()) {
       var widthOffset =
         3 * scrollbarPadding + CSSStyle.toNumber(this.contentDirScrollbarStyles.getWidth());
@@ -18713,7 +18753,7 @@ class Gantt extends TimeComponent {
 
     // The size of the canvas viewport
     this._canvasLength = this._backgroundWidth - doubleBorderWidth;
-    this._canvasSize = this._backgroundHeight - doubleBorderWidth;
+    this._canvasSize = Math.max(0, this._backgroundHeight - doubleBorderWidth);
   }
 
   /**
@@ -18951,7 +18991,7 @@ class Gantt extends TimeComponent {
    * @return {number} the databody height
    */
   getDatabodyHeight() {
-    return this.getCanvasSize() - this.getAxesHeight();
+    return Math.max(0, this.getCanvasSize() - this.getAxesHeight());
   }
 
   /**
@@ -19331,7 +19371,7 @@ class Gantt extends TimeComponent {
       this.getStartXOffset() - this.getTimeZoomCanvas().getTranslateX(),
       this._databodyStart - this._databody.getTranslateY(),
       this._canvasLength,
-      this._canvasSize - this.getAxesHeight()
+      this.getDatabodyHeight()
     );
   }
 

package/dist/js/libs/oj/debug_esm/ojgauge-toolkit.js

@@ -5,7 +5,7 @@
  * as shown at https://oss.oracle.com/licenses/upl/
  * @ignore
  */
-import { Rectangle, ResourceUtils, OutputText, TextUtils, MultilineText, Automation, EventManager, TouchEvent, KeyboardEvent, KeyboardHandler, Agent, EventFactory, BaseComponent, Rect, Container, AriaUtils, SelectionEffectUtils, CustomDatatipPeer, SimpleObjPeer, CSSStyle, BlackBoxAnimationHandler, CustomAnimation, Animator, ParallelPlayable, Displayable, BaseComponentDefaults, JsonUtils, ColorUtils, Cache, Polygon, Circle, Path, DisplayableUtils, PathUtils, PolygonUtils, LinearGradientFill, Matrix, Use, ImageMarker, ImageLoader, ClipPath, Math as Math$1, Line, Stroke, Shadow, Easing } from 'ojs/ojdvt-toolkit';
+import { Rectangle, ResourceUtils, OutputText, TextUtils, MultilineText, Automation, EventManager, TouchEvent, KeyboardEvent, KeyboardHandler, Agent, EventFactory, BaseComponent, Rect, Container, SelectionEffectUtils, CustomDatatipPeer, SimpleObjPeer, CSSStyle, BlackBoxAnimationHandler, CustomAnimation, Animator, ParallelPlayable, AriaUtils, Displayable, BaseComponentDefaults, JsonUtils, ColorUtils, Cache, Polygon, Circle, Path, DisplayableUtils, PathUtils, PolygonUtils, LinearGradientFill, Matrix, Use, ImageMarker, ImageLoader, ClipPath, Math as Math$1, Line, Stroke, Shadow, Easing } from 'ojs/ojdvt-toolkit';
 import { LinearScaleAxisValueFormatter, DataAxisInfoMixin, BaseAxisInfo } from 'ojs/ojdvt-axis';
 
 /**
@@ -934,31 +934,7 @@ class DvtGauge extends BaseComponent {
     this.UpdateAriaAttributes();
 
     if (!this._bStaticRendering && !this.Options['readOnly']) {
-      container.setAriaRole('slider');
-      container.setAriaProperty('valuemin', this.Options['min']);
-      container.setAriaProperty('valuemax', this.Options['max']);
-      var value = DvtGaugeRenderer.getFormattedMetricLabel(this.Options['value'], this);
-      container.setAriaProperty('valuenow', value);
-      container.setAriaProperty('valuetext', value);
-      var tooltip = DvtGaugeRenderer.getTooltipString(this);
-
-      if (Agent.isTouchDevice()) {
-        this._container.setAriaProperty('live', 'assertive');
-        if (value !== tooltip) {
-          tooltip = value + AriaUtils.ARIA_LABEL_DESC_DELIMITER + tooltip;
-        }
-      }
-      if (value !== tooltip) {
-        container.setAriaProperty('label', tooltip);
-      }
-
-      // Generate a unique id to force screen readers to update for each render (and potential value change)
-      this._renderCount = this._renderCount != null ? this._renderCount + 1 : 0;
-      var ariaId = this.getId() + '_' + this._renderCount;
-      container.setId(ariaId);
-
-      // Set as the active element that's read by the screen reader.
-      this.getCtx().setActiveElement(container);
+      this.setAriaAttributes();
     }
 
     // . To support ADF action attribute
@@ -972,6 +948,23 @@ class DvtGauge extends BaseComponent {
     }
   }
 
+  setAriaAttributes() {
+    var parentContainer = this.getCtx().getContainer();
+    parentContainer.setAttribute('role', 'slider');
+    parentContainer.setAttribute('aria-valuemin', this.Options['min']);
+    parentContainer.setAttribute('aria-valuemax', this.Options['max']);
+    parentContainer.setAttribute('aria-valuenow', this.Options['value']);
+
+    var valueText = DvtGaugeRenderer.getFormattedMetricLabel(this.Options['value'], this);
+    if (Number(valueText) === this.Options['value']) valueText = this.Options['value'].toString();
+
+    parentContainer.setAttribute('aria-valuetext', valueText);
+
+    if (this.Options['disabled']) {
+      parentContainer.setAttribute('aria-disabled', true);
+    }
+  }
+
   /**
    * Creates and returns a logical object for this gauge.
    * @return {dvt.SimpleObjPeer}
@@ -1151,21 +1144,6 @@ class DvtGauge extends BaseComponent {
    */
   renderUpdate() {
     this.render();
-    this.UpdateAriaLiveValue(this._container);
-  }
-
-  /**
-   * Helper function for updating the live values that are read out while editing for accessibility
-   * @param {dvt.Container} container The slider container we are using.
-   * @param {number} value The value of the gauge
-   * @protected
-   */
-  UpdateAriaLiveValue(container, value) {
-    value = DvtGaugeRenderer.getFormattedMetricLabel(value ? value : this.Options['value'], this);
-    container.setAriaProperty('valuenow', value);
-    container.setAriaProperty('valuetext', value);
-
-    if (Agent.isTouchDevice()) container.setAriaProperty('label', value);
   }
 
   /**
@@ -1350,25 +1328,19 @@ class DvtGauge extends BaseComponent {
       var tooltip = DvtGaugeRenderer.getTooltipString(this);
       if (this.IsParentRoot()) {
         var translations = this.Options.translations;
+        this.getCtx().setAriaLabel(
+          ResourceUtils.format(translations.labelAndValue, [
+            translations.labelDataVisualization,
+            Displayable.generateAriaLabel(
+              AriaUtils.processAriaLabel(this.GetComponentDescription()),
+              tooltip ? [tooltip] : null
+            )
+          ])
+        );
         if (!this.IsInteractive()) {
           this.getCtx().setAriaRole('img');
-          this.getCtx().setAriaLabel(
-            ResourceUtils.format(translations.labelAndValue, [
-              translations.labelDataVisualization,
-              Displayable.generateAriaLabel(
-                AriaUtils.processAriaLabel(this.GetComponentDescription()),
-                tooltip ? [tooltip] : null
-              )
-            ])
-          );
         } else {
-          this.getCtx().setAriaRole('application');
-          this.getCtx().setAriaLabel(
-            ResourceUtils.format(translations.labelAndValue, [
-              translations.labelDataVisualization,
-              AriaUtils.processAriaLabel(this.GetComponentDescription())
-            ])
-          );
+          this.getCtx().setAriaRole('slider');
         }
       } else if (!this.IsInteractive()) {
         this.setAriaRole('img');
@@ -2588,7 +2560,6 @@ class RatingGauge extends DvtGauge {
         hoverContainer.setClipPath(hoverClip);
       }
     }
-    this.UpdateAriaLiveValue(container, value);
   }
 
   /**

package/dist/js/libs/oj/debug_esm/ojgauge.js

@@ -3419,6 +3419,10 @@ oj.__registerWidget('oj.ojStatusMeterGauge', $.oj.dvtBaseGauge, {
  * <p>title will no longer be required to make your rating gauge accessible. Use aria-label attribute instead for text to be used by screen readers, and datatip or tooltip for text to be shown on hover or focus.
  * The tooltip attribute is for readonly rating gauge only. If aria-label is not present for readonly rating gauge, tooltip will be used as aria-label. Note the datatip should be used for interactive gauges only and it takes a function.
  * </p>
+ * <h5>role change</h5>
+ * <p>The role of interactive oj-rating-gauge has been changed from 'application' to 'slider'. The screenreader behavior will be consistent with that of role slider.
+ * Slider related aria attributes (aria-valuenow, aria-valuemin, aria-valuemax, aria-valuetext) will be populated according to element value, min and max attribute.
+ * </p>
  * @ojfragment migrationDoc
  * @memberof oj.ojRatingGauge
  */
@@ -3699,9 +3703,9 @@ oj.__registerWidget('oj.ojStatusMeterGauge', $.oj.dvtBaseGauge, {
  * <p>
  * indicator-size attribute will no longer support values greater than 1 in oj-c-meter-bar and oj-c-meter-circle.
  * </p>
- * <h5>min height for vertical meter bar</h5>
- * <p>
- * The vertical oj-c-meter-bar now has a default min-height which is theme driven.
+ * <h5>role change</h5>
+ * <p>The role of interactive oj-status-meter-gauge has been changed from 'application' to 'slider'. The screenreader behavior will be consistent with that of role slider.
+ * Slider related aria attributes (aria-valuenow, aria-valuemin, aria-valuemax, aria-valuetext) will be populated according to element value, min and max attribute.
  * </p>
  * @ojfragment migrationDoc
  * @memberof oj.ojStatusMeterGauge

package/dist/js/libs/oj/debug_esm/ojlegend-toolkit.js

@@ -2034,7 +2034,11 @@ const DvtLegendRenderer = {
       marker.setStyle().setClassName();
     }
 
-    if ((hideAndShow != 'none' && hideAndShow != 'off') || item['shortDesc'] != null) {
+    if (
+      (hideAndShow != 'none' && hideAndShow != 'off') ||
+      item['shortDesc'] != null ||
+      item['drilling'] == 'on'
+    ) {
       itemRect.setAriaRole('img');
       peer.updateAriaLabel();
     }

package/dist/js/libs/oj/debug_esm/ojlistview.js

@@ -5356,7 +5356,7 @@ const _ojListView = _ListViewUtils.clazz(
         tempFocusDiv.setAttribute('tabindex', '-1');
         container.appendChild(tempFocusDiv);
       }
-      tempFocusDiv.focus();
+      tempFocusDiv.focus({ preventScroll: true });
     },
 
     /**

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

@@ -3993,6 +3993,7 @@ import { isElementRegistered } from 'ojs/ojcustomelement-registry';
       var element = this.element;
       var status = oj.ZOrderUtils.getStatus(element);
       if (status === oj.ZOrderUtils.STATUS.OPEN) {
+        CustomElementUtils.cleanComponentBindings(element[0]);
         element.remove();
       }
     },

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

@@ -1590,7 +1590,11 @@ MessagesViewModel.prototype._refresh = function () {
 };
 
 MessagesViewModel.prototype._surrogateRemoveHandler = function () {
-  this._closeOverlay();
+  var composite = $(this._composite);
+  if (oj.ZOrderUtils.getStatus(composite) === oj.ZOrderUtils.STATUS.OPEN) {
+    CustomElementUtils.cleanComponentBindings(composite[0]);
+    this._closeOverlay();
+  }
 };
 
 MessagesViewModel.prototype._createObservables = function () {

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

@@ -2351,6 +2351,7 @@ import { CustomElementUtils } from 'ojs/ojcustomelement-utils';
       var element = this.element;
       var status = oj.ZOrderUtils.getStatus(element);
       if (status === oj.ZOrderUtils.STATUS.OPEN) {
+        CustomElementUtils.cleanComponentBindings(element[0]);
         element.remove();
       }
     },

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

@@ -1177,11 +1177,11 @@ ZOrderUtils.STATUS = {
  * Key to store the current operation status of the popup.
  * @const
  * @private
- * @type {string}
+ * @type {Symbol}
  * @see ZOrderUtils.getStatus
  * @see ZOrderUtils.setStatus
  */
-ZOrderUtils._STATUS_DATA = 'oj-popup-status';
+ZOrderUtils._STATUS_DATA = Symbol('PopupStatus');
 
 /**
  * Returns the current operation status of the target popup element.
@@ -1196,9 +1196,12 @@ ZOrderUtils.getStatus = function (popup) {
   }
 
   /** @type {?} */
-  var status = popup.data(ZOrderUtils._STATUS_DATA);
-  if (isNaN(status)) {
-    return ZOrderUtils.STATUS.UNKNOWN;
+  var status = ZOrderUtils.STATUS.UNKNOWN;
+  if (popup.length) {
+    status = popup[0][ZOrderUtils._STATUS_DATA];
+    if (isNaN(status)) {
+      status = ZOrderUtils.STATUS.UNKNOWN;
+    }
   }
   return status;
 };
@@ -1214,9 +1217,10 @@ ZOrderUtils.setStatus = function (popup, status) {
     // eslint-disable-next-line no-param-reassign
     popup = $(popup);
   }
+  var popupDom = popup[0];
 
   if (status >= ZOrderUtils.STATUS.UNKNOWN && status <= ZOrderUtils.STATUS.CLOSE) {
-    popup.data(ZOrderUtils._STATUS_DATA, status);
+    popupDom[ZOrderUtils._STATUS_DATA] = status;
   }
 };
 
@@ -1324,7 +1328,7 @@ ZOrderUtils.addToAncestorLayer = function (
   popup.appendTo(layer); // @HTMLUpdateOK
 
   // link the popup to the layer @see ZOrderUtils.getOpenPopupLayer
-  popup.data(ZOrderUtils._LAYER_ID_DATA, layer.attr('id'));
+  popupDom[ZOrderUtils._LAYER_ID_DATA] = layer.attr('id');
 
   layer.appendTo(ancestorLayer); // @HTMLUpdateOK
   subtreeAttached(popupDom);
@@ -1487,11 +1491,13 @@ ZOrderUtils._removeSurrogate = function (layer) {
 ZOrderUtils.getOpenPopupLayer = function (popup) {
   /** @type {?} */
   var layer = popup.parent();
+  var popupDom = popup[0];
+
   if (!layer || layer.length === 0) {
     // the open popup has been detached from the layer before it was closed
     // use the backup pointer for better cleanup
     /** @type {?} */
-    var layerId = popup.data(ZOrderUtils._LAYER_ID_DATA);
+    var layerId = popupDom[ZOrderUtils._LAYER_ID_DATA];
     layer = $(document.getElementById(layerId));
   }
 
@@ -1517,9 +1523,10 @@ ZOrderUtils.removeFromAncestorLayer = function (popup) {
 
   layer.removeData(ZOrderUtils._EVENTS_DATA);
   layer.removeData(ZOrderUtils._MODALITY_DATA);
-  popup.removeData(ZOrderUtils._LAYER_ID_DATA);
 
   var popupDom = popup[0];
+  delete popupDom[ZOrderUtils._LAYER_ID_DATA];
+
   subtreeDetached(popupDom);
   var originatingSubtreeExists = ZOrderUtils._removeSurrogate(layer);
 
@@ -2289,10 +2296,10 @@ ZOrderUtils._SURROGATE_ATTR = 'data-oj-surrogate-id';
  *
  * @const
  * @private
- * @type {string}
+ * @type {Symbol}
  * @see ZOrderUtils.getOpenPopupLayer
  */
-ZOrderUtils._LAYER_ID_DATA = 'oj-popup-layer-id';
+ZOrderUtils._LAYER_ID_DATA = Symbol('PopupLayerId');
 
 /**
  * The attribute name assigned to the popup layer for open dialogs that have a
@@ -4044,7 +4051,7 @@ PopupSkipLink.prototype.Init = function () {
     link.on('keydown keyup keypress', PopupSkipLink._keyHandler);
   }
 
-  sibling.data(PopupSkipLink._SKIPLINK_ATTR, link);
+  sibling[0][PopupSkipLink._SKIPLINK_ATTR] = link;
 };
 
 /**
@@ -4080,8 +4087,8 @@ PopupSkipLink.prototype.destroy = function () {
   delete this._callback;
 
   if (sibling) {
-    var link = sibling.data(PopupSkipLink._SKIPLINK_ATTR);
-    sibling.removeData(PopupSkipLink._SKIPLINK_ATTR);
+    var link = sibling[0][PopupSkipLink._SKIPLINK_ATTR];
+    delete sibling[0][PopupSkipLink._SKIPLINK_ATTR];
     if (link) {
       link.off('click keydown keyup keypress');
       link.remove();
@@ -4101,7 +4108,7 @@ PopupSkipLink.prototype.getLink = function () {
   /** @type {jQuery} */
   var link;
   if (sibling) {
-    link = sibling.data(PopupSkipLink._SKIPLINK_ATTR);
+    link = sibling[0][PopupSkipLink._SKIPLINK_ATTR];
   }
   return link;
 };
@@ -4111,9 +4118,9 @@ PopupSkipLink.prototype.getLink = function () {
  * reference for the associated skip link.
  * @const
  * @private
- * @type {string}
+ * @type {Symbol}
  */
-PopupSkipLink._SKIPLINK_ATTR = 'oj-skiplink';
+PopupSkipLink._SKIPLINK_ATTR = Symbol('ojSkipLink');
 
 /**
  * Coordinate communications between an event being fulfilled and one or more promises

package/dist/js/libs/oj/debug_esm/ojselectbase.js

@@ -5188,6 +5188,11 @@ oj.__registerWidget('oj.ojSelectBase', $.oj.editableValue, {
     // which properties are set by the preact. Then we can use the flags as well as the latest value to determine
     // how to process the value and valueItem correctly.
     const elemState = CustomElementUtils.getElementState(this.OuterWrapper);
+    // JET-66502 - Dynamic UI Defect - oj-dyn form not working with JET 16 for select single LOVs
+    // We only need to enter this block if we're currently processing a value or valueItem
+    // being set.  Otherwise, if multiple properties are being set at the same time, properties
+    // handled here after the value or valueItem may not be processed correctly because we
+    // return early from this method.
     if (
       elemState.getBindingProviderType() === 'preact' &&
       (key === 'value' || key === this._GetValueItemPropertyName())
@@ -5291,6 +5296,14 @@ oj.__registerWidget('oj.ojSelectBase', $.oj.editableValue, {
           this._processingSetOptionsMicroTask = true;
           window.queueMicrotask(processValueAndValueItem);
         }
+        // JET-66502 - Dynamic UI Defect - oj-dyn form not working with JET 16 for select
+        // single LOVs
+        // Because we return early here based on the instance flags for processing a value or
+        // valueItem, we will also return early from here when we try to process any other
+        // properties set at the same time as the value or valueItem and processed after one
+        // of them.  This is why we only conditionally enter this whole block of code when
+        // we're currently processing a value or valueItem.  We skip this block of code for
+        // any other property.
         return;
       }
     }

package/dist/js/libs/oj/debug_esm/ojsunburst.js

@@ -969,6 +969,7 @@ var __oj_sunburst_metadata =
     }
   },
   "methods": {
+    "getAutomation": {},
     "getContextByNode": {},
     "getNode": {},
     "getProperty": {},
@@ -2159,6 +2160,16 @@ oj.__registerWidget('oj.ojSunburst', $.oj.dvtBaseComponent, {
     return this._component.getAutomation().getNode(subIdPath);
   },
 
+  /**
+   * Returns the sunburst automation component used by webdriver.
+   * @memberof oj.ojSunburst
+   * @instance
+   * @ojhidden
+   */
+  getAutomation: function () {
+    return this._component.getAutomation();
+  },
+
   /**
    * {@ojinclude "name":"nodeContextDoc"}
    * @param {!Element} node - {@ojinclude "name":"nodeContextParam"}