jquery-ui icon indicating copy to clipboard operation
jquery-ui copied to clipboard

Jquery Dialog content shrinks when continuously resizing

Open Josh-Reichman opened this issue 3 years ago • 2 comments

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 avatar Jun 03 '22 20:06 Josh-Reichman

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.

Josh-Reichman avatar Jun 03 '22 20:06 Josh-Reichman

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/

mgol avatar Jun 22 '22 10:06 mgol

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

Josh-Reichman avatar Oct 27 '22 20:10 Josh-Reichman

@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 avatar Nov 18 '22 12:11 mgol

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

Josh-Reichman avatar Nov 18 '22 14:11 Josh-Reichman

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 avatar Dec 14 '23 16:12 slominskir

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

mgol avatar Dec 14 '23 16:12 mgol

@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

slominskir avatar Dec 14 '23 17:12 slominskir

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.

mgol avatar Dec 14 '23 18:12 mgol

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

slominskir avatar Dec 18 '23 15:12 slominskir

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.

mgol avatar Dec 18 '23 17:12 mgol