@memori.ai/memori-react 9.0.0-alpha.2 → 9.0.0-alpha.20

package/src/components/ShareButton/ShareButton.css

@@ -23,8 +23,9 @@
     max-height: 1rem;
 }
 
-.memori-share-button .memori-button.memori-button--active:not(:disabled):not(.memori-button--disabled) {
-    box-shadow: 0 0 0 2px color-mix(in srgb, var(--memori-primary, #3b82f6) 40%, transparent);
+.memori-share-button .memori-icon-button.memori-button.memori-button--active:not(:disabled):not(.memori-button--disabled) {
+    /* Soft active owned by IconButton.css */
+    box-shadow: none;
 }
 
 .memori-share-button--trigger-menu-item {

package/src/components/ShareButton/ShareButton.tsx

@@ -16,6 +16,7 @@ import {
   createAlertOptions,
   Tooltip,
 } from '@memori.ai/ui';
+import IconButton from '../IconButton/IconButton';
 import cx from 'classnames';
 import {
   Tenant,
@@ -549,25 +550,26 @@ const ShareButton: React.FC<Props> = ({
           showChevron={false}
           className="memori-share-button--trigger"
           render={(props: React.ButtonHTMLAttributes<HTMLButtonElement>) => (
-            <Button
+            <IconButton
               {...props}
               variant={triggerVariant}
+              active={isShareMenuOpen}
+              title={String(triggerLabel || t('widget.share') || 'Share')}
               aria-label={String(
                 triggerLabel || t('widget.share') || 'Share'
               )}
               icon={<Share2 aria-hidden />}
-              className={cx(
-                {
-                  'memori-share-button--trigger-menu-item':
-                    triggerMode === 'menu-item',
-                },
-                isShareMenuOpen && 'memori-button--active'
-              )}
+              className={cx({
+                'memori-share-button--trigger-menu-item':
+                  triggerMode === 'menu-item',
+              })}
             >
-              {triggerMode === 'menu-item'
-                ? triggerLabel || t('widget.share')
-                : undefined}
-            </Button>
+              {triggerMode === 'menu-item' || triggerLabel ? (
+                <span className="memori-header--action-label">
+                  {triggerLabel || t('widget.share')}
+                </span>
+              ) : undefined}
+            </IconButton>
           )}
         />
       </Tooltip>

package/src/components/ShareButton/__snapshots__/ShareButton.test.tsx.snap

@@ -6,19 +6,20 @@ exports[`renders ShareButton aligned left unchanged 1`] = `
     class="memori-dropdown memori-share-button memori-share-button--align-left"
   >
     <span
-      class="styles-module__root___-ALla"
+      class="memori-tooltip"
     >
       <span
-        class="styles-module__trigger___V2aIF"
+        class="memori-tooltip__trigger"
         id="base-ui-:r1d:"
       >
         <button
           aria-expanded="false"
           aria-haspopup="menu"
           aria-label="widget.share"
-          class="memori-button memori-button--ghost memori-button--md memori-button--icon-only"
+          class="memori-button memori-button--toolbar memori-button--sm memori-button--icon-only memori-icon-button"
           id="base-ui-:r1e:"
           tabindex="0"
+          title="widget.share"
           type="button"
         >
           <span
@@ -88,19 +89,20 @@ exports[`renders ShareButton unchanged 1`] = `
     class="memori-dropdown memori-share-button"
   >
     <span
-      class="styles-module__root___-ALla"
+      class="memori-tooltip"
     >
       <span
-        class="styles-module__trigger___V2aIF"
+        class="memori-tooltip__trigger"
         id="base-ui-:r5:"
       >
         <button
           aria-expanded="false"
           aria-haspopup="menu"
           aria-label="widget.share"
-          class="memori-button memori-button--ghost memori-button--md memori-button--icon-only"
+          class="memori-button memori-button--toolbar memori-button--sm memori-button--icon-only memori-icon-button"
           id="base-ui-:r6:"
           tabindex="0"
+          title="widget.share"
           type="button"
         >
           <span
@@ -170,19 +172,20 @@ exports[`renders ShareButton with button not primary unchanged 1`] = `
     class="memori-dropdown memori-share-button"
   >
     <span
-      class="styles-module__root___-ALla"
+      class="memori-tooltip"
     >
       <span
-        class="styles-module__trigger___V2aIF"
+        class="memori-tooltip__trigger"
         id="base-ui-:rt:"
       >
         <button
           aria-expanded="false"
           aria-haspopup="menu"
           aria-label="widget.share"
-          class="memori-button memori-button--ghost memori-button--md memori-button--icon-only"
+          class="memori-button memori-button--toolbar memori-button--sm memori-button--icon-only memori-icon-button"
           id="base-ui-:ru:"
           tabindex="0"
+          title="widget.share"
           type="button"
         >
           <span
@@ -252,19 +255,20 @@ exports[`renders ShareButton with other tenant img set unchanged 1`] = `
     class="memori-dropdown memori-share-button"
   >
     <span
-      class="styles-module__root___-ALla"
+      class="memori-tooltip"
     >
       <span
-        class="styles-module__trigger___V2aIF"
+        class="memori-tooltip__trigger"
         id="base-ui-:r1t:"
       >
         <button
           aria-expanded="false"
           aria-haspopup="menu"
           aria-label="widget.share"
-          class="memori-button memori-button--ghost memori-button--md memori-button--icon-only"
+          class="memori-button memori-button--toolbar memori-button--sm memori-button--icon-only memori-icon-button"
           id="base-ui-:r1u:"
           tabindex="0"
+          title="widget.share"
           type="button"
         >
           <span
@@ -334,19 +338,20 @@ exports[`renders ShareButton with share chat unchanged 1`] = `
     class="memori-dropdown memori-share-button"
   >
     <span
-      class="styles-module__root___-ALla"
+      class="memori-tooltip"
     >
       <span
-        class="styles-module__trigger___V2aIF"
+        class="memori-tooltip__trigger"
         id="base-ui-:r25:"
       >
         <button
           aria-expanded="false"
           aria-haspopup="menu"
           aria-label="widget.share"
-          class="memori-button memori-button--ghost memori-button--md memori-button--icon-only"
+          class="memori-button memori-button--toolbar memori-button--sm memori-button--icon-only memori-icon-button"
           id="base-ui-:r26:"
           tabindex="0"
+          title="widget.share"
           type="button"
         >
           <span
@@ -416,19 +421,20 @@ exports[`renders ShareButton with tenant img set unchanged 1`] = `
     class="memori-dropdown memori-share-button"
   >
     <span
-      class="styles-module__root___-ALla"
+      class="memori-tooltip"
     >
       <span
-        class="styles-module__trigger___V2aIF"
+        class="memori-tooltip__trigger"
         id="base-ui-:r1l:"
       >
         <button
           aria-expanded="false"
           aria-haspopup="menu"
           aria-label="widget.share"
-          class="memori-button memori-button--ghost memori-button--md memori-button--icon-only"
+          class="memori-button memori-button--toolbar memori-button--sm memori-button--icon-only memori-icon-button"
           id="base-ui-:r1m:"
           tabindex="0"
+          title="widget.share"
           type="button"
         >
           <span
@@ -498,19 +504,20 @@ exports[`renders ShareButton with title unchanged 1`] = `
     class="memori-dropdown memori-share-button"
   >
     <span
-      class="styles-module__root___-ALla"
+      class="memori-tooltip"
     >
       <span
-        class="styles-module__trigger___V2aIF"
+        class="memori-tooltip__trigger"
         id="base-ui-:rl:"
       >
         <button
           aria-expanded="false"
           aria-haspopup="menu"
           aria-label="widget.share"
-          class="memori-button memori-button--ghost memori-button--md memori-button--icon-only"
+          class="memori-button memori-button--toolbar memori-button--sm memori-button--icon-only memori-icon-button"
           id="base-ui-:rm:"
           tabindex="0"
+          title="widget.share"
           type="button"
         >
           <span
@@ -580,19 +587,20 @@ exports[`renders ShareButton with url unchanged 1`] = `
     class="memori-dropdown memori-share-button"
   >
     <span
-      class="styles-module__root___-ALla"
+      class="memori-tooltip"
     >
       <span
-        class="styles-module__trigger___V2aIF"
+        class="memori-tooltip__trigger"
         id="base-ui-:rd:"
       >
         <button
           aria-expanded="false"
           aria-haspopup="menu"
           aria-label="widget.share"
-          class="memori-button memori-button--ghost memori-button--md memori-button--icon-only"
+          class="memori-button memori-button--toolbar memori-button--sm memori-button--icon-only memori-icon-button"
           id="base-ui-:re:"
           tabindex="0"
+          title="widget.share"
           type="button"
         >
           <span
@@ -662,19 +670,20 @@ exports[`renders ShareButton without qr code unchanged 1`] = `
     class="memori-dropdown memori-share-button"
   >
     <span
-      class="styles-module__root___-ALla"
+      class="memori-tooltip"
     >
       <span
-        class="styles-module__trigger___V2aIF"
+        class="memori-tooltip__trigger"
         id="base-ui-:r15:"
       >
         <button
           aria-expanded="false"
           aria-haspopup="menu"
           aria-label="widget.share"
-          class="memori-button memori-button--ghost memori-button--md memori-button--icon-only"
+          class="memori-button memori-button--toolbar memori-button--sm memori-button--icon-only memori-icon-button"
           id="base-ui-:r16:"
           tabindex="0"
+          title="widget.share"
           type="button"
         >
           <span

package/src/components/StartPanel/StartPanel.css

@@ -84,6 +84,8 @@
 .memori--title-container {
     position: relative;
     display: flex;
+    flex-direction: row;
+    flex-wrap: nowrap;
     align-items: flex-start;
     padding: var(--memori-spacing-lg) 0;
     /* position: absolute;
@@ -122,6 +124,7 @@
     overflow: hidden;
     width: var(--memori-spacing-4xl);
     height: var(--memori-spacing-4xl);
+    flex-shrink: 0;
     align-items: center;
     justify-content: center;
     border-radius: 50%;
@@ -135,13 +138,17 @@
 }
 
 .memori--title {
-    display: inline-block;
+    display: block;
+    overflow: hidden;
+    min-width: 0;
     margin: 0;
     margin-bottom: var(--memori-spacing-sm);
     color: var(--memori-text-color);
     font-size: var(--memori-text-size-heading-large);
     font-weight: var(--memori-text-weight-semibold);
     line-height: var(--memori-text-line-tight);
+    text-overflow: ellipsis;
+    white-space: nowrap;
 }
 
 .memori--description,
@@ -210,15 +217,6 @@
     font-size: var(--memori-text-size-small);
 }
 
-.memori--start-description {
-    margin-top: auto;
-    margin-top: var(--memori-spacing-sm);
-    margin-bottom: 0;
-    color: var(--memori-text-color);
-    font-size: var(--memori-text-size-small);
-    opacity: 0.85;
-}
-
 .memori--title-container__content {
     display: flex;
     min-width: 0;

package/src/components/StartPanel/StartPanel.stories.tsx

@@ -51,7 +51,7 @@ const integrationStylesheet =
 const dataTheme =
   integrationConfig?.theme === 'light' || integrationConfig?.theme === 'dark'
     ? integrationConfig.theme
-    : undefined;
+    : 'light';
 
 const Template: Story<Props> = args => (
   <I18nWrapper>
@@ -370,3 +370,25 @@ WithCompletionProviderDown.args = {
   onClickStart: () => {},
   _TEST_forceProviderStatus: 'major',
 };
+
+/** Public agent whose owner has run out of credits: the start button is
+ * disabled and a "not enough credits" badge is shown instead of starting a
+ * session or asking for a password. */
+export const NotEnoughCredits = Template.bind({});
+NotEnoughCredits.args = {
+  memori: {
+    ...memori,
+    privacyType: 'PUBLIC',
+  },
+  tenant,
+  language: 'it',
+  userLang: 'en',
+  setUserLang: () => {},
+  setVenue: () => {},
+  openPositionPopover: () => {},
+  instruct: false,
+  sessionId: sessionID,
+  clickedStart: false,
+  onClickStart: () => {},
+  notEnoughCredits: true,
+};

package/src/components/StartPanel/StartPanel.test.tsx

@@ -1,5 +1,6 @@
 import React from 'react';
-import { render } from '@testing-library/react';
+import { render, fireEvent } from '@testing-library/react';
+import '@testing-library/jest-dom';
 import { memori, tenant, sessionID, integration, user } from '../../mocks/data';
 import StartPanel from './StartPanel';
 
@@ -306,3 +307,65 @@ it('renders StartPanel with completion provider down unchanged', () => {
   );
   expect(container).toMatchSnapshot();
 });
+
+it('renders StartPanel with not enough credits unchanged', () => {
+  const { container } = render(
+    <StartPanel
+      memori={memori}
+      tenant={tenant}
+      language="it"
+      userLang="en"
+      setUserLang={() => {}}
+      setVenue={jest.fn()}
+      openPositionPopover={() => {}}
+      instruct={false}
+      clickedStart={false}
+      onClickStart={() => {}}
+      setShowLoginDrawer={jest.fn()}
+      notEnoughCredits
+    />
+  );
+  expect(container).toMatchSnapshot();
+});
+
+// When the agent owner has not enough credits, opening the chat for a PUBLIC
+// agent must not start a session nor ask for a password: the start button is
+// disabled and a "not enough credits" badge is shown instead.
+it('blocks start and shows credits badge when owner has not enough credits', () => {
+  const onClickStart = jest.fn();
+  const { container, getByText, queryByPlaceholderText } = render(
+    <StartPanel
+      memori={{ ...memori, privacyType: 'PUBLIC' }}
+      tenant={tenant}
+      language="it"
+      userLang="en"
+      setUserLang={() => {}}
+      setVenue={jest.fn()}
+      openPositionPopover={() => {}}
+      instruct={false}
+      clickedStart={false}
+      onClickStart={onClickStart}
+      setShowLoginDrawer={jest.fn()}
+      notEnoughCredits
+    />
+  );
+
+  const startButton = container.querySelector(
+    '.memori--start-actions__start'
+  ) as HTMLButtonElement;
+  expect(startButton).toBeInTheDocument();
+  expect(startButton).toBeDisabled();
+
+  // Clicking the disabled button must not attempt to open a session.
+  fireEvent.click(startButton);
+  expect(onClickStart).not.toHaveBeenCalled();
+
+  // No password field is ever rendered for a public agent.
+  expect(queryByPlaceholderText('Password')).not.toBeInTheDocument();
+
+  // The credits badge is rendered (tooltip content is portaled on hover by
+  // @memori.ai/ui and is covered by BlockedMemoriBadge unit tests).
+  const badge = container.querySelector('.blocked-memori-badge--wrapper');
+  expect(badge).toBeInTheDocument();
+  expect(getByText('memoriBlockedTitle')).toBeInTheDocument();
+});

package/src/components/StartPanel/StartPanel.tsx

@@ -27,6 +27,8 @@ import CompletionProviderStatus, {
   Props as CPSProps,
 } from '../CompletionProviderStatus/CompletionProviderStatus';
 import { Expandable } from '@memori.ai/ui';
+import { usePressTooltip } from '../../helpers/usePressTooltip';
+import { useWidgetSurfaceEl } from '../../context/widgetSurfaceContext';
 
 /**
  * Sentinel values used as non-selectable section headers in the language
@@ -96,6 +98,11 @@ const StartPanel: React.FC<Props> = ({
   showFullDescriptionOnMobile = false,
 }) => {
   const { t, i18n } = useTranslation();
+  const surfaceEl = useWidgetSurfaceEl();
+  const boardOfExpertsTooltip = usePressTooltip();
+  const nsfwTooltip = usePressTooltip();
+  const deepThoughtTooltip = usePressTooltip();
+  const privacyTooltip = usePressTooltip();
   const [translatedDescription, setTranslatedDescription] = useState(
     memori.description
   );
@@ -213,21 +220,35 @@ const StartPanel: React.FC<Props> = ({
       >
         {!!memori.enableBoardOfExperts && (
           <div className="memori--board-of-experts">
-            <Tooltip align="left" content={t('boardOfExperts')}>
+            <Tooltip
+              align="left"
+              content={t('boardOfExperts')}
+              {...boardOfExpertsTooltip.tooltipProps}
+            >
               <Button
                 type="button"
                 variant="ghost"
                 shape="circle"
                 aria-label={t('boardOfExperts') || 'Board of Experts'}
                 icon={<Users aria-hidden />}
+                {...boardOfExpertsTooltip.pressTriggerProps}
               />
             </Tooltip>
           </div>
         )}
         {!!memori.nsfw && (
           <div className="memori--nsfw">
-            <Tooltip align="left" content={t('nsfw')}>
-              <span title={t('nsfw') || 'NSFW'}>🔞</span>
+            <Tooltip
+              align="left"
+              content={t('nsfw')}
+              {...nsfwTooltip.tooltipProps}
+            >
+              <span
+                title={t('nsfw') || 'NSFW'}
+                {...nsfwTooltip.pressTriggerProps}
+              >
+                🔞
+              </span>
             </Tooltip>
           </div>
         )}
@@ -424,7 +445,7 @@ const StartPanel: React.FC<Props> = ({
                           </div>
                         </div>
                         <div className="memori--settings-section__actions-right">
-                          {!isUserLoggedIn && showLogin && (
+                          {!isMobile && !isUserLoggedIn && showLogin && (
                             <Button
                               variant="toolbar"
                               size="sm"
@@ -453,6 +474,7 @@ const StartPanel: React.FC<Props> = ({
                               leaveDelay={150}
                               sideOffset={8}
                               content={t('deepThoughtHelper')}
+                              {...deepThoughtTooltip.tooltipProps}
                               slotProps={{
                                 positioner: {
                                   className:
@@ -471,6 +493,7 @@ const StartPanel: React.FC<Props> = ({
                                 aria-label={String(
                                   t('deepThoughtHelper') ?? 'Deep Thought help'
                                 )}
+                                {...deepThoughtTooltip.pressTriggerProps}
                               >
                                 <Info className="memori--settings-section__inline-info-icon" />
                               </Button>
@@ -557,6 +580,7 @@ const StartPanel: React.FC<Props> = ({
                               </a>
                             </div>
                           }
+                          {...privacyTooltip.tooltipProps}
                           slotProps={{
                             positioner: {
                               className: 'memori--privacy-popover-positioner',
@@ -572,6 +596,7 @@ const StartPanel: React.FC<Props> = ({
                             size="sm"
                             className="memori--privacy-popover-trigger memori--settings-section__info-trigger"
                             aria-label={String(t('privacyPolicy') ?? '')}
+                            {...privacyTooltip.pressTriggerProps}
                           >
                             <Info className="memori--settings-section__inline-info-icon" />
                           </Button>
@@ -640,12 +665,6 @@ const StartPanel: React.FC<Props> = ({
                   )} */}
               </div>
 
-              <p className="memori--start-description">
-                {instruct
-                  ? t('write_and_speak.pageInstructExplanation')
-                  : t('write_and_speak.pageTryMeExplanation')}
-              </p>
-
               {(memori.blockedUntil || notEnoughCredits) && (
                 <BlockedMemoriBadge
                   memoriName={memori.name}
@@ -660,6 +679,7 @@ const StartPanel: React.FC<Props> = ({
           )}
       </div>
       <Modal
+        container={surfaceEl ?? undefined}
         open={isMobile && mobileInfoModal !== null}
         onClose={() => setMobileInfoModal(null)}
         title={