Ensure proper env vars are promoted for dev and test envs

src/components/input/Select.tsx

@@ -26,7 +26,7 @@ import {
 } from '../icons';
 import Checkbox from './Checkbox';
 import { inputGroupStyles } from './InputGroup';
-import type { SelectValueOptions } from './SelectContext';
+import type { ListboxOverflow, SelectValueOptions } from './SelectContext';
 import SelectContext from './SelectContext';
 
 export type SelectOptionStatus = {
@@ -190,22 +190,24 @@ function SelectOption<T>({
     >
       <div
         className={classnames(
-          // Make items stretch so that all have the same height. This is
-          // important for multi-selects, where the checkbox actionable surface
-          // should span to the very edges of the option containing it.
-          'flex justify-between items-stretch',
+          'flex justify-between items-center',
           'w-full rounded',
           {
             'hover:bg-grey-1 group-focus-visible:ring': !disabled,
             'bg-grey-1 hover:bg-grey-2': selected,
           },
         )}
       >
-        <div className="flex items-center py-2 pl-3">
+        <div
+          className={classnames('py-2 pl-3', {
+            truncate: selectContext.listboxOverflow === 'truncate',
+            'whitespace-normal': selectContext.listboxOverflow === 'wrap',
+          })}
+        >
           {optionChildren(children, { selected, disabled })}
         </div>
         {!multiple && (
-          <div className="flex items-center py-2 px-3">
+          <div className="px-3">
             <CheckIcon
               className={classnames('text-grey-6 scale-125', {
                 // Make the icon visible/invisible, instead of conditionally
@@ -219,7 +221,9 @@ function SelectOption<T>({
         {multiple && (
           <Checkbox
             containerClasses={classnames(
-              'flex items-center py-2 px-3',
+              // Make the checkbox stretch, so that its actionable surface spans
+              // to the very edges of the option containing it.
+              'self-stretch px-3',
               // The checkbox is sized based on the container's font size. Make
               // it a bit larger.
               'text-lg',
@@ -250,6 +254,11 @@ SelectOption.displayName = 'Select.Option';
 
 /** Small space to apply between the toggle button and the listbox */
 const LISTBOX_TOGGLE_GAP = '.25rem';
+/**
+ * Space in pixels to apply between the listbox and the viewport sides when the
+ * listbox grows to the very edges
+ */
+const LISTBOX_VIEWPORT_GAP = 8;
 
 type ListboxCSSProps =
   | 'top'
@@ -311,31 +320,51 @@ function useListboxPositioning(
       buttonDistanceToBottom < listboxHeight &&
       buttonDistanceToTop > buttonDistanceToBottom;
 
-    if (asPopover) {
-      const { top: bodyTop } = document.body.getBoundingClientRect();
-      const absBodyTop = Math.abs(bodyTop);
-
-      return setListboxCSSProps({
-        minWidth: `${buttonWidth}px`,
-        top: shouldListboxDropUp
-          ? `calc(${absBodyTop + buttonDistanceToTop - listboxHeight}px - ${LISTBOX_TOGGLE_GAP})`
-          : `calc(${absBodyTop + buttonDistanceToTop + buttonHeight}px + ${LISTBOX_TOGGLE_GAP})`,
-        left:
-          right && listboxWidth > buttonWidth
-            ? `${buttonLeft - (listboxWidth - buttonWidth)}px`
-            : `${buttonLeft}px`,
-      });
+    if (!asPopover) {
+      // Set styles for non-popover mode
+      if (shouldListboxDropUp) {
+        return setListboxCSSProps({
+          bottom: '100%',
+          marginBottom: LISTBOX_TOGGLE_GAP,
+        });
+      }
+
+      return setListboxCSSProps({ top: '100%', marginTop: LISTBOX_TOGGLE_GAP });
     }
 
-    // Set styles for non-popover mode
-    if (shouldListboxDropUp) {
-      return setListboxCSSProps({
-        bottom: '100%',
-        marginBottom: LISTBOX_TOGGLE_GAP,
-      });
+    const { top: bodyTop, width: bodyWidth } =
+      document.body.getBoundingClientRect();
+    const absBodyTop = Math.abs(bodyTop);
+
+    // Calculate space the listbox would have in regular conditions, to make
+    // sure it never increases the body size, which could cause horizontal
+    // scrollbars to appear
+    const regularAvailableSpace =
+      bodyWidth -
+      (right ? buttonLeft - buttonWidth : buttonLeft) -
+      LISTBOX_VIEWPORT_GAP;
+
+    let left = buttonLeft;
+    if (listboxWidth > regularAvailableSpace) {
+      // If the listbox is not going to fit the regular space, let it "grow"
+      // in the opposite direction
+      left = right
+        ? LISTBOX_VIEWPORT_GAP
+        : left - (listboxWidth - regularAvailableSpace);
+    } else if (right && listboxWidth > buttonWidth) {
+      // If a `right` listbox fits in the available space, but it's bigger
+      // than the button, move it to the left so that it is aligned to the
+      // right of the button
+      left -= listboxWidth - buttonWidth;
     }
 
-    return setListboxCSSProps({ top: '100%', marginTop: LISTBOX_TOGGLE_GAP });
+    return setListboxCSSProps({
+      minWidth: `${buttonWidth}px`,
+      top: shouldListboxDropUp
+        ? `calc(${absBodyTop + buttonDistanceToTop - listboxHeight}px - ${LISTBOX_TOGGLE_GAP})`
+        : `calc(${absBodyTop + buttonDistanceToTop + buttonHeight}px + ${LISTBOX_TOGGLE_GAP})`,
+      left: `${Math.max(LISTBOX_VIEWPORT_GAP, left)}px`,
+    });
   }, [asPopover, buttonRef, listboxOpen, listboxRef, right]);
 
   useLayoutEffect(() => {
@@ -405,6 +434,19 @@ type BaseSelectProps = CompositeProps & {
 
   /** A callback passed to the listbox onScroll */
   onListboxScroll?: JSX.HTMLAttributes<HTMLUListElement>['onScroll'];
+
+  /**
+   * Indicates how overflowing content should be handled in the listbox.
+   * * `truncate` -> Truncate the options so that they all fit in one line. An
+   *                 ellipsis will be rendered where needed.
+   *                 This is the default value.
+   * * `wrap` -> Let options content wrap multiple lines so that all content is
+   *             visible.
+   *
+   * This behavior can be also overwritten by providing more complex options
+   * content.
+   */
+  listboxOverflow?: ListboxOverflow;
 };
 
 export type SelectProps<T> = BaseSelectProps & SingleValueProps<T>;
@@ -435,6 +477,7 @@ function SelectMain<T>({
   onListboxScroll,
   right = false,
   multiple,
+  listboxOverflow = 'truncate',
   'aria-label': ariaLabel,
   'aria-labelledby': ariaLabelledBy,
   /* eslint-disable-next-line no-prototype-builtins */
@@ -550,11 +593,15 @@ function SelectMain<T>({
           value: value as typeof multiple extends false ? T : T[],
           selectValue,
           multiple,
+          listboxOverflow,
         }}
       >
         <ul
           className={classnames(
-            'absolute z-5 max-h-80 overflow-y-auto',
+            'absolute z-5 max-h-80 overflow-y-auto overflow-x-hidden',
+            // We don't want the listbox to ever render outside the viewport,
+            // and we give it a 16px gap
+            'max-w-[calc(100%-16px)]',
             'rounded border bg-white shadow hover:shadow-md focus-within:shadow-md',
             !listboxAsPopover && {
               // Hiding instead of unmounting to

src/components/input/SelectContext.ts

@@ -1,5 +1,7 @@
 import { createContext } from 'preact';
 
+export type ListboxOverflow = 'truncate' | 'wrap';
+
 export type SelectValueOptions = {
   closeListbox: boolean;
 };
@@ -16,9 +18,12 @@ type MultiSelectContext<T> = {
   multiple: true;
 };
 
-export type SelectContextType<T = unknown> =
+export type SelectContextType<T = unknown> = (
   | SingleSelectContext<T>
-  | MultiSelectContext<T>;
+  | MultiSelectContext<T>
+) & {
+  listboxOverflow: ListboxOverflow;
+};
 
 const SelectContext = createContext<SelectContextType | null>(null);