Merge pull request #740 from Safelite/SSR-1342-fix-link
Remove aria-label for links. Add sr-only class for injected links in alerts.
This commit is contained in:
commit
46bd1037a9
3 changed files with 2 additions and 3 deletions
|
|
@ -250,7 +250,8 @@ function mapStringToModal(str) {
|
||||||
);
|
);
|
||||||
const splitParams = params.split(',');
|
const splitParams = params.split(',');
|
||||||
|
|
||||||
const bodyText = `<a href="#" modalTarget="${splitParams[0]}" class="modal-text" role="link">${splitParams[1]}</a>`;
|
const bodyText = `<a href="#" modalTarget="${splitParams[0]}" class="modal-text">
|
||||||
|
<span class="sr-only">Link:</span>${splitParams[1]}</a>`;
|
||||||
|
|
||||||
let returnVal = str.replace(linkToReplace, bodyText);
|
let returnVal = str.replace(linkToReplace, bodyText);
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -6,7 +6,6 @@
|
||||||
linkType="text"
|
linkType="text"
|
||||||
:text="serviceZipLinkText"
|
:text="serviceZipLinkText"
|
||||||
href="#!"
|
href="#!"
|
||||||
aria-label="Modal window"
|
|
||||||
@clickEvent="openModal" />
|
@clickEvent="openModal" />
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|
|
||||||
|
|
@ -41,7 +41,6 @@
|
||||||
:text="getRouterLinkDisplayTextFromCopy(copy)"
|
:text="getRouterLinkDisplayTextFromCopy(copy)"
|
||||||
href="#!"
|
href="#!"
|
||||||
:data-bs-target="'#' + getRouterLinkRouteFromCopy(copy)"
|
:data-bs-target="'#' + getRouterLinkRouteFromCopy(copy)"
|
||||||
aria-label="Modal window"
|
|
||||||
@clickEvent="textLinkEmit(copy)" />
|
@clickEvent="textLinkEmit(copy)" />
|
||||||
</span>
|
</span>
|
||||||
</template>
|
</template>
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue