add mxid color toggle

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "cinny",
-  "version": "4.1.0",
+  "version": "4.2.1",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "cinny",
-      "version": "4.1.0",
+      "version": "4.2.1",
       "license": "AGPL-3.0-only",
       "dependencies": {
         "@atlaskit/pragmatic-drag-and-drop": "1.1.6",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "cinny",
-  "version": "4.1.0",
+  "version": "4.2.1",
   "description": "Yet another matrix client",
   "main": "index.js",
   "type": "module",

src/app/components/message/Reply.tsx

@@ -48,6 +48,7 @@ export const ThreadIndicator = as<'div'>(({ ...props }, ref) => (
 type ReplyProps = {
   mx: MatrixClient;
   room: Room;
+  mxidColor?: boolean;
   timelineSet?: EventTimelineSet | undefined;
   replyEventId: string;
   threadRootId?: string | undefined;
@@ -55,7 +56,7 @@ type ReplyProps = {
 };
 
 export const Reply = as<'div', ReplyProps>((_, ref) => {
-  const { mx, room, timelineSet, replyEventId, threadRootId, onClick, ...props } = _;
+  const { mx, room, mxidColor, timelineSet, replyEventId, threadRootId, onClick, ...props } = _;
   const [replyEvent, setReplyEvent] = useState<MatrixEvent | null | undefined>(
     timelineSet?.findEventById(replyEventId)
   );
@@ -101,7 +102,7 @@ export const Reply = as<'div', ReplyProps>((_, ref) => {
       )}
       <ReplyLayout
         as="button"
-        userColor={sender ? colorMXID(sender) : undefined}
+        userColor={sender && mxidColor ? colorMXID(sender) : undefined}
         username={
           sender && (
             <Text size="T300" truncate>

src/app/features/message-search/MessageSearch.tsx

@@ -55,6 +55,7 @@ export function MessageSearch({
   const allRooms = useRooms(mx, allRoomsAtom, mDirects);
   const [mediaAutoLoad] = useSetting(settingsAtom, 'mediaAutoLoad');
   const [urlPreview] = useSetting(settingsAtom, 'urlPreview');
+  const [mxidColor] = useSetting(settingsAtom, 'mxidColor');
   const searchInputRef = useRef<HTMLInputElement>(null);
   const scrollTopAnchorRef = useRef<HTMLDivElement>(null);
   const [searchParams, setSearchParams] = useSearchParams();
@@ -296,6 +297,7 @@ export function MessageSearch({
                     items={group.items}
                     mediaAutoLoad={mediaAutoLoad}
                     urlPreview={urlPreview}
+                    mxidColor={mxidColor}
                     onOpen={navigateRoom}
                   />
                 </VirtualTile>

src/app/features/message-search/SearchResultGroup.tsx

@@ -46,6 +46,7 @@ type SearchResultGroupProps = {
   items: ResultItem[];
   mediaAutoLoad?: boolean;
   urlPreview?: boolean;
+  mxidColor?: boolean;
   onOpen: (roomId: string, eventId: string) => void;
 };
 export function SearchResultGroup({
@@ -54,6 +55,7 @@ export function SearchResultGroup({
   items,
   mediaAutoLoad,
   urlPreview,
+  mxidColor,
   onOpen,
 }: SearchResultGroupProps) {
   const mx = useMatrixClient();
@@ -81,7 +83,15 @@ export function SearchResultGroup({
         handleSpoilerClick: spoilerClickHandler,
         handleMentionClick: mentionClickHandler,
       }),
-    [mx, room, linkifyOpts, highlightRegex, mentionClickHandler, spoilerClickHandler, useAuthentication]
+    [
+      mx,
+      room,
+      linkifyOpts,
+      highlightRegex,
+      mentionClickHandler,
+      spoilerClickHandler,
+      useAuthentication,
+    ]
   );
 
   const renderMatrixEvent = useMatrixEventRenderer<[IEventWithRoomId, string, GetContentCallback]>(
@@ -212,7 +222,14 @@ export function SearchResultGroup({
                         userId={event.sender}
                         src={
                           senderAvatarMxc
-                            ? mxcUrlToHttp(mx, senderAvatarMxc, useAuthentication, 48, 48, 'crop') ?? undefined
+                            ? mxcUrlToHttp(
+                                mx,
+                                senderAvatarMxc,
+                                useAuthentication,
+                                48,
+                                48,
+                                'crop'
+                              ) ?? undefined
                             : undefined
                         }
                         alt={displayName}
@@ -224,7 +241,7 @@ export function SearchResultGroup({
               >
                 <Box gap="300" justifyContent="SpaceBetween" alignItems="Center" grow="Yes">
                   <Box gap="200" alignItems="Baseline">
-                    <Username style={{ color: colorMXID(event.sender) }}>
+                    <Username style={{ color: mxidColor ? colorMXID(event.sender) : undefined }}>
                       <Text as="span" truncate>
                         <b>{displayName}</b>
                       </Text>
@@ -246,6 +263,7 @@ export function SearchResultGroup({
                   <Reply
                     mx={mx}
                     room={room}
+                    mxidColor={mxidColor}
                     replyEventId={replyEventId}
                     threadRootId={threadRootId}
                     onClick={handleOpenClick}

src/app/features/room/RoomInput.tsx

@@ -56,7 +56,13 @@ import {
 } from '../../components/editor';
 import { EmojiBoard, EmojiBoardTab } from '../../components/emoji-board';
 import { UseStateProvider } from '../../components/UseStateProvider';
-import { TUploadContent, encryptFile, getImageInfo, getMxIdLocalPart, mxcUrlToHttp } from '../../utils/matrix';
+import {
+  TUploadContent,
+  encryptFile,
+  getImageInfo,
+  getMxIdLocalPart,
+  mxcUrlToHttp,
+} from '../../utils/matrix';
 import { useTypingStatusUpdater } from '../../hooks/useTypingStatusUpdater';
 import { useFilePicker } from '../../hooks/useFilePicker';
 import { useFilePasteHandler } from '../../hooks/useFilePasteHandler';
@@ -120,6 +126,7 @@ export const RoomInput = forwardRef<HTMLDivElement, RoomInputProps>(
   ({ editor, fileDropContainerRef, roomId, room }, ref) => {
     const mx = useMatrixClient();
     const useAuthentication = useMediaAuthentication();
+    const [mxidColor] = useSetting(settingsAtom, 'mxidColor');
     const [enterForNewline] = useSetting(settingsAtom, 'enterForNewline');
     const [isMarkdown] = useSetting(settingsAtom, 'isMarkdown');
     const commands = useCommands(mx, room);
@@ -499,7 +506,7 @@ export const RoomInput = forwardRef<HTMLDivElement, RoomInputProps>(
                   <Box direction="Column">
                     {replyDraft.relation?.rel_type === RelationType.Thread && <ThreadIndicator />}
                     <ReplyLayout
-                      userColor={colorMXID(replyDraft.userId)}
+                      userColor={mxidColor ? colorMXID(replyDraft.userId) : undefined}
                       username={
                         <Text size="T300" truncate>
                           <b>

src/app/features/room/RoomTimeline.tsx

@@ -424,6 +424,7 @@ export function RoomTimeline({ room, eventId, roomInputRef, editor }: RoomTimeli
   const encryptedRoom = mx.isRoomEncrypted(room.roomId);
   const [messageLayout] = useSetting(settingsAtom, 'messageLayout');
   const [messageSpacing] = useSetting(settingsAtom, 'messageSpacing');
+  const [mxidColor] = useSetting(settingsAtom, 'mxidColor');
   const [hideMembershipEvents] = useSetting(settingsAtom, 'hideMembershipEvents');
   const [hideNickAvatarEvents] = useSetting(settingsAtom, 'hideNickAvatarEvents');
   const [mediaAutoLoad] = useSetting(settingsAtom, 'mediaAutoLoad');
@@ -978,6 +979,7 @@ export function RoomTimeline({ room, eventId, roomInputRef, editor }: RoomTimeli
             mEvent={mEvent}
             messageSpacing={messageSpacing}
             messageLayout={messageLayout}
+            mxidColor={mxidColor}
             collapse={collapse}
             highlight={highlighted}
             edit={editId === mEventId}
@@ -995,6 +997,7 @@ export function RoomTimeline({ room, eventId, roomInputRef, editor }: RoomTimeli
                 <Reply
                   mx={mx}
                   room={room}
+                  mxidColor={mxidColor}
                   timelineSet={timelineSet}
                   replyEventId={replyEventId}
                   threadRootId={threadRootId}
@@ -1050,6 +1053,7 @@ export function RoomTimeline({ room, eventId, roomInputRef, editor }: RoomTimeli
             mEvent={mEvent}
             messageSpacing={messageSpacing}
             messageLayout={messageLayout}
+            mxidColor={mxidColor}
             collapse={collapse}
             highlight={highlighted}
             edit={editId === mEventId}
@@ -1067,6 +1071,7 @@ export function RoomTimeline({ room, eventId, roomInputRef, editor }: RoomTimeli
                 <Reply
                   mx={mx}
                   room={room}
+                  mxidColor={mxidColor}
                   timelineSet={timelineSet}
                   replyEventId={replyEventId}
                   threadRootId={threadRootId}
@@ -1159,6 +1164,7 @@ export function RoomTimeline({ room, eventId, roomInputRef, editor }: RoomTimeli
             mEvent={mEvent}
             messageSpacing={messageSpacing}
             messageLayout={messageLayout}
+            mxidColor={mxidColor}
             collapse={collapse}
             highlight={highlighted}
             canDelete={canRedact || mEvent.getSender() === mx.getUserId()}

src/app/features/room/message/Message.tsx

@@ -51,7 +51,12 @@ import {
   getMemberAvatarMxc,
   getMemberDisplayName,
 } from '../../../utils/room';
-import { getCanonicalAliasOrRoomId, getMxIdLocalPart, isRoomAlias, mxcUrlToHttp } from '../../../utils/matrix';
+import {
+  getCanonicalAliasOrRoomId,
+  getMxIdLocalPart,
+  isRoomAlias,
+  mxcUrlToHttp,
+} from '../../../utils/matrix';
 import { MessageLayout, MessageSpacing } from '../../../state/settings';
 import { useMatrixClient } from '../../../hooks/useMatrixClient';
 import { useRecentEmoji } from '../../../hooks/useRecentEmoji';
@@ -615,6 +620,7 @@ export type MessageProps = {
   relations?: Relations;
   messageLayout: MessageLayout;
   messageSpacing: MessageSpacing;
+  mxidColor?: boolean;
   onUserClick: MouseEventHandler<HTMLButtonElement>;
   onUsernameClick: MouseEventHandler<HTMLButtonElement>;
   onReplyClick: MouseEventHandler<HTMLButtonElement>;
@@ -638,6 +644,7 @@ export const Message = as<'div', MessageProps>(
       relations,
       messageLayout,
       messageSpacing,
+      mxidColor,
       onUserClick,
       onUsernameClick,
       onReplyClick,
@@ -673,7 +680,7 @@ export const Message = as<'div', MessageProps>(
       >
         <Username
           as="button"
-          style={{ color: colorMXID(senderId) }}
+          style={mxidColor ? { color: colorMXID(senderId) } : undefined}
           data-user-id={senderId}
           onContextMenu={onUserClick}
           onClick={onUsernameClick}

src/app/hooks/useMediaAuthentication.ts

@@ -1,10 +1,11 @@
 import { useSpecVersions } from './useSpecVersions';
 
 export const useMediaAuthentication = (): boolean => {
-  const { versions } = useSpecVersions();
+  const { versions, unstable_features: unstableFeatures } = useSpecVersions();
 
   // Media authentication is introduced in spec version 1.11
-  const authenticatedMedia = versions.includes('v1.11');
+  const authenticatedMedia =
+    unstableFeatures?.['org.matrix.msc3916.stable'] || versions.includes('v1.11');
 
   return authenticatedMedia;
 };

src/app/organisms/settings/Settings.jsx

@@ -54,6 +54,7 @@ function AppearanceSection() {
   const [enterForNewline, setEnterForNewline] = useSetting(settingsAtom, 'enterForNewline');
   const [messageLayout, setMessageLayout] = useSetting(settingsAtom, 'messageLayout');
   const [messageSpacing, setMessageSpacing] = useSetting(settingsAtom, 'messageSpacing');
+  const [mxidColor, setMXIDColor] = useSetting(settingsAtom, 'mxidColor');
   const [twitterEmoji, setTwitterEmoji] = useSetting(settingsAtom, 'twitterEmoji');
   const [pageZoom, setPageZoom] = useSetting(settingsAtom, 'pageZoom');
   const [isMarkdown, setIsMarkdown] = useSetting(settingsAtom, 'isMarkdown');
@@ -189,6 +190,11 @@ function AppearanceSection() {
             />
           }
         />
+        <SettingTile
+          title="Username Color"
+          options={<Toggle isActive={mxidColor} onToggle={() => setMXIDColor(!mxidColor)} />}
+          content={<Text variant="b3">Color username with matrix userID.</Text>}
+        />
         <SettingTile
           title="Use ENTER for Newline"
           options={

src/app/pages/auth/AuthFooter.tsx

@@ -15,7 +15,7 @@ export function AuthFooter() {
         target="_blank"
         rel="noreferrer"
       >
-        v4.1.0
+        v4.2.1
       </Text>
       <Text as="a" size="T300" href="https://twitter.com/cinnyapp" target="_blank" rel="noreferrer">
         Twitter

src/app/pages/client/WelcomePage.tsx

@@ -24,7 +24,7 @@ export function WelcomePage() {
                   target="_blank"
                   rel="noreferrer noopener"
                 >
-                  v4.1.0
+                  v4.2.1
                 </a>
               </span>
             }

src/app/pages/client/inbox/Notifications.tsx

@@ -182,6 +182,7 @@ type RoomNotificationsGroupProps = {
   notifications: INotification[];
   mediaAutoLoad?: boolean;
   urlPreview?: boolean;
+  mxidColor?: boolean;
   onOpen: (roomId: string, eventId: string) => void;
 };
 function RoomNotificationsGroupComp({
@@ -189,6 +190,7 @@ function RoomNotificationsGroupComp({
   notifications,
   mediaAutoLoad,
   urlPreview,
+  mxidColor,
   onOpen,
 }: RoomNotificationsGroupProps) {
   const mx = useMatrixClient();
@@ -427,7 +429,14 @@ function RoomNotificationsGroupComp({
                         userId={event.sender}
                         src={
                           senderAvatarMxc
-                            ? mxcUrlToHttp(mx, senderAvatarMxc, useAuthentication, 48, 48, 'crop') ?? undefined
+                            ? mxcUrlToHttp(
+                                mx,
+                                senderAvatarMxc,
+                                useAuthentication,
+                                48,
+                                48,
+                                'crop'
+                              ) ?? undefined
                             : undefined
                         }
                         alt={displayName}
@@ -439,7 +448,7 @@ function RoomNotificationsGroupComp({
               >
                 <Box gap="300" justifyContent="SpaceBetween" alignItems="Center" grow="Yes">
                   <Box gap="200" alignItems="Baseline">
-                    <Username style={{ color: colorMXID(event.sender) }}>
+                    <Username style={{ color: mxidColor ? colorMXID(event.sender) : undefined }}>
                       <Text as="span" truncate>
                         <b>{displayName}</b>
                       </Text>
@@ -461,6 +470,7 @@ function RoomNotificationsGroupComp({
                   <Reply
                     mx={mx}
                     room={room}
+                    mxidColor={mxidColor}
                     replyEventId={replyEventId}
                     threadRootId={threadRootId}
                     onClick={handleOpenClick}
@@ -492,6 +502,7 @@ export function Notifications() {
   const mx = useMatrixClient();
   const [mediaAutoLoad] = useSetting(settingsAtom, 'mediaAutoLoad');
   const [urlPreview] = useSetting(settingsAtom, 'urlPreview');
+  const [mxidColor] = useSetting(settingsAtom, 'mxidColor');
   const screenSize = useScreenSizeContext();
 
   const { navigateRoom } = useRoomNavigate();
@@ -648,6 +659,7 @@ export function Notifications() {
                         <RoomNotificationsGroupComp
                           room={groupRoom}
                           notifications={group.notifications}
+                          mxidColor={mxidColor}
                           mediaAutoLoad={mediaAutoLoad}
                           urlPreview={urlPreview}
                           onOpen={navigateRoom}

src/app/state/settings.ts

@@ -17,6 +17,7 @@ export interface Settings {
   enterForNewline: boolean;
   messageLayout: MessageLayout;
   messageSpacing: MessageSpacing;
+  mxidColor: boolean;
   hideMembershipEvents: boolean;
   hideNickAvatarEvents: boolean;
   mediaAutoLoad: boolean;
@@ -41,6 +42,7 @@ const defaultSettings: Settings = {
   enterForNewline: false,
   messageLayout: 0,
   messageSpacing: '400',
+  mxidColor: true,
   hideMembershipEvents: false,
   hideNickAvatarEvents: true,
   mediaAutoLoad: true,

src/client/state/cons.js

@@ -1,5 +1,5 @@
 const cons = {
-  version: '4.1.0',
+  version: '4.2.1',
   secretKey: {
     ACCESS_TOKEN: 'cinny_access_token',
     DEVICE_ID: 'cinny_device_id',

src/index.tsx

@@ -21,30 +21,25 @@ import './app/i18n';
 document.body.classList.add(configClass, varsClass);
 settings.applyTheme();
 
-const registerServiceWorker = async () => {
+// Register Service Worker
 if ('serviceWorker' in navigator) {
   const swUrl =
     import.meta.env.MODE === 'production'
       ? `${trimTrailingSlash(import.meta.env.BASE_URL)}/sw.js`
       : `/dev-sw.js?dev-sw`;
 
-    await navigator.serviceWorker.register(swUrl);
+  navigator.serviceWorker.register(swUrl);
-
-    navigator.serviceWorker.ready.then((registration) => {
   navigator.serviceWorker.addEventListener('message', (event) => {
-        if (event.data?.type === 'token' && event.data?.messageId) {
+    if (event.data?.type === 'token' && event.data?.responseKey) {
+      // Get the token for SW.
       const token = localStorage.getItem('cinny_access_token') ?? undefined;
-          registration.active?.postMessage({
+      event.source!.postMessage({
-            messageId: event.data.messageId,
+        responseKey: event.data.responseKey,
         token,
       });
     }
   });
-    });
 }
-};
-
-window.addEventListener('load', registerServiceWorker);
 
 const mountApp = () => {
   const rootContainer = document.getElementById('root');

src/sw.ts

@@ -3,26 +3,16 @@
 export type {};
 declare const self: ServiceWorkerGlobalScope;
 
-type Message = { messageId: string };
-type MessageListener = (message: Message) => void;
-const messageListeners = new Map<string, MessageListener>();
-
-self.addEventListener('message', (event) => {
-  const { messageId } = event.data;
-  if (typeof messageId === 'string') {
-    messageListeners.get(messageId)?.(event.data);
-    messageListeners.delete(messageId);
-  }
-});
-
-type TokenMessage = Message & {
-  token?: string;
-};
 async function askForAccessToken(client: Client): Promise<string | undefined> {
   return new Promise((resolve) => {
-    const messageId = Math.random().toString(36);
+    const responseKey = Math.random().toString(36);
-    messageListeners.set(messageId, (message: TokenMessage) => resolve(message.token));
+    const listener = (event: ExtendableMessageEvent) => {
-    client.postMessage({ messageId, type: 'token' });
+      if (event.data.responseKey !== responseKey) return;
+      resolve(event.data.token);
+      self.removeEventListener('message', listener);
+    };
+    self.addEventListener('message', listener);
+    client.postMessage({ responseKey, type: 'token' });
   });
 }