Reverse pinned message list (#19)

element-hq · Sep 12, 2024 · f317763 · f317763
1 parent 85b4f17
commit f317763
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/src/components/views/right_panel/PinnedMessagesCard.tsx b/src/components/views/right_panel/PinnedMessagesCard.tsx
@@ -156,7 +156,7 @@ function PinnedMessages({ events, room, permalinkCreator }: PinnedMessagesProps)
                 })}
                 role="list"
             >
-                {events.reverse().map((event, i) => (
+                {events.map((event, i) => (
                     <>
                         <PinnedEventTile
                             key={event.getId()}

diff --git a/test/components/views/right_panel/PinnedMessagesCard-test.tsx b/test/components/views/right_panel/PinnedMessagesCard-test.tsx
@@ -190,7 +190,6 @@ describe("<PinnedMessagesCard />", () => {
     });
 
     it("should show two pinned messages", async () => {
-        //const room = mkRoom([pin1], [pin2]);
         const { asFragment } = await initPinnedMessagesCard([pin1], [pin2]);
 
         expect(screen.queryAllByRole("listitem")).toHaveLength(2);

diff --git a/test/components/views/right_panel/__snapshots__/PinnedMessagesCard-test.tsx.snap b/test/components/views/right_panel/__snapshots__/PinnedMessagesCard-test.tsx.snap
@@ -184,7 +184,7 @@ exports[`<PinnedMessagesCard /> should show two pinned messages 1`] = `
                 class="mx_EventTile_body translate"
                 dir="auto"
               >
-                First pinned message
+                The second one
               </div>
             </div>
           </div>
@@ -250,7 +250,7 @@ exports[`<PinnedMessagesCard /> should show two pinned messages 1`] = `
                 class="mx_EventTile_body translate"
                 dir="auto"
               >
-                The second one
+                First pinned message
               </div>
             </div>
           </div>
@@ -379,7 +379,7 @@ exports[`<PinnedMessagesCard /> unpin all should not allow to unpinall 1`] = `
                 class="mx_EventTile_body translate"
                 dir="auto"
               >
-                First pinned message
+                The second one
               </div>
             </div>
           </div>
@@ -445,7 +445,7 @@ exports[`<PinnedMessagesCard /> unpin all should not allow to unpinall 1`] = `
                 class="mx_EventTile_body translate"
                 dir="auto"
               >
-                The second one
+                First pinned message
               </div>
             </div>
           </div>