[@mantine/core] Fix searchable select with controlled value

[wuifdesign]

fixes #4909

optionsLockup needs to be memorized as they are used in a effect further down. otherwise the effect will be triggered each render - preventing search changes to take effect.

mantine/src/mantine-core/src/components/Select/Select.tsx

Lines 179 to 187 in 7dc4a72

	useEffect(() => {
	if (value === null) {
	setSearch('');
	}
	if (typeof value === 'string' && optionsLockup[value]) {
	setSearch(optionsLockup[value].label);
	}
	}, [value, optionsLockup]);

[wuifdesign]

Other ideas to fix this, i can do so if wanted:

• rework optionsLockup to useRef so it will not trigger the useEffect
• rework the useEffect -> this may have the effect that external value changes are not displayed correctly

@rtivital

[rtivital]

I think current solution is perfectly fine, thanks!