Skip to content

Commit c64cc2d

Browse files
committed
fix: improve message action contrast
1 parent 537300c commit c64cc2d

2 files changed

Lines changed: 11 additions & 2 deletions

File tree

apps/web/app/routes/my.messages.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,8 @@ export default function Messages({ loaderData }: Route.ComponentProps) {
9898
<Badge variant={unreadMsgs.length > 0 ? "default" : "secondary"}>
9999
{unreadMsgs.length} 条新消息
100100
</Badge>
101-
<Button size="sm" variant="outline" onClick={() => markAllRead()} disabled={unreadMsgs.length === 0 || markingAll}>
102-
<CheckCheck /> {markingAll ? "处理中" : "全部已读"}
101+
<Button size="sm" variant="secondary" onClick={() => markAllRead()} disabled={unreadMsgs.length === 0 || markingAll}>
102+
<CheckCheck data-icon="inline-start" /> {markingAll ? "处理中" : "全部已读"}
103103
</Button>
104104
</div>
105105
} />

apps/web/tests/Messages.test.tsx

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,15 @@ describe("messages page", () => {
7676
expect(screen.queryByText(/<p>|<\/p>/)).not.toBeInTheDocument();
7777
});
7878

79+
it("keeps the mark-all action readable against the brand header", () => {
80+
renderMessages([], [makeMessage("unread", "新消息")]);
81+
82+
expect(screen.getByRole("button", { name: "全部已读" })).toHaveClass(
83+
"bg-secondary",
84+
"text-secondary-foreground",
85+
);
86+
});
87+
7988
it("omits empty summaries and limits long content", () => {
8089
renderMessages(
8190
[makeMessage("empty", " ", true)],

0 commit comments

Comments
 (0)