Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#13685: Upgrading jQuery to 3.3.1 #15531

Closed
wants to merge 17 commits into from

Conversation

kirmorozov
Copy link
Member

@kirmorozov kirmorozov commented May 26, 2018

  • Updating jQuery to 3.3.1 (originally from npm)
  • Updating jQuery Migrate 3.0.1 (originally from npm)
  • jQuery Mobile 1.5.0-alpha.1, generated online

Description

Upgrading initial jQuery and jQuery Migrate

In previous series:
#14556
#14637

Fixed Issues (if relevant)

  1. Incompatibilies with jQuery 3 #13685: Incompatibilies with jQuery 3
  2. ...

Manual testing scenarios

  1. I can checkout without any problems
  2. I can use admin without any problems

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

* Updating jQuery to 3.3.1 (originally from npm)
* Updating jQuery Migrate 3.0.1 (originally from npm)
* Test compatibility for with jQuery 3.x
* Updated jQuery UI to 1.12
+ moved plugin zIndex from previous version
* Fix ajax success in tests.
* Fixing Success/Error as towards done/fail in jQuery 3.x
* File upload success/fail
* Update jQuery UI for backend.
@kirmorozov kirmorozov changed the title 2.3 develop jquery3 #13685: Upgrading jQuery to 3.3.1 May 26, 2018
@magento-engcom-team
Copy link
Contributor

@kirmorozov Closing and reopening Pull Request in order to trigger Travis CI build on it.

@kirmorozov
Copy link
Member Author

Work in progress, resolving incompatibility problems with tabs in admin after jQuery-ui update.
from 1.9 to 1.12.

@kirmorozov
Copy link
Member Author

@Igloczek Hi fellow, how many reviewers must approve to have it merged into the core?

@Igloczek
Copy link
Contributor

@kirmorozov It's not defined, only community engineering team decide what should be merged.
The community can just give them advice if something looks good or not.

@kirmorozov
Copy link
Member Author

@piotrekkaminski Can you merge it earlier to make sure vendors are compatible when release date comes?

…uery3

# Conflicts:
#	lib/web/mage/translate-inline.js
Revert "MAGETWO-95280: Storefront Images Don't Scale Back To Original Size In Mobile"

This reverts commit 13d360d
@kirmorozov
Copy link
Member Author

OK, there was only one bug:
MAGETWO-95280: Storefront Images Don't Scale Back To Original Size In Mobile
Easy one to kill.

@ihor-sviziev
Copy link
Contributor

Hi,
Do you think it's a good idea to use "alpha" version of jquery mobile? I think it's better to use latest stable version

@sivaschenko
Copy link
Member

The pull request was discussed on architecture meeting and it was decided to include it in major 2.4 release magento/architecture#59 (comment)

@sivaschenko sivaschenko self-assigned this Jan 25, 2019
@okorshenko okorshenko removed this from the Release: 2.3.1 milestone Jan 28, 2019
@kirmorozov
Copy link
Member Author

@ihor-sviziev As of jquery mobile, you can check difference yourself, no changes since forever.
jquery-archive/jquery-mobile@1.5.0-alpha.1...1.5.0-rc1

@kirmorozov
Copy link
Member Author

Found issue with tabs in admin, resolved here for 2.3.1
https://github.com/morozov-group/magento2-edge/tree/2.3.1-jquery3
Feel free to try on your local

curl https://github.com/magento/magento2/compare/2.3.1...morozov-group:2.3.1-jquery3.patch |\
    sed 's#/app/code/Magento/Catalog/#/vendor/magento/module-catalog/#g' |\
    sed 's#/app/code/Magento/ConfigurableProduct/#/vendor/magento/module-configurable-product/#g' |\
    sed 's#/app/code/Magento/ProductVideo/#/vendor/magento/module-product-video/#g' |\
    sed 's#/app/code/Magento/Theme/#/vendor/magento/module-theme/#g' |\
    sed 's#/app/code/Magento/Ui/#/vendor/magento/module-ui/#g' |\
    sed 's#/app/code/Magento/User/#/vendor/magento/module-user/#g' > _jquery3.patch
patch -p1 --forward < _jquery3.patch

@sidolov sidolov changed the base branch from 2.3-develop to 2.4-develop December 5, 2019 16:01
Copy link
Contributor

@ihor-sviziev ihor-sviziev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @kirmorozov,
Finally branch 2.4-develop was created, now your PR could be merged! Could you resolve merge conflicts and update to the latest versions.

@ghost ghost assigned ihor-sviziev Dec 5, 2019
@ihor-sviziev ihor-sviziev mentioned this pull request Dec 5, 2019
4 tasks
@BassemN
Copy link

BassemN commented Dec 6, 2019

@kirmorozov please upgrade to jQuery 3.4.1

Copy link
Contributor

@krzksz krzksz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that since jQuery UI modules are now divided into separate files as of a5c6783, this PR has to be reworked.

@ghost ghost assigned krzksz Dec 7, 2019
@ihor-sviziev
Copy link
Contributor

Hi @kirmorozov,
Are you interested in finalizing this pull request?
It looks really amazing!

@ihor-sviziev
Copy link
Contributor

@kirmorozov Unfortunately I need to close this PR now due to inactivity.
Please reopen and update if you wish to continue.
Thank you for the collaboration!

@m2-assistant
Copy link

m2-assistant bot commented Feb 25, 2020

Hi @kirmorozov, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants