diff --git a/e2e-tests/cypress/tests/integration/channels/enterprise/system_console/team_guest_channel_spec.js b/e2e-tests/cypress/tests/integration/channels/enterprise/system_console/team_guest_channel_spec.js index 2c56e8bf65..d14c4bebcf 100644 --- a/e2e-tests/cypress/tests/integration/channels/enterprise/system_console/team_guest_channel_spec.js +++ b/e2e-tests/cypress/tests/integration/channels/enterprise/system_console/team_guest_channel_spec.js @@ -89,7 +89,7 @@ describe('Team Scheme Guest Permissions Test', () => { // * Ensure all checkboxes have the correct unchecked state verifyAllGuestPermissions(false); - cy.get('.cancel-button').click(); + cy.findByTestId('permission-scheme-cancel-button').should('be.visible').click(); //Clean up - Delete scheme cy.findByText(`TestScheme-${randomId}`).siblings('.actions').children('.delete-button').click().wait(TIMEOUTS.HALF_SEC); diff --git a/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/incoming_webhook_is_image_only_spec.js b/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/incoming_webhook_is_image_only_spec.js index a48e4c1c8a..5d77c0b290 100644 --- a/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/incoming_webhook_is_image_only_spec.js +++ b/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/incoming_webhook_is_image_only_spec.js @@ -62,7 +62,7 @@ describe('Incoming webhook', () => { // * Verify that the hook ID in the URL matches with the one shown in a page // * Verify that the copy link is shown. cy.findByText(hookUrl).should('be.visible'). - parent().siblings('.fa-copy').should('be.visible'); + parent().siblings('[data-testid="copyText"]').should('be.visible'); // # Click "Done" and verify that it redirects to incoming webhooks URL cy.findByText('Done').click(); diff --git a/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/setup_incoming_webhook_spec.js b/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/setup_incoming_webhook_spec.js index 8ad451b039..eb1156f33f 100644 --- a/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/setup_incoming_webhook_spec.js +++ b/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/setup_incoming_webhook_spec.js @@ -64,7 +64,7 @@ describe('Incoming webhook', () => { // * Verify that the hook ID in the URL matches with the one shown in a page // * Verify that the copy link is shown. cy.findByText(hookUrl).should('be.visible'). - parent().siblings('.fa-copy').should('be.visible'); + parent().siblings('[data-testid="copyText"]').should('be.visible'); // # Click "Done" and verify that it redirects to incoming webhooks URL cy.findByText('Done').click(); diff --git a/webapp/channels/src/components/actions_menu/__snapshots__/actions_menu.test.tsx.snap b/webapp/channels/src/components/actions_menu/__snapshots__/actions_menu.test.tsx.snap index 5863912c1c..ace3a145f9 100644 --- a/webapp/channels/src/components/actions_menu/__snapshots__/actions_menu.test.tsx.snap +++ b/webapp/channels/src/components/actions_menu/__snapshots__/actions_menu.test.tsx.snap @@ -182,12 +182,12 @@ exports[`components/actions_menu/ActionsMenu no actions - sysadmin - menu should className="visit-marketplace" > diff --git a/webapp/channels/src/components/admin_console/ip_filtering/add_edit_ip_filter_modal.scss b/webapp/channels/src/components/admin_console/ip_filtering/add_edit_ip_filter_modal.scss index 908f3aea11..2d53a7e728 100644 --- a/webapp/channels/src/components/admin_console/ip_filtering/add_edit_ip_filter_modal.scss +++ b/webapp/channels/src/components/admin_console/ip_filtering/add_edit_ip_filter_modal.scss @@ -140,29 +140,6 @@ .buttons { margin-top: 32px; text-align: right; - - .confirm-btn { - &:hover, - &:active, - &:focus, - &:active:focus { - background: - linear-gradient(0deg, rgba(0, 0, 0, 0.16), rgba(0, 0, 0, 0.16)), - var(--button-bg); - } - - height: 40px; - flex: none; - padding: 12px 20px; - border: none; - border-radius: 4px; - margin-left: 8px; - background: var(--button-bg); - color: var(--sys-button-color); - font-size: 14px; - font-weight: 600; - line-height: 14px; - } } } @@ -175,28 +152,5 @@ font-style: normal; font-weight: 600; line-height: 20px; - - .btn-cancel { - padding: 10px 20px; - border: none; - border-radius: 4px; - background: var(--button-bg-8, rgba(28, 88, 217, 0.08)); - color: var(--button-bg, #1c58d9); - } - - .btn-save { - &:disabled { - background: rgba(63, 67, 80, 0.08); - color: rgba(63, 67, 80, 0.32); - cursor: not-allowed; - } - - padding: 10px 20px; - border: none; - border-radius: 4px; - margin-left: 8px; - background: var(--button-bg, #1c58d9); - color: var(--button-color, #fff); - } } } diff --git a/webapp/channels/src/components/admin_console/ip_filtering/add_edit_ip_filter_modal.tsx b/webapp/channels/src/components/admin_console/ip_filtering/add_edit_ip_filter_modal.tsx index cf5c2dbe00..e1d2b3f410 100644 --- a/webapp/channels/src/components/admin_console/ip_filtering/add_edit_ip_filter_modal.tsx +++ b/webapp/channels/src/components/admin_console/ip_filtering/add_edit_ip_filter_modal.tsx @@ -126,7 +126,7 @@ export default function IPFilteringAddOrEditModal({onExited, onSave, existingRan diff --git a/webapp/channels/src/components/announcement_bar/notification_permission_bar/__snapshots__/index.test.tsx.snap b/webapp/channels/src/components/announcement_bar/notification_permission_bar/__snapshots__/index.test.tsx.snap index 1724b7476c..4d57b8c32f 100644 --- a/webapp/channels/src/components/announcement_bar/notification_permission_bar/__snapshots__/index.test.tsx.snap +++ b/webapp/channels/src/components/announcement_bar/notification_permission_bar/__snapshots__/index.test.tsx.snap @@ -14,7 +14,9 @@ exports[`NotificationPermissionBar should render the NotificationPermissionNever We need your permission to show notifications in the browser. - @@ -42,7 +44,9 @@ exports[`NotificationPermissionBar should render the NotificationUnsupportedBar Your browser does not support browser notifications. - diff --git a/webapp/channels/src/components/announcement_bar/purchase_link/purchase_link.scss b/webapp/channels/src/components/announcement_bar/purchase_link/purchase_link.scss index 6ee247cce4..ecc0cdafb6 100644 --- a/webapp/channels/src/components/announcement_bar/purchase_link/purchase_link.scss +++ b/webapp/channels/src/components/announcement_bar/purchase_link/purchase_link.scss @@ -2,16 +2,5 @@ .annnouncementBar__purchaseNow { display: block; - padding: 12px 20px; - border: none; - border-radius: 4px; margin-top: 15px; - background: var(--sys-button-bg); - color: var(--sys-button-color); - - span { - font-size: 14px; - font-weight: 600; - line-height: 14px; - } } diff --git a/webapp/channels/src/components/announcement_bar/purchase_link/purchase_link.tsx b/webapp/channels/src/components/announcement_bar/purchase_link/purchase_link.tsx index 10b40b135a..f946416b88 100644 --- a/webapp/channels/src/components/announcement_bar/purchase_link/purchase_link.tsx +++ b/webapp/channels/src/components/announcement_bar/purchase_link/purchase_link.tsx @@ -27,7 +27,7 @@ const PurchaseLink: React.FC = (props: Props) => { return (