-
Notifications
You must be signed in to change notification settings - Fork 5.3k
jQuery UI Dialog content shrinks when continuously resizing #2083
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
Comments
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. |
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: |
@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. |
Fantastic! My team will be glad this will be fixed for our application. Thanks @mgol! |
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. |
@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 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. |
@mgol I appreciate your efforts. 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 |
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. |
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). |
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. |
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
The text was updated successfully, but these errors were encountered: