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
rg/32 report incident form #35
Conversation
Nicely done |
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.
nicely done !
zoom={12} | ||
zoomControl={false} | ||
className="min-h-screen w-full" | ||
> | ||
<SetViewOnUserLocation /> |
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.
Maybe in the future, we can move this out to it's own component
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.
Like create a new file for it? Instead of colocating it with Map?
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.
yeah I guess so but we can refactor later thanks!
packages/client/lib/components/ReportForm/FormViews/LocationView.jsx
Outdated
Show resolved
Hide resolved
packages/client/lib/components/ReportForm/FormViews/LocationView.jsx
Outdated
Show resolved
Hide resolved
packages/client/lib/components/ReportForm/NavigationButtons/Submit.jsx
Outdated
Show resolved
Hide resolved
packages/client/lib/components/ReportForm/NavigationButtons/GoBack.jsx
Outdated
Show resolved
Hide resolved
…random hover effect
f0328fc
to
7910afc
Compare
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.
@ranegray excellent work on this man. It was a big ticket and you did good. Thanks for creating issues as well that's much appreciated!
Built out multipage form from issue #32. Added Zustand FormStore and multiple view components and navigation components. Data is persisting globally right now and being stored in Zustand formStore.js.
I've commented out Map and NavBar while working on this component.