TAC: Fix hover state when expanded (#12337)
* Fix TAC hover state * Add playwright test * Update playwright snapshot after last compound style changespull/28217/head
parent
49be30bbc3
commit
e309410266
|
@ -265,6 +265,13 @@ export class Helpers {
|
||||||
return this.getTacButton().click();
|
return this.getTacButton().click();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Hover over the Threads Activity Centre button
|
||||||
|
*/
|
||||||
|
hoverTacButton() {
|
||||||
|
return this.getTacButton().hover();
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Click on a room in the Threads Activity Centre
|
* Click on a room in the Threads Activity Centre
|
||||||
* @param name - room name
|
* @param name - room name
|
||||||
|
|
|
@ -134,4 +134,14 @@ test.describe("Threads Activity Centre", () => {
|
||||||
await toggleSpotlight();
|
await toggleSpotlight();
|
||||||
await expect(page.locator(".mx_SpotlightDialog")).not.toBeVisible();
|
await expect(page.locator(".mx_SpotlightDialog")).not.toBeVisible();
|
||||||
});
|
});
|
||||||
|
|
||||||
|
test("should have the correct hover state", async ({ util, page }) => {
|
||||||
|
await util.hoverTacButton();
|
||||||
|
await expect(util.getSpacePanel()).toMatchScreenshot("tac-hovered.png");
|
||||||
|
|
||||||
|
// Expand the space panel, hover the button and take a screenshot
|
||||||
|
await util.expandSpacePanel();
|
||||||
|
await util.hoverTacButton();
|
||||||
|
await expect(util.getSpacePanel()).toMatchScreenshot("tac-hovered-expanded.png");
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
Binary file not shown.
After Width: | Height: | Size: 15 KiB |
Binary file not shown.
After Width: | Height: | Size: 5.8 KiB |
|
@ -25,6 +25,12 @@
|
||||||
margin: 18px auto auto auto;
|
margin: 18px auto auto auto;
|
||||||
|
|
||||||
&.expanded {
|
&.expanded {
|
||||||
|
/**
|
||||||
|
* override compound default background color when hovered
|
||||||
|
* should disappear when the space panel will be migrated to compound
|
||||||
|
*/
|
||||||
|
background-color: transparent !important;
|
||||||
|
|
||||||
/* align with settings icon */
|
/* align with settings icon */
|
||||||
margin-left: 21px;
|
margin-left: 21px;
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue