Skip to content

Commit fe8ad69

Browse files
committed
Add new fix for hiding arrows from screen reades
1 parent ee953f0 commit fe8ad69

File tree

2 files changed

+16
-2
lines changed

2 files changed

+16
-2
lines changed

src/driver.css

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -245,3 +245,7 @@
245245
.driver-popover-arrow-none {
246246
display: none;
247247
}
248+
249+
button.driver-popover-next-btn {
250+
margin-left: 4px;
251+
}

src/popover.ts

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,13 +74,23 @@ export function renderPopover(element: Element, step: DriveStep) {
7474
disableButtons,
7575
showProgress,
7676

77-
nextBtnText = getConfig("nextBtnText") || "Next →",
78-
prevBtnText = getConfig("prevBtnText") || "← Previous",
77+
nextBtnText = getConfig("nextBtnText") || "Next",
78+
prevBtnText = getConfig("prevBtnText") || "Previous",
7979
progressText = getConfig("progressText") || "{current} of {total}",
8080
} = step.popover || {};
8181

82+
const rightArrow = document.createElement("span");
83+
rightArrow.setAttribute("aria-hidden", "true");
84+
rightArrow.innerHTML = " →";
85+
86+
const leftArrow = document.createElement("span");
87+
leftArrow.setAttribute("aria-hidden", "true");
88+
leftArrow.innerHTML = "← ";
89+
8290
popover.nextButton.innerHTML = nextBtnText;
91+
popover.nextButton.appendChild(rightArrow);
8392
popover.previousButton.innerHTML = prevBtnText;
93+
popover.previousButton.insertBefore(leftArrow, popover.previousButton.firstChild);
8494
popover.progress.innerHTML = progressText;
8595

8696
if (title) {

0 commit comments

Comments
 (0)