@apify/ui-library 0.73.0 → 0.73.1-fixa11yaudit-b2e9ed.16

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@apify/ui-library",
-    "version": "0.73.0",
+    "version": "0.73.1-fixa11yaudit-b2e9ed.16+263d6454bff",
     "description": "React UI library used by apify.com",
     "license": "Apache-2.0",
     "type": "module",
@@ -66,5 +66,5 @@
         "typescript": "^5.1.6",
         "typescript-eslint": "^8.24.0"
     },
-    "gitHead": "7903abd6e4c4a66e2568e45ee76aa35a33cc15b7"
+    "gitHead": "263d6454bff1e09d3c9b6d4a9a62408f41c3eb2c"
 }

package/src/components/button.tsx

@@ -1,3 +1,4 @@
+import type { AriaAttributes } from 'react';
 import React, { forwardRef } from 'react';
 import styled, { css } from 'styled-components';
 
@@ -27,8 +28,10 @@ export interface RegularButtonProps {
     disabled?: boolean;
 }
 
+type ButtonAriaProps = Pick<AriaAttributes, 'aria-label' | 'aria-labelledby'>;
+
 // Props shared for both types of the button
-export type CommonButtonProps = RegularButtonProps & TransientButtonProps & MarginSpacingProps;
+export type CommonButtonProps = RegularButtonProps & ButtonAriaProps & TransientButtonProps & MarginSpacingProps;
 // Regular button must have onClick defined
 export type ButtonProps = CommonButtonProps & WithRequired<RegularBoxProps, 'onClick'>;
 // OR it can pass to to become anchor link - in that case onClick is not needed

package/src/components/code/action_button.tsx

@@ -89,7 +89,7 @@ interface CopyButtonProps extends React.ButtonHTMLAttributes<HTMLButtonElement>
 export const CopyButton = ({ code, ...props }: CopyButtonProps) => {
     const [isCopied, handleClick] = useCopyToClipboard({ text: code });
     return (
-        <ActionButton onClick={handleClick} data-test='copy_to_clipboard' {...props}>
+        <ActionButton onClick={handleClick} data-test='copy_to_clipboard' aria-label="Copy to clipboard" {...props}>
             {isCopied ? <CheckIcon size="16" /> : <CopyIcon size="16" />}
         </ActionButton>
     );

package/src/components/code/prism_highlighter.tsx

@@ -46,12 +46,12 @@ const loadLanguages = async () => {
 
 void loadLanguages();
 
-type StyledPreProps = {
+type PreWrapperProps = {
     $hasLinePrefixes: boolean,
     $isSingleLine: boolean
 };
 
-const StyledPre = styled.pre<StyledPreProps>`
+const PreWrapper = styled.pre<PreWrapperProps>`
     width: 100%;
     position: relative;
     overflow: auto;
@@ -93,10 +93,10 @@ const StyledPre = styled.pre<StyledPreProps>`
                 }
             `}
 
-            ${({ $isSingleLine }) => ($isSingleLine ? css<StyledPreProps>`
+            ${({ $isSingleLine }) => ($isSingleLine ? css<PreWrapperProps>`
                 white-space: nowrap;
                 vertical-align: text-top;
-            ` : css<StyledPreProps>`
+            ` : css<PreWrapperProps>`
                 padding-left: ${({ $hasLinePrefixes }) => ($hasLinePrefixes ? theme.space.space8 : theme.space.space16)};
                 padding-right: ${theme.space.space16};
 
@@ -153,7 +153,8 @@ export const PrismSyntaxHighlighter = forwardRef<HTMLPreElement, SyntaxHighlight
             {...rest}
         >
             {({ className: prismClassName, tokens, getLineProps, getTokenProps }) => (
-                <StyledPre
+                <PreWrapper
+                    tabIndex={0}
                     className={clsx(className, prismClassName)}
                     $hasLinePrefixes={hasLinePrefixes}
                     $isSingleLine={isSingleLine}
@@ -173,7 +174,7 @@ export const PrismSyntaxHighlighter = forwardRef<HTMLPreElement, SyntaxHighlight
                             </div>
                         ))}
                     </Text>
-                </StyledPre>
+                </PreWrapper>
             )}
         </Highlight>
     );

package/src/components/tabs/tabs.tsx

@@ -66,7 +66,7 @@ const TabsWrapper = styled(Box)<TabsWrapperProps>`
     display: flex;
     position: relative;
 
-    [role='tabpanel'] {
+    [role='tablist'] {
         min-width: 0;
         display: flex;
         flex-grow: 1;
@@ -152,9 +152,9 @@ export const Tabs = ({ variant = 'default', tabs, activeTab, onSelect, ...props
         setOveflowState(isTabsOverflowing(event.currentTarget as HTMLDivElement));
     }, []);
 
-    return <TabsWrapper {...props} role="tablist" data-test="tabs-wrapper" $variant={variant}>
+    return <TabsWrapper {...props} data-test="tabs-wrapper" $variant={variant}>
         <div role="cell" aria-hidden={!overflowState?.left} />
-        <div ref={ref} role="tabpanel" onScroll={scrollHandler}>
+        <div ref={ref} role="tablist" onScroll={scrollHandler}>
             {tabs.map((tab) => (
                 <Tab
                     {...tab}

package/src/components/to_consolidate/pagination.tsx

@@ -83,6 +83,7 @@ export const PaginationButtons: FC<PaginationButtonsProps> = ({
                 disabled={page <= 0}
                 onClick={() => onPageChange(page - 1)}
                 trackingId='PaginationPrevPage'
+                aria-label="Previous page"
             >
                 <ChevronLeftIcon size="16" />
             </PaginationButtonBase>
@@ -128,6 +129,7 @@ export const PaginationButtons: FC<PaginationButtonsProps> = ({
                 disabled={page >= totalPages - 1}
                 onClick={() => onPageChange(page + 1)}
                 trackingId='PaginationNextPage'
+                aria-label="Next page"
             >
                 <ChevronRightIcon size="16" />
             </PaginationButtonBase>