Skip to content

Commit 03d331c

Browse files
authored
fix: enforce tabsheet panels to be hidden (#10002) (#10005)
1 parent d4252b3 commit 03d331c

File tree

3 files changed

+17
-0
lines changed

3 files changed

+17
-0
lines changed

packages/tabsheet/src/vaadin-lit-tabsheet.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,10 @@ class TabSheet extends TabSheetMixin(ThemableMixin(ElementMixin(PolylitMixin(Lit
4949
min-width: 8em;
5050
}
5151
52+
::slotted([hidden]) {
53+
display: none !important;
54+
}
55+
5256
[part='content'] {
5357
position: relative;
5458
flex: 1;

packages/tabsheet/src/vaadin-tabsheet.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,10 @@ class TabSheet extends TabSheetMixin(ThemableMixin(ElementMixin(ControllerMixin(
8686
min-width: 8em;
8787
}
8888
89+
::slotted([hidden]) {
90+
display: none !important;
91+
}
92+
8993
[part='content'] {
9094
position: relative;
9195
flex: 1;

packages/tabsheet/test/tabsheet.test.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -162,6 +162,15 @@ describe('tabsheet', () => {
162162
expect(getPanels()[2].hidden).to.be.true;
163163
});
164164

165+
it('should override display style when panel is hidden', async () => {
166+
tabsheet.selected = 1;
167+
await nextFrame();
168+
const panel2 = getPanels()[2];
169+
panel2.style.display = 'contents';
170+
const display = getComputedStyle(panel2).display;
171+
expect(display).to.equal('none');
172+
});
173+
165174
it('should bind dynamically added tab and panel', async () => {
166175
// Create a new tab and panel
167176
const tab = document.createElement('vaadin-tab');

0 commit comments

Comments
 (0)