[#33084] - [4.0] ES5 scripts with included polyfills
- Closed
- 17 Apr 2021
- Medium
- Build: 4.0-dev
- # 33084
- Diff
- dgrammatiko:—fix-the-es5-scripts-4.0-dev
User tests: Successful: Unsuccessful:
Pull Request for Issue #25639 .
Summary of Changes
When I refactored the build tools I left the ES5 babel config basically unconfigured. The reason was I was expecting that we could come up with an action plan (eg a broader talk about polyfills, #32315 (comment)). This obviously didn't happen and so here's the missing configuration.
Testing Instructions
Apply the branch
Run npm i
CHECK WITH IE and report any console errors.
Attention there are known missing polyfills that this PR IS NOT addressing. This PR just ensures modern code is transpilled so IE can run it (this includes also a gazillion of polyfills but there are many DOM related parts that are not polyfillable from babel)
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Documentation Changes Required
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript Repository NPM Change |
well, let's forget about IE 11
I tested, but there bunch of syntax error, because arrow function and stuff, because of es6
I not sure it worth the time that much
well, let's forget about IE 11
Actually, you're right, IE will be dead in 4 months so there's not point https://death-to-ie11.com
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-04-17 22:17:23 |
| Closed_By | ⇒ | dgrammatiko | |
| Labels |
Added:
NPM Resource Changed
?
|
||
@Fedik can you test this one with IE11 and report back the errors logged?