Skip to content

Commit 8a7d583

Browse files
authored
chore(PageHeader): Remove the CSS modules feature flag from the PageHeader component (#5877)
1 parent 365d53d commit 8a7d583

File tree

3 files changed

+168
-450
lines changed

3 files changed

+168
-450
lines changed

.changeset/tough-dodos-retire.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@primer/react': minor
3+
---
4+
5+
Remove the CSS modules feature flag from the PageHeader component

packages/react/src/PageHeader/PageHeader.test.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ describe('PageHeader', () => {
4545
* They are testing the internal implementation of the component and checking if the component
4646
* is rendering the correct styles.This approach was necessary due to the impracticality of CSS media queries testing with Jest.
4747
*/
48-
it('respects default visibility of ContextArea and renders CSS media styles correctly', () => {
48+
it.skip('respects default visibility of ContextArea and renders CSS media styles correctly', () => {
4949
const expectedStyles = {
5050
'-ms-flex-align': 'center',
5151
'-ms-flex-direction': 'row',
@@ -71,7 +71,7 @@ describe('PageHeader', () => {
7171
expect.objectContaining(expectedStyles),
7272
)
7373
})
74-
it('respects the hidden prop of ContextArea and renders CSS media styles correctly', () => {
74+
it.skip('respects the hidden prop of ContextArea and renders CSS media styles correctly', () => {
7575
const expectedStyles = {
7676
'-ms-flex-align': 'center',
7777
'-ms-flex-direction': 'row',
@@ -110,7 +110,7 @@ describe('PageHeader', () => {
110110
),
111111
).toEqual(expect.objectContaining(expectedStyles))
112112
})
113-
it('respects default visibility of LeadingAction and TrailingAction and renders CSS media styles correctly', () => {
113+
it.skip('respects default visibility of LeadingAction and TrailingAction and renders CSS media styles correctly', () => {
114114
const expectedStyles = {
115115
'-ms-flex-align': 'center',
116116
'-webkit-align-items': 'center',

0 commit comments

Comments
 (0)