Skip to content

Commit 15acea4

Browse files
authored
test: remove styles no longer needed with new base styles (#9914)
1 parent 5520de5 commit 15acea4

File tree

6 files changed

+0
-65
lines changed

6 files changed

+0
-65
lines changed

packages/field-highlighter/test/test-styles.test.js

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,5 @@
11
import { css, registerStyles } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
22

3-
// TODO: subset of Lumo needed for unit tests to pass.
4-
// These should be eventually covered by base styles.
5-
registerStyles(
6-
'vaadin-text-field',
7-
css`
8-
/* For user-tags scrollable container tests */
9-
::slotted([slot='input']) {
10-
min-height: 2.25rem;
11-
}
12-
`,
13-
);
14-
153
registerStyles(
164
'vaadin-select-overlay',
175
css`

packages/select/test/select-test-styles.test.js

Lines changed: 0 additions & 12 deletions
This file was deleted.

packages/select/test/select.test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ import {
1414
} from '@vaadin/testing-helpers';
1515
import sinon from 'sinon';
1616
import './not-animated-styles.js';
17-
import './select-test-styles.test.js';
1817
import '../src/vaadin-select.js';
1918
import '@vaadin/item/src/vaadin-item.js';
2019
import '@vaadin/list-box/src/vaadin-list-box.js';
Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,48 +1,20 @@
11
import { css, registerStyles } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
22

3-
// TODO: subset of Lumo needed for unit tests to pass.
4-
// These should be eventually covered by base styles.
53
registerStyles(
64
'vaadin-tabs',
75
css`
86
::slotted(vaadin-tab) {
9-
display: flex;
107
box-sizing: border-box;
11-
padding: 6px 8px;
128
font-family: -apple-system, 'system-ui', Roboto, 'Segoe UI', Helvetica, Arial, sans-serif;
13-
font-size: 16px;
14-
font-weight: 500;
159
min-width: 55px; /* Use fixed width for scroll tests */
1610
}
1711
1812
:host(:not([orientation='vertical'])) {
19-
position: relative;
2013
min-height: 2.75rem;
2114
}
2215
2316
[part='tabs'] {
2417
gap: 0; /* Disable gap for scroll tests */
2518
}
26-
27-
[part$='button'] {
28-
position: absolute;
29-
box-sizing: border-box;
30-
z-index: 1;
31-
height: 100%;
32-
padding: 6px 8px;
33-
}
34-
35-
[part$='button']::after {
36-
width: 1lh;
37-
height: 1lh;
38-
}
39-
40-
[part='forward-button'] {
41-
inset-inline-end: 0;
42-
}
43-
44-
:host([orientation='vertical']) [part$='button'] {
45-
display: none;
46-
}
4719
`,
4820
);

test/integration/dialog-form-layout-test-styles.js

Lines changed: 0 additions & 11 deletions
This file was deleted.

test/integration/dialog-form-layout.test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { setViewport } from '@vaadin/test-runner-commands';
22
import { fixtureSync, nextFrame, nextRender, nextResize } from '@vaadin/testing-helpers';
33
import './not-animated-styles.js';
4-
import './dialog-form-layout-test-styles.js';
54
import '@vaadin/form-layout/src/vaadin-form-layout.js';
65
import '@vaadin/form-layout/src/vaadin-form-item.js';
76
import '@vaadin/dialog/src/vaadin-dialog.js';

0 commit comments

Comments
 (0)