remove unnecessary _accInfo property and improve keydown handler

Author: dobrinyonkov

packages/fiori/src/ProductSwitch.js

@@ -128,12 +128,6 @@ class ProductSwitch extends UI5Element {
 
 	onBeforeRendering() {
 		this.desktopColumns = this.items.length > 6 ? 4 : 3;
-
-		this.items.forEach(item => {
-			item._accInfo = {
-				role: "listitem",
-			};
-		});
 	}
 
 	_handleResize() {
@@ -163,9 +157,7 @@ class ProductSwitch extends UI5Element {
 	_onkeydown(event) {
 		if (isDown(event)) {
 			this._handleDown(event);
-		}
-
-		if (isUp(event)) {
+		} else if (isUp(event)) {
 			this._handleUp(event);
 		}
 	}

packages/fiori/src/ProductSwitchItem.hbs

@@ -13,7 +13,7 @@
 	</a>
 {{else}}
 	<div
-		role="{{_accInfo.role}}"
+		role="listitem"
 		class="ui5-product-switch-item-root"
 		data-sap-focus-ref
 		@focusout="{{_onfocusout}}"

packages/fiori/src/ProductSwitchItem.js

@@ -102,10 +102,6 @@ const metadata = {
 			defaultValue: "-1",
 			noAttribute: true,
 		},
-
-		_accInfo: {
-			type: Object,
-		},
 	},
 	slots: /** @lends  sap.ui.webcomponents.fiori.ProductSwitchItem.prototype */ {
 	},