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

Update common_scripts.blade.php (jqueryui/1.10.4 to jqueryui/1.13.1) #23

Closed
wants to merge 1 commit into from

Conversation

AfzalSabbir
Copy link

jqueryui/1.10.4 to jqueryui/1.13.1

Because some of designs were breaking which solved in this new version and elFinder itself used this new jQueryUI.

WHY

BEFORE - What was wrong? What was happening before this PR?

?? Some of designs were breaking ( https://prnt.sc/OtGVbr3V1Gnc )

AFTER - What is happening after this PR?

?? This type of breaking are solved ( https://prnt.sc/ynytaD8uXQSu )

HOW

How did you achieve that, in technical terms?

?? Just changed from jqueryui/1.10.4 to jqueryui/1.13.1

Is it a breaking change or non-breaking change?

?? Yes, it was a large issue, without that we could not close or reopen (pop up) the area ever again!

How can we test the before & after?

?? Just change the version in file \resources\views\common_scripts.blade.php

jqueryui/1.10.4 to jqueryui/1.13.1

Because some of designs were breaking which solved in this new version and elFinder itself used this new jQueryUI.
@AfzalSabbir AfzalSabbir changed the title Update common_scripts.blade.php Update common_scripts.blade.php (jqueryui/1.10.4 to jqueryui/1.13.1) Jun 16, 2022
@promatik
Copy link
Member

Hi @AfzalSabbir!
Thank you for your PR and sorry for the late response.

These updates are going to be merged on the new filemanager version which will be supported by bassets 👌
The PR for it is here; #25
I'll close this one for now.

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

Successfully merging this pull request may close these issues.

None yet

2 participants