remove the useMemo hook from CitySelectionDialog

This commit is contained in:
aryabenade
2026-03-20 11:49:11 +05:30
parent a09d53db7d
commit 0a60ba58a3

View File

@@ -53,14 +53,6 @@ export function CitySelectionDialog({
setSearch(e.target.value)
}
const filteredCities = useMemo(() =>
cities?.filter((city: City) =>
city.cityName.toLowerCase().includes(search.toLowerCase())
) ?? [],
[cities, search]
);
return (
<Dialog open={isOpen} onOpenChange={onClose}>
<DialogContent className="max-w-md w-full p-0 gap-0 font-poppins">
@@ -99,7 +91,7 @@ export function CitySelectionDialog({
<div className="px-6 pb-6 max-h-[60vh] overflow-y-auto">
<AnimatePresence>
<div className="grid grid-cols-2 gap-3">
{filteredCities && filteredCities.map((city: City) => (
{cities && cities.map((city: City) => (
<motion.button
key={city.id}
onClick={() => handleCityClick(city)}
@@ -126,7 +118,7 @@ export function CitySelectionDialog({
</div>
</AnimatePresence>
{filteredCities?.length === 0 && (
{cities?.length === 0 && (
<div className="text-center py-8">
<p className="text-gray-500 font-poppins">
No cities found matching "{search}"