Skip to content

Commit

Permalink
Fix css classes used in e2e tests
Browse files Browse the repository at this point in the history
  • Loading branch information
talldan committed Feb 28, 2022
1 parent 0f73a32 commit c79bdba
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion packages/e2e-test-utils/src/click-on-more-menu-item.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ import { toggleMoreMenu } from './toggle-more-menu';

const SELECTORS = {
postEditorMenuContainer:
'//*[contains(concat(" ", @class, " "), " interface-more-menu__content ")]',
'//*[contains(concat(" ", @class, " "), " interface-more-menu-dropdown__content ")]',
siteEditorMenuContainer:
'//*[contains(concat(" ", @class, " "), " edit-site-more-menu__content ")]',
};
Expand Down
2 changes: 1 addition & 1 deletion packages/e2e-test-utils/src/toggle-more-menu.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
const SELECTORS = {
postEditorMenuContent: '.interface-more-menu-dropdown__content',
siteEditorMenuContent: '.edit-site-more-menu__content',
postEditorMenu: '.interface-more-menu [aria-label="Options"]',
postEditorMenu: '.interface-more-menu-dropdown [aria-label="Options"]',
siteEditorMenu: '.edit-site-more-menu [aria-label="More tools & options"]',
};

Expand Down
6 changes: 3 additions & 3 deletions packages/e2e-tests/specs/editor/various/popovers.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,16 @@ describe( 'popovers', () => {
describe( 'dropdown', () => {
it( 'toggles via click', async () => {
const isMoreMenuOpen = async () =>
!! ( await page.$( '.interface-more-menu__content' ) );
!! ( await page.$( '.interface-more-menu-dropdown__content' ) );

expect( await isMoreMenuOpen() ).toBe( false );

// Toggle opened.
await page.click( '.interface-more-menu > button' );
await page.click( '.interface-more-menu-dropdown > button' );
expect( await isMoreMenuOpen() ).toBe( true );

// Toggle closed.
await page.click( '.interface-more-menu > button' );
await page.click( '.interface-more-menu-dropdown > button' );
expect( await isMoreMenuOpen() ).toBe( false );
} );
} );
Expand Down
2 changes: 1 addition & 1 deletion packages/edit-post/src/components/header/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@
padding: 0 #{ $grid-unit-15 * 0.5 };
}

.interface-more-menu .components-button,
.interface-more-menu-dropdown .components-button,
.interface-pinned-items .components-button {
margin-right: 0;
}
Expand Down
4 changes: 2 additions & 2 deletions packages/nux/src/components/dot-tip/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -114,12 +114,12 @@ $dot-scale: 3; // How much the pulse animation should scale up by in size
margin-right: 0;
}

&.components-popover.interface-more-menu__content:not([data-y-axis="middle"])[data-y-axis="right"] .components-popover__content {
&.components-popover.interface-more-menu-dropdown__content:not([data-y-axis="middle"])[data-y-axis="right"] .components-popover__content {
/*!rtl:ignore*/
margin-left: -12px;
}

&.components-popover.interface-more-menu__content:not([data-y-axis="middle"])[data-y-axis="left"] .components-popover__content {
&.components-popover.interface-more-menu-dropdown__content:not([data-y-axis="middle"])[data-y-axis="left"] .components-popover__content {
/*!rtl:ignore*/
margin-right: -12px;
}
Expand Down

0 comments on commit c79bdba

Please sign in to comment.