Skip to content

Commit

Permalink
Rename exported noCopy marker util
Browse files Browse the repository at this point in the history
- in case consumers want to use it outside SR text - thanks Davis for the prompt!
  • Loading branch information
cee-chen committed Sep 18, 2024
1 parent 9a22861 commit 1e749f2
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions packages/eui/src/components/basic_table/basic_table.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -668,9 +668,9 @@ export class EuiBasicTable<T extends object = any> extends Component<
return (
<EuiScreenReaderOnly>
<caption css={euiTableCaptionStyles} className="euiTableCaption">
{tabularCopyMarkers.ariaHiddenNoCopyBoundary}
{tabularCopyMarkers.hiddenNoCopyBoundary}
<EuiDelayRender>{captionElement}</EuiDelayRender>
{tabularCopyMarkers.ariaHiddenNoCopyBoundary}
{tabularCopyMarkers.hiddenNoCopyBoundary}
</caption>
</EuiScreenReaderOnly>
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -744,9 +744,9 @@ const CellScreenReaderDescription: FunctionComponent<{
return (
<EuiScreenReaderOnly>
<p>
{tabularCopyMarkers.ariaHiddenNoCopyBoundary}
{tabularCopyMarkers.hiddenNoCopyBoundary}
{` - ${cellPosition}${canExpandCell ? `. ${enterKeyPrompt}` : ''}`}
{tabularCopyMarkers.ariaHiddenNoCopyBoundary}
{tabularCopyMarkers.hiddenNoCopyBoundary}
</p>
</EuiScreenReaderOnly>
);
Expand Down
2 changes: 1 addition & 1 deletion packages/eui/src/services/copy/tabular_copy.testenv.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export const tabularCopyMarkers = {
hiddenTab: <></>,
hiddenNewline: <></>,
hiddenWrapperBoundary: <></>,
ariaHiddenNoCopyBoundary: <></>,
hiddenNoCopyBoundary: <></>,
};

// Don't bother initializing in Kibana Jest either
Expand Down
2 changes: 1 addition & 1 deletion packages/eui/src/services/copy/tabular_copy.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ export const tabularCopyMarkers = {
hiddenWrapperBoundary: (
<VisuallyHide type="boundary">{CHARS.TABULAR_CONTENT_BOUND}</VisuallyHide>
),
ariaHiddenNoCopyBoundary: (
hiddenNoCopyBoundary: (
<VisuallyHide type="no-copy">{CHARS.NO_COPY_BOUND}</VisuallyHide>
),
};
Expand Down

0 comments on commit 1e749f2

Please sign in to comment.