Fix: Mouseup event was not being removed from input element for search, resulting in a memory leak

AllanJard · AllanJard · commit 7050f36c9c7e · 2023-05-16T11:17:48.000+01:00
https://datatables.net/forums/discussion/76200
diff --git a/js/core/core.filter.js b/js/core/core.filter.js
@@ -63,7 +63,7 @@ function _fnFeatureHtmlFilter ( settings )
 				_fnThrottle( searchFn, searchDelay ) :
 				searchFn
 		)
-		.on( 'mouseup', function(e) {
+		.on( 'mouseup.DT', function(e) {
 			// Edge fix! Edge 17 does not trigger anything other than mouse events when clicking
 			// on the clear icon (Edge bug 17584515). This is safe in other browsers as `searchFn`
 			// checks the value to see if it has changed. In other browsers it won't have.

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ function _fnFeatureHtmlFilter ( settings )`
`63`	`63`	`_fnThrottle( searchFn, searchDelay ) :`
`64`	`64`	`searchFn`
`65`	`65`	`)`
`66`		`- .on( 'mouseup', function(e) {`
	`66`	`+ .on( 'mouseup.DT', function(e) {`
`67`	`67`	`// Edge fix! Edge 17 does not trigger anything other than mouse events when clicking`
`68`	`68`	// on the clear icon (Edge bug 17584515). This is safe in other browsers as `searchFn`
`69`	`69`	`// checks the value to see if it has changed. In other browsers it won't have.`