diff --git a/.changeset/tiny-crews-kneel.md b/.changeset/tiny-crews-kneel.md new file mode 100644 index 0000000000..de0685f3d0 --- /dev/null +++ b/.changeset/tiny-crews-kneel.md @@ -0,0 +1,7 @@ +--- +'@primer/view-components': patch +--- + +Update popover support in older browsers. + + diff --git a/app/components/primer/alpha/overlay.pcss b/app/components/primer/alpha/overlay.pcss index fac1343e97..bca8a381eb 100644 --- a/app/components/primer/alpha/overlay.pcss +++ b/app/components/primer/alpha/overlay.pcss @@ -26,10 +26,3 @@ anchored-position.not-anchored::backdrop { display: revert; } } - -/* This reverts the declaration above for native popover, where `:open` is supported (Chrome 113, Safari TP) */ -@supports selector(:open) { - anchored-position[popover]:not(.\:popover-open) { - display: revert; - } -} diff --git a/package-lock.json b/package-lock.json index a8d3aa356d..9e27c2d65a 100644 --- a/package-lock.json +++ b/package-lock.json @@ -18,7 +18,7 @@ "@github/include-fragment-element": "^6.1.1", "@github/relative-time-element": "^4.0.0", "@github/tab-container-element": "^3.1.2", - "@oddbird/popover-polyfill": "^0.2.1", + "@oddbird/popover-polyfill": "^0.2.3", "@primer/behaviors": "^1.3.4" }, "devDependencies": { @@ -1559,9 +1559,9 @@ } }, "node_modules/@oddbird/popover-polyfill": { - "version": "0.2.2", - "resolved": "https://registry.npmjs.org/@oddbird/popover-polyfill/-/popover-polyfill-0.2.2.tgz", - "integrity": "sha512-ko7x+PDZA9bHwA6hSfxjL1IhBP91JukfZq/NAe85u9rT0akFn9RKvSXymX/mS7S2mfNjE+Zw9JdLUPGvPabQAA==" + "version": "0.2.3", + "resolved": "https://registry.npmjs.org/@oddbird/popover-polyfill/-/popover-polyfill-0.2.3.tgz", + "integrity": "sha512-XDK+V/gUeE4NEsWp79eVzhlK3wuVcRDJuaas63qo0IJLJpyOLHqycJLFYvuq8kebgT1nl87P3sbSb5ZN6Vyf5g==" }, "node_modules/@playwright/test": { "version": "1.35.1", @@ -11377,9 +11377,9 @@ } }, "@oddbird/popover-polyfill": { - "version": "0.2.2", - "resolved": "https://registry.npmjs.org/@oddbird/popover-polyfill/-/popover-polyfill-0.2.2.tgz", - "integrity": "sha512-ko7x+PDZA9bHwA6hSfxjL1IhBP91JukfZq/NAe85u9rT0akFn9RKvSXymX/mS7S2mfNjE+Zw9JdLUPGvPabQAA==" + "version": "0.2.3", + "resolved": "https://registry.npmjs.org/@oddbird/popover-polyfill/-/popover-polyfill-0.2.3.tgz", + "integrity": "sha512-XDK+V/gUeE4NEsWp79eVzhlK3wuVcRDJuaas63qo0IJLJpyOLHqycJLFYvuq8kebgT1nl87P3sbSb5ZN6Vyf5g==" }, "@playwright/test": { "version": "1.35.1", diff --git a/package.json b/package.json index 44b2e714df..6ffcc28cc3 100644 --- a/package.json +++ b/package.json @@ -51,7 +51,7 @@ "@github/include-fragment-element": "^6.1.1", "@github/relative-time-element": "^4.0.0", "@github/tab-container-element": "^3.1.2", - "@oddbird/popover-polyfill": "^0.2.1", + "@oddbird/popover-polyfill": "^0.2.3", "@primer/behaviors": "^1.3.4" }, "devDependencies": { @@ -63,6 +63,7 @@ "@playwright/test": "^1.35.1", "@primer/css": "21.0.2", "@primer/primitives": "7.12.0", + "@primer/stylelint-config": "^12.7.2", "@rollup/plugin-node-resolve": "^13.3.0", "@rollup/plugin-typescript": "^8.3.3", "@typescript-eslint/eslint-plugin": "^5.31.0", @@ -90,7 +91,6 @@ "stylelint": "^14.13.0", "stylelint-config-prettier": "^9.0.3", "stylelint-config-standard": "^28.0.0", - "@primer/stylelint-config": "^12.7.2", "tslib": "^2.4.0", "typescript": "^4.7.4" },