@brightspace-ui/core 3.235.1 → 3.236.0

package/components/inputs/docs/input-checkbox.md

@@ -87,7 +87,6 @@ When provided with a `name`, the checkbox will participate in forms if it is `ch
 | `label` | String, required | Label for the input |
 | `label-hidden` | Boolean | Hides the label visually (moves it to the input's `aria-label` attribute) |
 | `name` | String | Name of the form control. Submitted with the form as part of a name/value pair. |
-| `not-tabbable` | Boolean | Sets `tabindex="-1"` on the checkbox. Note that an alternative method of focusing is necessary to implement if using this property. |
 | `supporting-hidden-when-unchecked` | Boolean | Hides the supporting slot when unchecked. |
 | `value` | String | Value of the input |
 

package/components/inputs/input-checkbox.js

@@ -168,11 +168,6 @@ class InputCheckbox extends FormElementMixin(InputInlineHelpMixin(FocusMixin(Ske
 			 * @type {boolean}
 			 */
 			labelHidden: { attribute: 'label-hidden', reflect: true, type: Boolean },
-			/**
-			 * ACCESSIBILITY: ADVANCED: Sets "tabindex="-1"" on the checkbox. Note that an alternative method of focusing is necessary to implement if using this property.
-			 * @type {boolean}
-			 */
-			notTabbable: { type: Boolean, attribute: 'not-tabbable' },
 			/**
 			 * Hides the supporting slot when unchecked
 			 * @type {boolean}
@@ -251,7 +246,6 @@ class InputCheckbox extends FormElementMixin(InputInlineHelpMixin(FocusMixin(Ske
 		this.label = '';
 		this.labelHidden = false;
 		this.name = '';
-		this.notTabbable = false;
 		this.supportingHiddenWhenUnchecked = false;
 		this.value = 'on';
 		this._hasSupporting = false;
@@ -263,7 +257,6 @@ class InputCheckbox extends FormElementMixin(InputInlineHelpMixin(FocusMixin(Ske
 	}
 
 	render() {
-		const tabindex = this.notTabbable ? -1 : undefined;
 		const supportingContentVisible = this._hasSupporting && (this.checked || !this.supportingHiddenWhenUnchecked);
 		const textClasses = {
 			'd2l-input-checkbox-text': true,
@@ -296,7 +289,6 @@ class InputCheckbox extends FormElementMixin(InputInlineHelpMixin(FocusMixin(Ske
 					id="${this.#inputId}"
 					.indeterminate="${this.indeterminate}"
 					name="${ifDefined(this.name)}"
-					tabindex="${ifDefined(tabindex)}"
 					type="checkbox"
 					.value="${this.value}"></span><span class="${classMap(textClasses)}">${label}<slot></slot></span>
 			</label>

package/components/list/list-item-link-mixin.js

@@ -1,7 +1,6 @@
 import '../colors/colors.js';
 import { css, html, nothing } from 'lit';
 import { isInteractiveInListItemComposedPath, ListItemMixin } from './list-item-mixin.js';
-import { getFlag } from '../../helpers/flags.js';
 import { getUniqueId } from '../../helpers/uniqueId.js';
 import { ifDefined } from 'lit/directives/if-defined.js';
 
@@ -46,7 +45,6 @@ export const ListItemLinkMixin = superclass => class extends ListItemMixin(super
 		super();
 		this.actionHref = null;
 		this._primaryActionId = getUniqueId();
-		this._propagateLinkClickEvent = getFlag('GAUD-8733-list-item-propagate-link-click-event', true);
 	}
 
 	willUpdate(changedProperties) {
@@ -65,18 +63,6 @@ export const ListItemLinkMixin = superclass => class extends ListItemMixin(super
 		} else {
 			/** Dispatched when the item's primary link action is clicked */
 			this.dispatchEvent(new CustomEvent('d2l-list-item-link-click', { bubbles: true }));
-
-			if (!this._propagateLinkClickEvent) {
-				e.stopPropagation();
-
-				// Dispatches click event from the list item to maintain existing functionality in consumers that listen for the click event
-				const listItemClickEvent = new e.constructor(e.type, e);
-				listItemClickEvent.preventDefault = () => {
-					e.preventDefault();
-				};
-				/** @ignore */
-				this.dispatchEvent(listItemClickEvent);
-			}
 		}
 	}
 

package/custom-elements.json

@@ -5412,12 +5412,6 @@
           "type": "boolean",
           "default": "false"
         },
-        {
-          "name": "not-tabbable",
-          "description": "ACCESSIBILITY: ADVANCED: Sets \"tabindex=\"-1\"\" on the checkbox. Note that an alternative method of focusing is necessary to implement if using this property.",
-          "type": "boolean",
-          "default": "false"
-        },
         {
           "name": "supporting-hidden-when-unchecked",
           "description": "Hides the supporting slot when unchecked",
@@ -5490,13 +5484,6 @@
           "type": "boolean",
           "default": "false"
         },
-        {
-          "name": "notTabbable",
-          "attribute": "not-tabbable",
-          "description": "ACCESSIBILITY: ADVANCED: Sets \"tabindex=\"-1\"\" on the checkbox. Note that an alternative method of focusing is necessary to implement if using this property.",
-          "type": "boolean",
-          "default": "false"
-        },
         {
           "name": "supportingHiddenWhenUnchecked",
           "attribute": "supporting-hidden-when-unchecked",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brightspace-ui/core",
-  "version": "3.235.1",
+  "version": "3.236.0",
   "description": "A collection of accessible, free, open-source web components for building Brightspace applications",
   "type": "module",
   "repository": "https://github.com/BrightspaceUI/core.git",