fix(MainNavigationBar): allow right slot to expand

src/__screenshot_tests__/navigation-bar-screenshot-test.tsx

@@ -285,3 +285,18 @@ test.each(['large' /* 'small' */])('MainNavigationBar inverse with %s menu in DE
     await page.click(startButton);
     expect(await page.screenshot()).toMatchImageSnapshot({failureThreshold: 0.00002});
 });
+
+test('MainNavigationBar large with expanded right slot', async () => {
+    const page = await openStoryPage({
+        id: 'components-navigation-bars-mainnavigationbar--default',
+        device: 'DESKTOP',
+        args: {
+            large: true,
+            expandedRightSlot: true,
+        },
+    });
+
+    const image = await page.screenshot();
+
+    expect(image).toMatchImageSnapshot();
+});

src/__stories__/main-navigation-bar-story.tsx

@@ -3,10 +3,12 @@ import {
     Avatar,
     Badge,
     IconShoppingCartRegular,
+    Inline,
     MainNavigationBar,
     NavigationBarAction,
     NavigationBarActionGroup,
     Placeholder,
+    SearchField,
     Stack,
     Text3,
     useScreenSize,
@@ -45,6 +47,7 @@ type Args = {
     topSlotBackgroundColor: string;
     wide: boolean;
     paddingX: PadSize | 'undefined';
+    expandedRightSlot: boolean;
 };
 
 export const Default: StoryComponent<Args> = ({
@@ -60,10 +63,34 @@ export const Default: StoryComponent<Args> = ({
     topSlotBackgroundColor,
     wide,
     paddingX,
+    expandedRightSlot,
 }) => {
     const [selectedIndex, setSelectedIndex] = React.useState(0);
     const {isDesktopOrBigger} = useScreenSize();
 
+    const right = expandedRightSlot ? (
+        <Inline fullWidth space={16} alignItems="center" growItems={0}>
+            <SearchField fullWidth name="search" label="Search" />
+
+            <NavigationBarAction onPress={() => {}} aria-label="Entrar">
+                <Avatar src={avatarImg} size={isDesktopOrBigger ? 32 : 24} initials="ML" />
+                {isDesktopOrBigger && 'Entrar'}
+            </NavigationBarAction>
+        </Inline>
+    ) : (
+        <NavigationBarActionGroup>
+            <NavigationBarAction onPress={() => {}} aria-label="shopping cart with 2 items">
+                <Badge value={2}>
+                    <IconShoppingCartRegular color="currentColor" />
+                </Badge>
+            </NavigationBarAction>
+            <NavigationBarAction onPress={() => {}} aria-label="Open profile">
+                <Avatar src={avatarImg} size={isDesktopOrBigger ? 32 : 24} initials="ML" />
+                {isDesktopOrBigger && 'María López Serrano'}
+            </NavigationBarAction>
+        </NavigationBarActionGroup>
+    );
+
     return (
         <MainNavigationBar
             variant={variant}
@@ -116,19 +143,7 @@ export const Default: StoryComponent<Args> = ({
                     : undefined
             }
             selectedIndex={selectedIndex}
-            right={
-                <NavigationBarActionGroup>
-                    <NavigationBarAction onPress={() => {}} aria-label="shopping cart with 2 items">
-                        <Badge value={2}>
-                            <IconShoppingCartRegular color="currentColor" />
-                        </Badge>
-                    </NavigationBarAction>
-                    <NavigationBarAction onPress={() => {}} aria-label="Open profile">
-                        <Avatar src={avatarImg} size={isDesktopOrBigger ? 32 : 24} initials="ML" />
-                        {isDesktopOrBigger && 'María López Serrano'}
-                    </NavigationBarAction>
-                </NavigationBarActionGroup>
-            }
+            right={right}
             wide={wide ? (paddingX === 'undefined' ? true : {paddingX}) : false}
         />
     );
@@ -149,6 +164,7 @@ Default.args = {
     topSlotBackgroundColor: '',
     wide: false,
     paddingX: 'undefined',
+    expandedRightSlot: false,
 };
 
 Default.argTypes = {
@@ -171,4 +187,7 @@ Default.argTypes = {
         control: {type: 'select'},
         if: {arg: 'wide'},
     },
+    expandedRightSlot: {
+        control: {type: 'boolean'},
+    },
 };

src/inline.css.ts

@@ -98,6 +98,15 @@ export const stringSpaceWithWrap = style({
     justifyContent: space,
 });
 
+export const flexLayout = style({
+    display: 'flex',
+});
+
+export const growItem = style({
+    flex: 1,
+    minWidth: 0,
+});
+
 globalStyle(`${marginInline} > div`, {
     marginLeft: space,
     marginTop: fallbackVar(verticalSpace, space),

src/inline.tsx

@@ -79,6 +79,23 @@ type Props = {
     fullWidth?: boolean;
     dataAttributes?: DataAttributes;
     wrap?: boolean;
+    growItems?: number | ReadonlyArray<number>;
+};
+
+const getChildKey = (child: React.ReactNode, index: number): React.Key => {
+    if (!React.isValidElement(child)) {
+        return index;
+    }
+
+    return child.key !== null && child.key !== undefined ? child.key : index;
+};
+
+const shouldGrowItem = (growItems: Props['growItems'], index: number): boolean => {
+    if (growItems === undefined) {
+        return false;
+    }
+
+    return Array.isArray(growItems) ? growItems.includes(index) : growItems === index;
 };
 
 const Inline = ({
@@ -93,28 +110,39 @@ const Inline = ({
     fullWidth,
     wrap,
     dataAttributes,
+    growItems,
 }: Props): JSX.Element => {
     const {platformOverrides} = useTheme();
     const isStringSpace = typeof space === 'string';
+    const childrenArray = React.Children.toArray(children).filter((child) => !!child || child === 0);
+
+    const hasGrowItem = fullWidth && childrenArray.some((_, index) => shouldGrowItem(growItems, index));
 
     return (
         <div
             className={classnames(
                 className,
                 styles.inline,
                 wrap ? styles.wrap : fullWidth ? styles.fullWidth : styles.noFullWidth,
-                isStringSpace ? (wrap ? styles.stringSpaceWithWrap : styles.stringSpace) : styles.marginInline
+                isStringSpace
+                    ? wrap
+                        ? styles.stringSpaceWithWrap
+                        : styles.stringSpace
+                    : styles.marginInline,
+                hasGrowItem && styles.flexLayout
             )}
             style={{...applyCssVars(calcInlineVars(space, verticalSpace)), alignItems}}
             role={role}
             aria-label={ariaLabel}
             aria-labelledby={ariaLabel ? undefined : ariaLabelledBy}
             {...getPrefixedDataAttributes(dataAttributes, 'Inline')}
         >
-            {React.Children.map(children, (child) =>
-                !!child || child === 0 ? (
+            {childrenArray.map((child, index) => {
+                return (
                     <div
+                        key={getChildKey(child, index)}
                         role={role === 'list' ? 'listitem' : undefined}
+                        className={classnames(shouldGrowItem(growItems, index) && styles.growItem)}
                         style={{
                             // Hack to fix https://jira.tid.es/browse/WEB-1683
                             // In iOS the inline component sometimes cuts the last line of the content
@@ -126,8 +154,8 @@ const Inline = ({
                     >
                         {child}
                     </div>
-                ) : null
-            )}
+                );
+            })}
         </div>
     );
 };

src/navigation-bar.css.ts

@@ -313,9 +313,16 @@ export const navigationBarContentRightExpanded = style([
     }),
     {
         paddingLeft: 136,
+        minWidth: 0,
+        width: 0,
     },
 ]);
 
+export const navigationBarContentRightExpandedContent = style({
+    width: '100%',
+    minWidth: 0,
+});
+
 const spacerMobile = style({
     '@media': {
         [mq.tabletOrSmaller]: {

src/navigation-bar.tsx

@@ -115,6 +115,8 @@ type NavigationBarContentContainerProps = {
 
 const NavigationBarContentContainer = React.forwardRef<HTMLDivElement, NavigationBarContentContainerProps>(
     ({right, children, desktopOnly, expandRightContent}, ref) => {
+        const shouldWrapRightContent =
+            expandRightContent && React.isValidElement<{fullWidth?: boolean}>(right) && right.props.fullWidth;
         return (
             <div
                 ref={ref}
@@ -129,7 +131,11 @@ const NavigationBarContentContainer = React.forwardRef<HTMLDivElement, Navigatio
                                 : styles.navigationBarContentRight
                         }
                     >
-                        {right}
+                        {shouldWrapRightContent ? (
+                            <div className={styles.navigationBarContentRightExpandedContent}>{right}</div>
+                        ) : (
+                            right
+                        )}
                     </div>
                 )}
             </div>