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

jQuery UI Dialog content shrinks when continuously resizing #2083

Closed
Josh-Reichman opened this issue Jun 3, 2022 · 11 comments
Closed

jQuery UI Dialog content shrinks when continuously resizing #2083

Josh-Reichman opened this issue Jun 3, 2022 · 11 comments

Comments

@Josh-Reichman
Copy link

Josh-Reichman commented Jun 3, 2022

When reducing the size of a dialog when the content has a scroll bar, it continuously shrinks each time you click (as in click down, move, click up, repeat). I've tested this on Chrome (102.0.5005.63), Edge (102.0.1245.33), and Firefox (101.0). Firefox works fine while Chrome and Edge do not.

I've attached a video with the bug
https://user-images.githubusercontent.com/9708451/171943601-c9fc6e38-cb7a-47fa-881a-1999e4b08748.mp4

I can try to give more info if needed. I'm unsure if this is related to #1979 but it appears that issue hasn't been updated since last year

@Josh-Reichman
Copy link
Author

I'm not sure what jQuery and jQuery ui versions are being used on the jQuery ui site but I can confirm this is an issue when using jquery ui 1.13.1 and jquery core 3.6.0 for my application I am working on.

@mgol
Copy link
Member

mgol commented Jun 22, 2022

Thanks for the report. This looks like a 1.13 regression as I can't reproduce the behavior on an older version of the website that used 1.12.1:
https://web.archive.org/web/20190308021648/https://jqueryui.com/dialog/
It's not just a Web Archive thing as the archived latest version of the site using 1.13.1 has the issue:
https://web.archive.org/web/20220622032021/https://jqueryui.com/dialog/

@Josh-Reichman
Copy link
Author

@mgol I saw #1979 was updated with a commit earlier this month. Do you know if it fixed this issue? If it did, this issue can be closed.

@mgol mgol changed the title Jquery Dialog content shrinks when continuously resizing Oct 28, 2022
@mgol
Copy link
Member

mgol commented Nov 18, 2022

@Josh-Reichman Yes, it looks like it was fixed by #1979, see: https://jsfiddle.net/m_gol/Lgnfyuk4/3/

Thanks for the note! The fix will be included in jQuery UI 1.13.3.

@mgol mgol closed this as completed Nov 18, 2022
@Josh-Reichman
Copy link
Author

Fantastic! My team will be glad this will be fixed for our application. Thanks @mgol!

@slominskir
Copy link

When is jQuery UI 1.13.3 being released? This bug exists in the latest version 1.13.2 and there hasn't been a new version in well over a year.

@mgol
Copy link
Member

mgol commented Dec 14, 2023

@slominskir jQuery UI is in maintenance mode (see the project status at https://blog.jqueryui.com/2021/10/jquery-maintainers-update-and-transition-jquery-ui-as-part-of-overall-modernization-efforts/) so the activity is limited. You can help getting the release out the door quicker by submitting PRs for open 1.13.3 issues and by submitting fixes for current test failures - see the red entries at https://swarm.jquery.org/project/jqueryui and the Next page; everything run last Sunday / Monday.

Otherwise, you need to wait until I have time to address those issues. Considering my main focus right now is the jQuery 4.0.0 release, it may take a while.

@slominskir
Copy link

@mgol I appreciate your efforts.

dependency
https://xkcd.com/2347/

A shame a single IE issue is holding up the release (should IE be dropped in newer releases?). Also, the release process must be very onerous to sit on a fix for a regression that has been already solved (release more often?). Looks like the fiddle above uses a pre-release that might resolve this though: https://releases.jquery.com/git/ui/jquery-ui-git.js

@mgol
Copy link
Member

mgol commented Dec 14, 2023

It's not just a single IE issue; there are some test failures with some browser/jQuery combinations that you can see at https://swarm.jquery.org/job/14283; this is the bigger issue that I think absolutely need solving before a release.

@slominskir
Copy link

Presumably a pull request is not merged unless it passes the tests so I'm assuming the test failures are created each time a new version of a browser is released things break? Seems like incrementally adding value to the project (with a release containing a subset of all possible fixes) would be a reasonable approach (especially considering the latest stable version has at least the regression mention in this ticket and bug free releases are impractical).

@mgol
Copy link
Member

mgol commented Dec 18, 2023

Presumably a pull request is not merged unless it passes the tests

We are not running all tests with all supported jQuery versions against all supported browsers in the PR builder; that's a complicated task and we're not going to achieve it here anymore.

Basic testing is done but that's mostly Chromium tests against a fixed jQuery version.

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