@apify/ui-library 0.73.1-fixa11yaudit-b2e9ed.18 → 0.74.0

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@apify/ui-library",
-    "version": "0.73.1-fixa11yaudit-b2e9ed.18+820f7f41382",
+    "version": "0.74.0",
     "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": "820f7f413825fb33b814c6b93708873edfd89257"
+    "gitHead": "275202abde49f18ecb5f20ffba07d52dbde6011e"
 }

package/src/components/button.tsx

@@ -1,4 +1,3 @@
-import type { AriaAttributes } from 'react';
 import React, { forwardRef } from 'react';
 import styled, { css } from 'styled-components';
 
@@ -28,10 +27,8 @@ 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 & ButtonAriaProps & TransientButtonProps & MarginSpacingProps;
+export type CommonButtonProps = RegularButtonProps & 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' aria-label="Copy to clipboard" {...props}>
+        <ActionButton onClick={handleClick} data-test='copy_to_clipboard' {...props}>
             {isCopied ? <CheckIcon size="16" /> : <CopyIcon size="16" />}
         </ActionButton>
     );

package/src/components/code/prism_highlighter.tsx

@@ -46,24 +46,33 @@ const loadLanguages = async () => {
 
 void loadLanguages();
 
-type PreWrapperProps = {
+type StyledPreProps = {
     $hasLinePrefixes: boolean,
     $isSingleLine: boolean
 };
 
-const PreWrapper = styled.pre<PreWrapperProps>`
+const StyledPre = styled.pre<StyledPreProps>`
     width: 100%;
     position: relative;
-    overflow: auto;
     z-index: 2;
     padding: ${({ $isSingleLine }) => ($isSingleLine ? 0 : `${theme.space.space16} 0`)};
     margin: 0;
+    overflow: auto;
 
     code {
         min-width: 100%;
         display: inline-block;
         vertical-align: middle;
 
+        /* The following CSS deal with https://github.com/apify/apify-core/issues/20920 */
+        ${({ $isSingleLine }) => ($isSingleLine ? css`
+            overflow-y: hidden;
+            scrollbar-width: none;
+        ` : css`
+            scrollbar-width: thin;
+            overflow: hidden;
+        `)};
+
         *,
         *::before,
         *::after {
@@ -93,10 +102,10 @@ const PreWrapper = styled.pre<PreWrapperProps>`
                 }
             `}
 
-            ${({ $isSingleLine }) => ($isSingleLine ? css<PreWrapperProps>`
+            ${({ $isSingleLine }) => ($isSingleLine ? css<StyledPreProps>`
                 white-space: nowrap;
                 vertical-align: text-top;
-            ` : css<PreWrapperProps>`
+            ` : css<StyledPreProps>`
                 padding-left: ${({ $hasLinePrefixes }) => ($hasLinePrefixes ? theme.space.space8 : theme.space.space16)};
                 padding-right: ${theme.space.space16};
 
@@ -153,8 +162,7 @@ export const PrismSyntaxHighlighter = forwardRef<HTMLPreElement, SyntaxHighlight
             {...rest}
         >
             {({ className: prismClassName, tokens, getLineProps, getTokenProps }) => (
-                <PreWrapper
-                    tabIndex={0}
+                <StyledPre
                     className={clsx(className, prismClassName)}
                     $hasLinePrefixes={hasLinePrefixes}
                     $isSingleLine={isSingleLine}
@@ -174,7 +182,7 @@ export const PrismSyntaxHighlighter = forwardRef<HTMLPreElement, SyntaxHighlight
                             </div>
                         ))}
                     </Text>
-                </PreWrapper>
+                </StyledPre>
             )}
         </Highlight>
     );

package/src/components/floating/menu.tsx

@@ -44,7 +44,6 @@ export interface MenuOption {
 }
 
 export interface MenuProps<T = MenuOption> {
-    ariaLabel?: string,
     options: T[],
     value?: string,
     onSelect: (newValue: string, selectedBy: SelectActionType) => void,
@@ -65,7 +64,6 @@ const defaultRenderOption = ({ label }: MenuOption) => (<Text as='span'>{label}<
  * It tries to mimic API of react-select as closely as possible
  */
 export const Menu = <T extends MenuOption>({
-    ariaLabel,
     options,
     value,
     onSelect,
@@ -131,8 +129,7 @@ export const Menu = <T extends MenuOption>({
             {...rest}
         >
             <MenuBaseComponent
-                aria-label={ariaLabel}
-                aria-labelledby={ariaLabel ? undefined : 'select-label'}
+                aria-labelledby="select-label"
                 aria-autocomplete="none"
                 ref={refs.setReference}
                 {...getReferenceProps()}

package/src/components/tabs/tabs.tsx

@@ -66,7 +66,7 @@ const TabsWrapper = styled(Box)<TabsWrapperProps>`
     display: flex;
     position: relative;
 
-    [role='tablist'] {
+    [role='tabpanel'] {
         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} data-test="tabs-wrapper" $variant={variant}>
+    return <TabsWrapper {...props} role="tablist" data-test="tabs-wrapper" $variant={variant}>
         <div role="cell" aria-hidden={!overflowState?.left} />
-        <div ref={ref} role="tablist" onScroll={scrollHandler}>
+        <div ref={ref} role="tabpanel" onScroll={scrollHandler}>
             {tabs.map((tab) => (
                 <Tab
                     {...tab}

package/src/components/to_consolidate/pagination.tsx

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