Widget: Internal list of elements with classes can't be cleaned up

From the demo page: "This is not a supported or even complete widget."

It's not the widget that is the problem. It is the code behind it that duplicates the data. Did you read what I said at all?

Yes, I read what you wrote. I don't see how the code could get up to 200,000 elements from 3,000. It does one concatenation, and then removes the duplicates, so the max count should never be more than 6,000. But even that number should never be reached. Because you've said that there are many interactions involved, and each interaction is actually its own completely separate operation with a new list each time, I don't see the correlation. Your stack trace shows processClassString() being called once and that doesn't provide any information about the number of elements.

Unless you can provide a concrete way for us to see that there is a problem, there's not much we can do.

I created a video on how to reproduce the issue with the jquery autocomplete example. I have significantly more import options than what the example has.

https://www.youtube.com/watch?v=9zNP5DsyKqA

This is the chunk of code I have found so far where the ui-menu-items are added

		// Don't refresh list items that are already adapted
		newItems = items.not( ".ui-menu-item, .ui-menu-divider" );
		newWrappers = newItems.children()
			.not( ".ui-menu" )
				.uniqueId()
				.attr( {
					tabIndex: -1,
					role: this._itemRole()
				} );
		this._addClass( newItems, "ui-menu-item" )
			._addClass( newWrappers, "ui-menu-item-wrapper" );

I can see them adding to the list in newItems

Ok, now I see what's happening. The widget keeps track of the element that have classes applied, but in the case of autocomplete, the menu is constantly rebuilt, so the list grows with every render. Widgets need a way to signify that elements have been removed and should be taken out of the list.

I reverted jquery UI JS to 1.9 and this problem does not exist. Somewhere between 1.9 and 1.12 it is not working correctly.

That somewhere is 1.12.0, which is why I corrected the version field.

In [changeset:"89af4c292eaa5fc1c83437ca71085264dcbef34a" 89af4c2]:

#!CommitTicketReference repository="" revision="89af4c292eaa5fc1c83437ca71085264dcbef34a"
Widget: Untrack classes elements when they are removed from the DOM

Fixes #15043
Closes gh-1744