Skip to content

Commit

Permalink
search result aria label should prioritze content over location (#179795
Browse files Browse the repository at this point in the history
)

Fixes #179718
  • Loading branch information
andreamah authored Apr 12, 2023
1 parent 0ae8769 commit c639eeb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/vs/workbench/contrib/search/browser/searchResultsView.ts
Original file line number Diff line number Diff line change
Expand Up @@ -401,10 +401,10 @@ export class SearchAccessibilityProvider implements IListAccessibilityProvider<R
const range = match.range();
const matchText = match.text().substr(0, range.endColumn + 150);
if (replace) {
return nls.localize('replacePreviewResultAria', "Replace '{0}' with '{1}' at column {2} in line {3}", matchString, match.replaceString, range.startColumn + 1, matchText);
return nls.localize('replacePreviewResultAria', "Replace '{0}' with '{1}' in line '{2}' at column {3}", matchString, match.replaceString, matchText, range.startColumn + 1);
}

return nls.localize('searchResultAria', "Found '{0}' at column {1} in line '{2}'", matchString, range.startColumn + 1, matchText);
return nls.localize('searchResultAria', "Found '{0}' in line '{1}' at column {2}", matchString, matchText, range.startColumn + 1);
}
return null;
}
Expand Down

0 comments on commit c639eeb

Please sign in to comment.