Merge pull request #3400 from element-hq/robin/reactions-small
Prefer showing the reaction button at small screen widths
This commit is contained in:
@@ -40,6 +40,11 @@ test("Start a new call then leave and show the feedback screen", async ({
|
|||||||
// The tooltip with the name should be visible
|
// The tooltip with the name should be visible
|
||||||
await expect(page.getByTestId("name_tag")).toContainText("John Doe");
|
await expect(page.getByTestId("name_tag")).toContainText("John Doe");
|
||||||
|
|
||||||
|
// Resize the window to resemble a small mobile phone
|
||||||
|
await page.setViewportSize({ width: 350, height: 660 });
|
||||||
|
// We should still be able to send reactions at this screen size
|
||||||
|
await expect(page.getByRole("button", { name: "Reactions" })).toBeVisible();
|
||||||
|
|
||||||
// leave the call
|
// leave the call
|
||||||
await page.getByTestId("incall_leave").click();
|
await page.getByTestId("incall_leave").click();
|
||||||
await expect(page.getByRole("heading")).toContainText(
|
await expect(page.getByRole("heading")).toContainText(
|
||||||
|
|||||||
@@ -108,13 +108,6 @@ Please see LICENSE in the repository root for full details.
|
|||||||
}
|
}
|
||||||
|
|
||||||
@media (max-width: 370px) {
|
@media (max-width: 370px) {
|
||||||
.raiseHand {
|
|
||||||
display: none;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
@media (max-width: 340px) {
|
|
||||||
.invite,
|
|
||||||
.shareScreen {
|
.shareScreen {
|
||||||
display: none;
|
display: none;
|
||||||
}
|
}
|
||||||
@@ -126,6 +119,13 @@ Please see LICENSE in the repository root for full details.
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@media (max-width: 320px) {
|
||||||
|
.invite,
|
||||||
|
.raiseHand {
|
||||||
|
display: none;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
@media (max-height: 400px) {
|
@media (max-height: 400px) {
|
||||||
.footer {
|
.footer {
|
||||||
padding-block: var(--cpd-space-4x);
|
padding-block: var(--cpd-space-4x);
|
||||||
|
|||||||
Reference in New Issue
Block a user