jquery / jquery-ui

The official jQuery user interface library.
https://jqueryui.com
Other
11.26k stars 5.33k forks source link

jQuery UI Dialog content shrinks when continuously resizing #2083

Closed Josh-Reichman closed 1 year ago

Josh-Reichman commented 2 years ago

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 https://github.com/jquery/jquery-ui/issues/1979 but it appears that issue hasn't been updated since last year

Josh-Reichman commented 2 years ago

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 commented 2 years ago

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 commented 1 year ago

@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 commented 1 year ago

@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.

Josh-Reichman commented 1 year ago

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

slominskir commented 10 months ago

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 commented 10 months ago

@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 commented 10 months ago

@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 commented 10 months ago

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 commented 10 months ago

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 commented 10 months ago

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.