Skip to content
This repository was archived by the owner on Oct 22, 2024. It is now read-only.

Reverse order of pinned message list #19

Merged
merged 2 commits into from
Sep 12, 2024
Merged

Reverse order of pinned message list #19

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
41e5465
Reverse pinned message list
florianduros Sep 9, 2024
ebce1aa
Merge branch 'develop' into florianduros/pinned-messages/reverse-pinn…
florianduros Sep 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/components/views/right_panel/PinnedMessagesCard.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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()}
Expand Down
1 change: 0 additions & 1 deletion test/components/views/right_panel/PinnedMessagesCard-test.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down
8 changes: 4 additions & 4 deletions test/components/views/right_panel/__snapshots__/PinnedMessagesCard-test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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>
Expand Down Expand Up @@ -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>
Expand Down Expand Up @@ -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>
Expand Down Expand Up @@ -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>
Expand Down
Loading