merge from develop

Author: XingY

packages/components/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@labkey/components",
-  "version": "7.23.3-fb-parseComma.1",
+  "version": "7.23.3",
   "description": "Components, models, actions, and utility functions for LabKey applications and pages",
   "sideEffects": false,
   "files": [
@@ -75,8 +75,7 @@
     "react-select": "~5.10.2",
     "react-treebeard": "~3.2.4",
     "vis-data": "~8.0.3",
-    "vis-network": "~10.0.2",
-    "papaparse": "5.5.3"
+    "vis-network": "~10.0.2"
   },
   "devDependencies": {
     "@labkey/build": "9.0.0",

packages/components/package.json

@@ -1,6 +1,12 @@
 # @labkey/components
 Components, models, actions, and utility functions for LabKey applications and pages
 
+### version 7.23.3
+*Released*: 19 March 2026
+- GitHub Issue 942: Add error for duplicate values for MVTC fields
+- GitHub Issue 961: Clicking the "Allow multiple selections" label doesn't toggle the checkbox
+- GitHub Issue 932: No help text for disabled Multi-Value checkbox in designer
+
 ### version 7.23.2
 *Released*: 18 March 2026
 - Merge from release26.3-SNAPSHOT to develop

packages/components/releaseNotes/components.md

@@ -126,6 +126,7 @@ interface Props extends PropsWithChildren {
     className?: string;
     delay?: number;
     id?: string;
+    noShow?: boolean;
     overlay: ReactElement<OverlayComponent>; // See note in doc string below
     style?: CSSProperties;
     triggerType?: TriggerType;
@@ -155,6 +156,7 @@ export const OverlayTrigger: FC<Props> = ({
     overlay,
     triggerType = 'hover',
     style,
+    noShow,
 }) => {
     const id_ = useMemo(() => id ?? generateId(), [id]);
     const { onMouseEnter, onMouseLeave, onClick, portalEl, show, targetRef } = useOverlayTriggerState(
@@ -167,6 +169,8 @@ export const OverlayTrigger: FC<Props> = ({
     const className_ = classNames('overlay-trigger', className);
     const clonedContent = cloneElement(overlay, { targetRef });
 
+    if (noShow) return children;
+
     return (
         <div
             className={className_}

packages/components/src/internal/OverlayTrigger.tsx

@@ -126,8 +126,6 @@ describe('TextChoiceOptions', () => {
         const multiCheckbox = document.querySelector('input.domain-text-choice-multi') as HTMLInputElement;
         expect(multiCheckbox).toBeInTheDocument();
         expect(multiCheckbox).toBeDisabled();
-        const labelSpan = screen.getByText('Allow multiple selections');
-        expect(labelSpan.getAttribute('title')).toBe('Multiple values are currently used by at least one data row.');
     });
 
     test('multi-choice checkbox not present', () => {

packages/components/src/internal/components/domainproperties/TextChoiceOptions.test.tsx

@@ -27,6 +27,8 @@ import { getTextChoiceInUseValues, TextChoiceInUseValues } from './actions';
 import { createFormInputId } from './utils';
 import { isFieldFullyLocked } from './propertiesUtil';
 import { MULTI_CHOICE_TYPE, TEXT_CHOICE_TYPE } from './PropDescType';
+import { Popover } from '../../Popover';
+import { OverlayTrigger } from '../../OverlayTrigger';
 
 const MIN_VALUES_FOR_SEARCH_COUNT = 2;
 const HELP_TIP_BODY = <p>The set of values to be used as drop-down options to restrict data entry into this field.</p>;
@@ -100,6 +102,7 @@ export const TextChoiceOptionsImpl: FC<ImplProps> = memo(props => {
     const [showAddValuesModal, setShowAddValuesModal] = useState<boolean>();
     const [search, setSearch] = useState<string>('');
     const fieldTypeId = createFormInputId(DOMAIN_FIELD_TYPE, domainIndex, index);
+    const mvPopOverId = useMemo(() => createFormInputId('mv-in-use-popover', domainIndex, index), [domainIndex, index]);
     const isMultiChoiceField = field.dataType.name === MULTI_CHOICE_TYPE.name;
 
     // keep a map from the updated values for the in-use field values to their original values
@@ -280,25 +283,26 @@ export const TextChoiceOptionsImpl: FC<ImplProps> = memo(props => {
                             title={`Add Values (max ${maxValueCount})`}
                         />
                         {allowMultiChoice && (
-                            <>
-                                <input
-                                    checked={field.dataType.name === 'multiChoice'}
-                                    className="domain-text-choice-multi"
-                                    disabled={isFieldFullyLocked(field.lockType) || hasMultiValueInUse}
-                                    id={createFormInputId(DOMAIN_FIELD_TEXTCHOICE_MULTI, domainIndex, index)}
-                                    onChange={onAllowMultiChange}
-                                    type="checkbox"
-                                />
-                                <span
-                                    title={
-                                        hasMultiValueInUse
-                                            ? 'Multiple values are currently used by at least one data row.'
-                                            : ''
-                                    }
-                                >
-                                    Allow multiple selections
-                                </span>
-                            </>
+                            <OverlayTrigger
+                                noShow={!hasMultiValueInUse}
+                                overlay={
+                                    <Popover id={mvPopOverId} placement="top">
+                                        Multiple values are currently used by at least one data row.
+                                    </Popover>
+                                }
+                            >
+                                <label className="label-weight-normal" id={mvPopOverId}>
+                                    <input
+                                        checked={field.dataType.name === 'multiChoice'}
+                                        className="domain-text-choice-multi"
+                                        disabled={isFieldFullyLocked(field.lockType) || hasMultiValueInUse}
+                                        id={createFormInputId(DOMAIN_FIELD_TEXTCHOICE_MULTI, domainIndex, index)}
+                                        onChange={onAllowMultiChange}
+                                        type="checkbox"
+                                    />
+                                    <span>Allow multiple selections</span>
+                                </label>
+                            </OverlayTrigger>
                         )}
                     </div>
                     <div className="col-xs-6 col-lg-4">