Skip to content

Commit

Permalink
- Linting (ESLint compat): Add eslint-plugin-compat to get browser su…
Browse files Browse the repository at this point in the history
…pport

    warnings
- Docs (README): Indicate minimal polyfills needed for older browsers
    (IE <= 11, IE Mobile, Opera Mini, Blackberry Browser <= 10,
    Android Browser 4.4.3-4.4.4)
- npm: Update devDeps
  • Loading branch information
brettz9 committed Oct 29, 2018
1 parent 14aad38 commit 9b9c900
Show file tree
Hide file tree
Showing 5 changed files with 369 additions and 356 deletions.
10 changes: 8 additions & 2 deletions .eslintrc
Original file line number Diff line number Diff line change
@@ -1,13 +1,19 @@
{
"extends": ["standard", "plugin:qunit/recommended", "plugin:testcafe/recommended"],
"extends": [
"standard", "plugin:compat/recommended",
"plugin:qunit/recommended", "plugin:testcafe/recommended"
],
"parserOptions": {
"sourceType": "module"
},
"plugins": ["qunit", "testcafe"],
"plugins": ["compat", "qunit", "testcafe"],
"env": {
"node": false,
"browser": true
},
"settings": {
"polyfills": ["url", "promises", "fetch", "queryselector"]
},
"rules": {
"semi": [2, "always"],
"indent": ["error", 2, {"outerIIFEBody": 0}],
Expand Down
6 changes: 6 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,13 @@
# ?

- Linting (ESLint compat): Add eslint-plugin-compat to get browser support
warnings
- Docs (README): Indicate minimal polyfills needed for older browsers
(IE <= 11, IE Mobile, Opera Mini, Blackberry Browser <= 10,
Android Browser 4.4.3-4.4.4)
- Docs (3.0.0): Clarify
- Docs (ReleaseInstructions): Clarify need for npm permissions to publish
- npm: Update devDeps

# 3.1.1

Expand Down
15 changes: 15 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,21 @@ For testing the latest version in `master`, you may use
<iframe src="svgedit/editor/svg-editor.html?extensions=" width="100%" height="100%"></iframe>
```

Note that if you want support for the following browsers, you will at least
need some polyfills.

For Android Browser 4.4.3-4.4.4, you will need at least `fetch`.

For the following, you will need at least `URL`, `Promise`, and `fetch`:

- IE <= 11
- IE Mobile
- Opera Mini
- Blackberry Browser <= 10

And for still older browsers (e.g., IE 8), you will at minimum need a
`querySelector` polyfill.

### Integrating SVG-edit into your own npm package

These steps are only needed if you wish to set up your own npm package
Expand Down
Loading

0 comments on commit 9b9c900

Please sign in to comment.