@justeattakeaway/pie-modal 0.16.0 → 0.17.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.eslintignore +0 -1
- package/.turbo/turbo-build.log +5 -5
- package/CHANGELOG.md +10 -0
- package/README.md +14 -0
- package/dist/index.js +961 -567
- package/dist/react.js +0 -0
- package/dist/types/index.d.ts +0 -0
- package/dist/types/packages/components/pie-modal/src/defs.d.ts +0 -0
- package/dist/types/packages/components/pie-modal/src/defs.d.ts.map +0 -0
- package/dist/types/packages/components/pie-modal/src/index.d.ts +0 -0
- package/dist/types/packages/components/pie-modal/src/index.d.ts.map +1 -1
- package/dist/types/packages/components/pie-modal/src/react.d.ts +0 -0
- package/dist/types/packages/components/pie-modal/src/react.d.ts.map +0 -0
- package/dist/types/react.d.ts +0 -0
- package/package.json +4 -1
- package/src/index.ts +10 -5
- package/src/modal.scss +1 -0
package/.eslintignore
CHANGED
package/.turbo/turbo-build.log
CHANGED
|
@@ -1,14 +1,14 @@
|
|
|
1
|
-
[3:
|
|
1
|
+
[3:19:34 PM] @custom-elements-manifest/analyzer: Created new manifest.
|
|
2
2
|
react wrapper has been added!
|
|
3
3
|
[36mvite v4.3.9 [32mbuilding for production...[36m[39m
|
|
4
4
|
transforming...
|
|
5
|
-
[32m✓[39m
|
|
5
|
+
[32m✓[39m 28 modules transformed.
|
|
6
6
|
rendering chunks...
|
|
7
7
|
computing gzip size...
|
|
8
|
-
[2mdist/[22m[36mindex.js [39m[1m[2m54.46 kB[22m[1m[22m[2m │ gzip: 14.59 kB[22m
|
|
9
8
|
[2mdist/[22m[36mreact.js [39m[1m[2m59.27 kB[22m[1m[22m[2m │ gzip: 15.99 kB[22m
|
|
9
|
+
[2mdist/[22m[36mindex.js [39m[1m[2m70.71 kB[22m[1m[22m[2m │ gzip: 19.04 kB[22m
|
|
10
10
|
[32m
|
|
11
11
|
[36m[vite:dts][32m Start generate declaration files...[39m
|
|
12
|
-
[32m✓ built in
|
|
13
|
-
[32m[36m[vite:dts][32m Declaration files built in
|
|
12
|
+
[32m✓ built in 12.06s[39m
|
|
13
|
+
[32m[36m[vite:dts][32m Declaration files built in 10809ms.
|
|
14
14
|
[39m
|
package/CHANGELOG.md
CHANGED
|
@@ -1,5 +1,15 @@
|
|
|
1
1
|
# @justeattakeaway/pie-modal
|
|
2
2
|
|
|
3
|
+
## 0.17.0
|
|
4
|
+
|
|
5
|
+
### Minor Changes
|
|
6
|
+
|
|
7
|
+
- - [Changed] - updated Modal component dependencies adding the `dialog-polyfill` package ([#678](https://github.com/justeattakeaway/pie/pull/678)) by [@fernandofranca](https://github.com/fernandofranca)
|
|
8
|
+
|
|
9
|
+
- [Changed] - updated Modal component to use `dialog-polyfill`
|
|
10
|
+
- [Changed] - updated Modal documentation regarding the legacy browser support for Dialog element and polyfill usage and limitations
|
|
11
|
+
- [Changed] - vanilla example app for testing Modal with `dialog-polyfill`
|
|
12
|
+
|
|
3
13
|
## 0.16.0
|
|
4
14
|
|
|
5
15
|
### Minor Changes
|
package/README.md
CHANGED
|
@@ -15,6 +15,7 @@
|
|
|
15
15
|
3. [Importing the component](#importing-the-component)
|
|
16
16
|
4. [Props](#props)
|
|
17
17
|
5. [Testing](#testing)
|
|
18
|
+
5. [Legacy browser support](#legacy-browser-support)
|
|
18
19
|
|
|
19
20
|
# pie-modal
|
|
20
21
|
|
|
@@ -118,3 +119,16 @@ Under scripts `test:visual` replace the environment variable with the below:
|
|
|
118
119
|
```bash
|
|
119
120
|
PERCY_TOKEN_PIE_MODAL=abcde
|
|
120
121
|
```
|
|
122
|
+
|
|
123
|
+
## Legacy browser support
|
|
124
|
+
|
|
125
|
+
`pie-modal` uses the Dialog element which might not be supported by legacy browsers.
|
|
126
|
+
|
|
127
|
+
To support them, `pie-modal` uses the [dialog-polyfill](https://github.com/GoogleChrome/dialog-polyfill) package. It works automatically and doesn't need any setup.
|
|
128
|
+
|
|
129
|
+
The polyfill comes with a few limitations, as noted on its [documentation page](https://github.com/GoogleChrome/dialog-polyfill#limitations):
|
|
130
|
+
- Dialogs should not be contained by parents that create a stacking context
|
|
131
|
+
- The browser's chrome may not always be accessible via the tab key
|
|
132
|
+
- Changes to the CSS top/bottom values while open aren't retained
|
|
133
|
+
|
|
134
|
+
For more details, check the package documentation mentioned above.
|