-
Notifications
You must be signed in to change notification settings - Fork 4
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
Added the stop-marker with custom bus and directions #21
Conversation
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.
Looks good! I think there's some fit and finish work we can do on the arrows themselves, but that can wait. let me know about the custom component, though, please.
src/components/map/GoogleMap.svelte
Outdated
@@ -66,21 +67,65 @@ | |||
} | |||
|
|||
function addMarker(s) { | |||
const glyphImg = document.createElement('img'); | |||
glyphImg.src = busIcon; | |||
const markerElement = document.createElement('div'); |
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.
Do you think you can turn the non-Google custom marker code into a new, reusable client-side component? https://svelte.dev/docs/client-side-component-api.
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.
Could you please provide more details on how you envision turning the custom marker code into a reusable client-side 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.
Great work! this looks perfect
Fixes #8
Tasks done:
Screenshot: