-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/combobox #173
base: main
Are you sure you want to change the base?
Feat/combobox #173
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…ocationSelect to the dialog map
@@ -76,7 +44,7 @@ export const LocationSelect = ({ field }: LocationSelectProps) => { | |||
<div style={{ minHeight: 200, height: 200 }}> | |||
<LocationMap /> | |||
</div> | |||
<Paragraph>{address}</Paragraph> | |||
<Paragraph>{formStoreState.address?.weergave_naam}</Paragraph> | |||
<MapProvider> | |||
<MapDialog |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wil je gelijk ervoor zorgen dat je met tab niet meer op de kaart terecht komt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dit is nog niet opgelost
@@ -1,7 +1,7 @@ | |||
import { LocationMap } from '@/components/ui/LocationMap' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Besproken: zorgen dat de knop "verder zonder object selectere" alleen zichtbaar is wanneer er een adres is ingevuld anders werkt het wat verwarrend
} | ||
|
||
// Handle click on feature marker, set selectedFeatures and show error if maxNumberOfAssets is reached | ||
const handleFeatureMarkerClick = (event: MarkerEvent, feature: Feature) => { | ||
const handleFeatureMarkerClick = async ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wanneer je in stap 2 een adres hebt geselecteerd, vervolgens het formulier doorwerkt en vanuit stap 4 terug navigeert naar stap 2 en de kaart opent is de marker niet meer te zien.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gaat ook fout bij:
stap 1: lantaarn kapot
stap 2: vul adres in input veld -> verschijnt marker -> klik op kaart om objecten te selecteren -> marker weg
|
||
return [] | ||
}, [formState.selectedFeatures, mapFeatures?.features, dialogMap?.getZoom()]) | ||
|
||
return ( | ||
<Dialog.Root> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wanneer je met de cursor over de muis gaat verandert deze in een handje. Dit doet suggereren dat je de kaart kan bewegen, echter kan dit niet
@@ -115,20 +160,36 @@ const MapDialog = ({ | |||
|
|||
// Handle click on map, setIsMapSelected to true | |||
// TODO: Reset selectedFeatures if click was right on map? (open for discussion) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wat wordt hiermee bedoeld eigenlijk?
onChange={onChangeAddress} | ||
onClose={() => setQuery('')} | ||
> | ||
<ComboboxInput |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
onClose={() => setQuery('')} | ||
> | ||
<ComboboxInput | ||
aria-label="Adres" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wanneer je op enter drukt terwijl het input veld actief is/in focus wordt de submit getriggered
@@ -37,6 +37,7 @@ import { Feature, FeatureCollection } from 'geojson' | |||
import { PublicQuestion } from '@/types/form' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ik vind het denk ik verwarrend dat het geselecteerde adres versprint wanneer je objecten selecteert op de kaart bij de asset select
Integrates combobox with default utrecht combobox styling, next to that also a small refactor of MapDialog related code.