[Glitch] Profile fields redesign

Port 047338e684 to glitch-soc Signed-off-by: Claire <claire.github-309c@sitedethib.com>
2026-03-29 11:11:11 +02:00 · 2026-01-16 13:44:49 +01:00
parent d5fb44da91
commit 40d5c00c3a
11 changed files with 380 additions and 116 deletions
--- a/app/javascript/flavours/glitch/components/mini_card/list.tsx
+++ b/app/javascript/flavours/glitch/components/mini_card/list.tsx
@@ -11,11 +11,13 @@ import classes from './styles.module.css';

 interface MiniCardListProps {
  cards?: (Pick<MiniCardProps, 'label' | 'value'> & { key?: Key })[];
+  className?: string;
  onOverflowClick?: MouseEventHandler;
 }

 export const MiniCardList: FC<MiniCardListProps> = ({
  cards = [],
+  className,
  onOverflowClick,
 }) => {
  const {
@@ -27,29 +29,37 @@ export const MiniCardList: FC<MiniCardListProps> = ({
    maxWidth,
  } = useOverflow();

+  if (!cards.length) {
+    return null;
+  }
+
  return (
-    <div className={classes.wrapper} ref={wrapperRef}>
+    <div className={classNames(classes.wrapper, className)} ref={wrapperRef}>
      <dl className={classes.list} ref={listRef} style={{ maxWidth }}>
        {cards.map((card, index) => (
          <MiniCard
            key={card.key ?? index}
            label={card.label}
            value={card.value}
-            hidden={index >= hiddenIndex}
+            hidden={hasOverflow && index >= hiddenIndex}
          />
        ))}
      </dl>
-      <button
-        type='button'
-        className={classNames(classes.more, !hasOverflow && classes.hidden)}
-        onClick={onOverflowClick}
-      >
-        <FormattedMessage
-          id='minicard.more_items'
-          defaultMessage='+ {count} more'
-          values={{ count: hiddenCount }}
-        />
-      </button>
+      {cards.length > 1 && (
+        <div>
+          <button
+            type='button'
+            className={classNames(classes.more, !hasOverflow && classes.hidden)}
+            onClick={onOverflowClick}
+          >
+            <FormattedMessage
+              id='minicard.more_items'
+              defaultMessage='+{count}'
+              values={{ count: hiddenCount }}
+            />
+          </button>
+        </div>
+      )}
    </div>
  );
 };
--- a/app/javascript/flavours/glitch/components/mini_card/mini_card.stories.tsx
+++ b/app/javascript/flavours/glitch/components/mini_card/mini_card.stories.tsx
@@ -7,18 +7,6 @@ const meta = {
  title: 'Components/MiniCard',
  component: MiniCardList,
  args: {
-    cards: [
-      { label: 'Pronouns', value: 'they/them' },
-      {
-        label: 'Website',
-        value: <a href='https://example.com'>bowie-the-db.meow</a>,
-      },
-      {
-        label: 'Free playlists',
-        value: <a href='https://soundcloud.com/bowie-the-dj'>soundcloud.com</a>,
-      },
-      { label: 'Location', value: 'Purris, France' },
-    ],
    onOverflowClick: action('Overflow clicked'),
  },
  render(args) {
@@ -43,7 +31,22 @@ export default meta;

 type Story = StoryObj<typeof meta>;

-export const Default: Story = {};
+export const Default: Story = {
+  args: {
+    cards: [
+      { label: 'Pronouns', value: 'they/them' },
+      {
+        label: 'Website',
+        value: <a href='https://example.com'>bowie-the-db.meow</a>,
+      },
+      {
+        label: 'Free playlists',
+        value: <a href='https://soundcloud.com/bowie-the-dj'>soundcloud.com</a>,
+      },
+      { label: 'Location', value: 'Purris, France' },
+    ],
+  },
+};

 export const LongValue: Story = {
  args: {
@@ -60,3 +63,9 @@ export const LongValue: Story = {
    ],
  },
 };
+
+export const OneCard: Story = {
+  args: {
+    cards: [{ label: 'Pronouns', value: 'they/them' }],
+  },
+};
--- a/app/javascript/flavours/glitch/components/mini_card/styles.module.css
+++ b/app/javascript/flavours/glitch/components/mini_card/styles.module.css
@@ -6,7 +6,6 @@
 }

 .list {
-  min-width: 0;
  display: flex;
  gap: 4px;
  overflow: hidden;
@@ -21,16 +20,19 @@
  flex-shrink: 0;
 }

-.card {
-  max-width: 20vw;
-  overflow: hidden;
-}
-
 .more {
  color: var(--color-text-secondary);
  font-weight: 600;
  appearance: none;
  background: none;
+  aspect-ratio: 1;
+  height: 100%;
+  transition: all 300ms linear;
+}
+
+.more:hover {
+  background-color: var(--color-bg-brand-softer);
+  color: var(--color-text-primary);
 }

 .hidden {
--- a/app/javascript/flavours/glitch/features/account_timeline/common.ts
+++ b/app/javascript/flavours/glitch/features/account_timeline/common.ts
@@ -0,0 +1,5 @@
+import { isClientFeatureEnabled } from '@/flavours/glitch/utils/environment';
+
+export function isRedesignEnabled() {
+  return isClientFeatureEnabled('profile_redesign');
+}
--- a/app/javascript/flavours/glitch/features/account_timeline/components/account_header.tsx
+++ b/app/javascript/flavours/glitch/features/account_timeline/components/account_header.tsx
@@ -1,18 +1,16 @@
 import { useCallback } from 'react';

-import { useIntl, FormattedMessage } from 'react-intl';
+import { useIntl } from 'react-intl';

 import classNames from 'classnames';
 import { Helmet } from 'react-helmet';

 import { AccountBio } from '@/flavours/glitch/components/account_bio';
-import { AccountFields } from '@/flavours/glitch/components/account_fields';
 import { DisplayName } from '@/flavours/glitch/components/display_name';
 import { AnimateEmojiProvider } from '@/flavours/glitch/components/emoji/context';
 import LockIcon from '@/material-icons/400-24px/lock.svg?react';
 import { openModal } from 'flavours/glitch/actions/modal';
 import { Avatar } from 'flavours/glitch/components/avatar';
-import { FormattedDateWrapper } from 'flavours/glitch/components/formatted_date';
 import { Icon } from 'flavours/glitch/components/icon';
 import { AccountNote } from 'flavours/glitch/features/account/components/account_note';
 import { DomainPill } from 'flavours/glitch/features/account/components/domain_pill';
@@ -31,6 +29,7 @@ import { ActionBar } from '../../account/components/action_bar';
 import { AccountBadges } from './badges';
 import { AccountButtons } from './buttons';
 import { FamiliarFollowers } from './familiar_followers';
+import { AccountHeaderFields } from './fields';
 import { AccountInfo } from './info';
 import { MemorialNote } from './memorial_note';
 import { MovedNote } from './moved_note';
@@ -197,29 +196,7 @@ export const AccountHeader: React.FC<{
                  className='account__header__content'
                />

-                <div className='account__header__fields'>
-                  <dl>
-                    <dt>
-                      <FormattedMessage
-                        id='account.joined_short'
-                        defaultMessage='Joined'
-                      />
-                    </dt>
-                    <dd>
-                      <FormattedDateWrapper
-                        value={account.created_at}
-                        year='numeric'
-                        month='short'
-                        day='2-digit'
-                      />
-                    </dd>
-                  </dl>
-
-                  <AccountFields
-                    fields={account.fields}
-                    emojis={account.emojis}
-                  />
-                </div>
+                <AccountHeaderFields accountId={accountId} />
              </div>
            </div>
          )}
--- a/app/javascript/flavours/glitch/features/account_timeline/components/fields.tsx
+++ b/app/javascript/flavours/glitch/features/account_timeline/components/fields.tsx
@@ -0,0 +1,97 @@
+import { useCallback, useMemo } from 'react';
+import type { FC } from 'react';
+
+import { FormattedMessage } from 'react-intl';
+
+import { openModal } from '@/flavours/glitch/actions/modal';
+import { AccountFields } from '@/flavours/glitch/components/account_fields';
+import { EmojiHTML } from '@/flavours/glitch/components/emoji/html';
+import { FormattedDateWrapper } from '@/flavours/glitch/components/formatted_date';
+import { MiniCardList } from '@/flavours/glitch/components/mini_card/list';
+import { useElementHandledLink } from '@/flavours/glitch/components/status/handled_link';
+import { useAccount } from '@/flavours/glitch/hooks/useAccount';
+import type { Account } from '@/flavours/glitch/models/account';
+import { useAppDispatch } from '@/flavours/glitch/store';
+
+import { isRedesignEnabled } from '../common';
+
+import classes from './redesign.module.scss';
+
+export const AccountHeaderFields: FC<{ accountId: string }> = ({
+  accountId,
+}) => {
+  const account = useAccount(accountId);
+
+  if (!account) {
+    return null;
+  }
+
+  if (isRedesignEnabled()) {
+    return <RedesignAccountHeaderFields account={account} />;
+  }
+
+  return (
+    <div className='account__header__fields'>
+      <dl>
+        <dt>
+          <FormattedMessage id='account.joined_short' defaultMessage='Joined' />
+        </dt>
+        <dd>
+          <FormattedDateWrapper
+            value={account.created_at}
+            year='numeric'
+            month='short'
+            day='2-digit'
+          />
+        </dd>
+      </dl>
+
+      <AccountFields fields={account.fields} emojis={account.emojis} />
+    </div>
+  );
+};
+
+const RedesignAccountHeaderFields: FC<{ account: Account }> = ({ account }) => {
+  const htmlHandlers = useElementHandledLink();
+  const cards = useMemo(
+    () =>
+      account.fields.toArray().map(({ value_emojified, name_emojified }) => ({
+        label: (
+          <EmojiHTML
+            htmlString={name_emojified}
+            extraEmojis={account.emojis}
+            className='translate'
+            as='span'
+            {...htmlHandlers}
+          />
+        ),
+        value: (
+          <EmojiHTML
+            as='span'
+            htmlString={value_emojified}
+            extraEmojis={account.emojis}
+            {...htmlHandlers}
+          />
+        ),
+      })),
+    [account.emojis, account.fields, htmlHandlers],
+  );
+
+  const dispatch = useAppDispatch();
+  const handleOverflowClick = useCallback(() => {
+    dispatch(
+      openModal({
+        modalType: 'ACCOUNT_FIELDS',
+        modalProps: { accountId: account.id },
+      }),
+    );
+  }, [account.id, dispatch]);
+
+  return (
+    <MiniCardList
+      cards={cards}
+      className={classes.fieldList}
+      onOverflowClick={handleOverflowClick}
+    />
+  );
+};
--- a/app/javascript/flavours/glitch/features/account_timeline/components/fields_modal.tsx
+++ b/app/javascript/flavours/glitch/features/account_timeline/components/fields_modal.tsx
@@ -0,0 +1,80 @@
+import type { FC } from 'react';
+
+import { FormattedMessage, useIntl } from 'react-intl';
+
+import { DisplayName } from '@/flavours/glitch/components/display_name';
+import { AnimateEmojiProvider } from '@/flavours/glitch/components/emoji/context';
+import { EmojiHTML } from '@/flavours/glitch/components/emoji/html';
+import { IconButton } from '@/flavours/glitch/components/icon_button';
+import { LoadingIndicator } from '@/flavours/glitch/components/loading_indicator';
+import { useElementHandledLink } from '@/flavours/glitch/components/status/handled_link';
+import { useAccount } from '@/flavours/glitch/hooks/useAccount';
+import CloseIcon from '@/material-icons/400-24px/close.svg?react';
+
+import classes from './redesign.module.scss';
+
+export const AccountFieldsModal: FC<{
+  accountId: string;
+  onClose: () => void;
+}> = ({ accountId, onClose }) => {
+  const intl = useIntl();
+  const account = useAccount(accountId);
+  const htmlHandlers = useElementHandledLink();
+
+  if (!account) {
+    return (
+      <div className='modal-root__modal dialog-modal'>
+        <LoadingIndicator />
+      </div>
+    );
+  }
+
+  return (
+    <div className='modal-root__modal dialog-modal'>
+      <div className='dialog-modal__header'>
+        <IconButton
+          icon='close'
+          className={classes.modalCloseButton}
+          onClick={onClose}
+          iconComponent={CloseIcon}
+          title={intl.formatMessage({
+            id: 'account_fields_modal.close',
+            defaultMessage: 'Close',
+          })}
+        />
+        <span className={`${classes.modalTitle} dialog-modal__header__title`}>
+          <FormattedMessage
+            id='account_fields_modal.title'
+            defaultMessage="{name}'s info"
+            values={{
+              name: <DisplayName account={account} variant='simple' />,
+            }}
+          />
+        </span>
+      </div>
+      <div className='dialog-modal__content'>
+        <AnimateEmojiProvider>
+          <dl className={classes.modalFieldsList}>
+            {account.fields.map((field, index) => (
+              <div key={index} className={classes.modalFieldItem}>
+                <EmojiHTML
+                  as='dt'
+                  htmlString={field.name_emojified}
+                  extraEmojis={account.emojis}
+                  className='translate'
+                  {...htmlHandlers}
+                />
+                <EmojiHTML
+                  as='dd'
+                  htmlString={field.value_emojified}
+                  extraEmojis={account.emojis}
+                  {...htmlHandlers}
+                />
+              </div>
+            ))}
+          </dl>
+        </AnimateEmojiProvider>
+      </div>
+    </div>
+  );
+};
--- a/app/javascript/flavours/glitch/features/account_timeline/components/links.tsx
+++ b/app/javascript/flavours/glitch/features/account_timeline/components/links.tsx
@@ -1,58 +0,0 @@
-import type { FC } from 'react';
-
-import { useIntl } from 'react-intl';
-
-import { NavLink } from 'react-router-dom';
-
-import {
-  FollowersCounter,
-  FollowingCounter,
-  StatusesCounter,
-} from '@/flavours/glitch/components/counters';
-import { ShortNumber } from '@/flavours/glitch/components/short_number';
-import { useAccount } from '@/flavours/glitch/hooks/useAccount';
-
-export const AccountLinks: FC<{ accountId: string }> = ({ accountId }) => {
-  const intl = useIntl();
-  const account = useAccount(accountId);
-
-  if (!account) {
-    return null;
-  }
-
-  return (
-    <div className='account__header__extra__links'>
-      <NavLink
-        to={`/@${account.acct}`}
-        title={intl.formatNumber(account.statuses_count)}
-      >
-        <ShortNumber
-          value={account.statuses_count}
-          renderer={StatusesCounter}
-        />
-      </NavLink>
-
-      <NavLink
-        exact
-        to={`/@${account.acct}/following`}
-        title={intl.formatNumber(account.following_count)}
-      >
-        <ShortNumber
-          value={account.following_count}
-          renderer={FollowingCounter}
-        />
-      </NavLink>
-
-      <NavLink
-        exact
-        to={`/@${account.acct}/followers`}
-        title={intl.formatNumber(account.followers_count)}
-      >
-        <ShortNumber
-          value={account.followers_count}
-          renderer={FollowersCounter}
-        />
-      </NavLink>
-    </div>
-  );
-};
--- a/app/javascript/flavours/glitch/features/account_timeline/components/number_fields.tsx
+++ b/app/javascript/flavours/glitch/features/account_timeline/components/number_fields.tsx
@@ -0,0 +1,94 @@
+import type { FC } from 'react';
+
+import { FormattedMessage, useIntl } from 'react-intl';
+
+import classNames from 'classnames';
+import { NavLink } from 'react-router-dom';
+
+import {
+  FollowersCounter,
+  FollowingCounter,
+  StatusesCounter,
+} from '@/flavours/glitch/components/counters';
+import { FormattedDateWrapper } from '@/flavours/glitch/components/formatted_date';
+import { ShortNumber } from '@/flavours/glitch/components/short_number';
+import { useAccount } from '@/flavours/glitch/hooks/useAccount';
+
+import { isRedesignEnabled } from '../common';
+
+import classes from './redesign.module.scss';
+
+export const AccountNumberFields: FC<{ accountId: string }> = ({
+  accountId,
+}) => {
+  const intl = useIntl();
+  const account = useAccount(accountId);
+
+  if (!account) {
+    return null;
+  }
+
+  return (
+    <div
+      className={classNames(
+        'account__header__extra__links',
+        isRedesignEnabled() && classes.fieldNumbersWrapper,
+      )}
+    >
+      {!isRedesignEnabled() && (
+        <NavLink
+          to={`/@${account.acct}`}
+          title={intl.formatNumber(account.statuses_count)}
+        >
+          <ShortNumber
+            value={account.statuses_count}
+            renderer={StatusesCounter}
+          />
+        </NavLink>
+      )}
+
+      <NavLink
+        exact
+        to={`/@${account.acct}/following`}
+        title={intl.formatNumber(account.following_count)}
+      >
+        <ShortNumber
+          value={account.following_count}
+          renderer={FollowingCounter}
+        />
+      </NavLink>
+
+      <NavLink
+        exact
+        to={`/@${account.acct}/followers`}
+        title={intl.formatNumber(account.followers_count)}
+      >
+        <ShortNumber
+          value={account.followers_count}
+          renderer={FollowersCounter}
+        />
+      </NavLink>
+
+      {isRedesignEnabled() && (
+        <NavLink exact to={`/@${account.acct}`}>
+          <FormattedMessage
+            id='account.joined_long'
+            defaultMessage='Joined on {date}'
+            values={{
+              date: (
+                <strong>
+                  <FormattedDateWrapper
+                    value={account.created_at}
+                    year='numeric'
+                    month='short'
+                    day='2-digit'
+                  />
+                </strong>
+              ),
+            }}
+          />
+        </NavLink>
+      )}
+    </div>
+  );
+};
--- a/app/javascript/flavours/glitch/features/account_timeline/components/redesign.module.scss
+++ b/app/javascript/flavours/glitch/features/account_timeline/components/redesign.module.scss
@@ -0,0 +1,47 @@
+.fieldList {
+  margin-top: 16px;
+}
+
+.fieldNumbersWrapper {
+  a {
+    font-weight: unset;
+  }
+}
+
+.modalCloseButton {
+  padding: 8px;
+  border-radius: 50%;
+  border: 1px solid var(--color-border-primary);
+}
+
+.modalTitle {
+  flex-grow: 1;
+  text-align: center;
+}
+
+.modalFieldsList {
+  padding: 24px;
+}
+
+.modalFieldItem {
+  &:not(:first-child) {
+    padding-top: 12px;
+  }
+
+  &:not(:last-child)::after {
+    content: '';
+    display: block;
+    border-bottom: 1px solid var(--color-border-primary);
+    margin-top: 12px;
+  }
+
+  dt {
+    color: var(--color-text-secondary);
+    font-size: 13px;
+  }
+
+  dd {
+    font-weight: 600;
+    font-size: 15px;
+  }
+}
--- a/app/javascript/flavours/glitch/features/ui/components/modal_root.jsx
+++ b/app/javascript/flavours/glitch/features/ui/components/modal_root.jsx
@@ -93,6 +93,7 @@ export const MODAL_COMPONENTS = {
  'IGNORE_NOTIFICATIONS': IgnoreNotificationsModal,
  'ANNUAL_REPORT': AnnualReportModal,
  'COMPOSE_PRIVACY': () => Promise.resolve({ default: VisibilityModal }),
+  'ACCOUNT_FIELDS': () => import('flavours/glitch/features/account_timeline/components/fields_modal.tsx').then(module => ({ default: module.AccountFieldsModal })),
 };

 export default class ModalRoot extends PureComponent {