Merge pull request salesforce#1442 from vernak2539/remove-js-void-from-tree

Update Tree branch and item to use new SLDS HTML tags

Author: interactivellama

components/tree/private/branch.jsx

@@ -310,17 +310,17 @@ const renderBranch = (children, props) => {
 					tabIndex="-1"
 				/>
 				{/* eslint-disable no-script-url */}
-				<a
-					id={`${props.htmlId}__label`}
-					href="javascript:void(0)"
-					// eslint-disable-next-line jsx-a11y/no-interactive-element-to-noninteractive-role
-					role="presentation"
-					className="slds-truncate"
-					tabIndex="-1"
-				>
+				<span className="slds-size_1-of-1" id={`${props.htmlId}__label`}>
 					{/* eslint-enable no-script-url */}
-					{<Highlighter search={props.searchTerm}>{props.label}</Highlighter>}
-				</a>
+					{
+						<Highlighter
+							search={props.searchTerm}
+							className="slds-tree__item-label slds-truncate"
+						>
+							{props.label}
+						</Highlighter>
+					}
+				</span>
 			</div>
 			{isLoading ? loader : null}
 			<ul

components/tree/private/item.jsx

@@ -174,16 +174,15 @@ const Item = (props) => {
 					disabled
 				/>
 				{/* eslint-disable no-script-url */}
-				<a
-					tabIndex="-1"
-					href="javascript:void(0)"
-					// eslint-disable-next-line jsx-a11y/no-interactive-element-to-noninteractive-role
-					role="presentation"
-					className="slds-truncate"
-				>
+				<span className="slds-size_1-of-1">
 					{/* eslint-enable no-script-url */}
-					<Highlighter search={props.searchTerm}>{props.label}</Highlighter>
-				</a>
+					<Highlighter
+						search={props.searchTerm}
+						className="slds-tree__item-label slds-truncate"
+					>
+						{props.label}
+					</Highlighter>
+				</span>
 			</div>
 		</li>
 	);