Skip to content

Commit c8d3d58

Browse files
authored
Merge pull request #3400 from element-hq/robin/reactions-small
Prefer showing the reaction button at small screen widths
2 parents 7961bb3 + 3002487 commit c8d3d58

File tree

2 files changed

+12
-7
lines changed

2 files changed

+12
-7
lines changed

playwright/create-call.spec.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,11 @@ test("Start a new call then leave and show the feedback screen", async ({
4040
// The tooltip with the name should be visible
4141
await expect(page.getByTestId("name_tag")).toContainText("John Doe");
4242

43+
// Resize the window to resemble a small mobile phone
44+
await page.setViewportSize({ width: 350, height: 660 });
45+
// We should still be able to send reactions at this screen size
46+
await expect(page.getByRole("button", { name: "Reactions" })).toBeVisible();
47+
4348
// leave the call
4449
await page.getByTestId("incall_leave").click();
4550
await expect(page.getByRole("heading")).toContainText(

src/room/InCallView.module.css

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -108,13 +108,6 @@ Please see LICENSE in the repository root for full details.
108108
}
109109

110110
@media (max-width: 370px) {
111-
.raiseHand {
112-
display: none;
113-
}
114-
}
115-
116-
@media (max-width: 340px) {
117-
.invite,
118111
.shareScreen {
119112
display: none;
120113
}
@@ -126,6 +119,13 @@ Please see LICENSE in the repository root for full details.
126119
}
127120
}
128121

122+
@media (max-width: 320px) {
123+
.invite,
124+
.raiseHand {
125+
display: none;
126+
}
127+
}
128+
129129
@media (max-height: 400px) {
130130
.footer {
131131
padding-block: var(--cpd-space-4x);

0 commit comments

Comments
 (0)