Skip to content
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

Merged
merged 4 commits into from
Jul 12, 2024

Conversation

tarunsinghofficial
Copy link
Collaborator

Fixes #8

Tasks done:

  • Replace standard Google Maps markers with custom HTML elements
  • Add direction arrows to markers using Font Awesome icons
  • Implement light and dark mode styles for markers
  • Use Google Maps OverlayView for improved performance and flexibility

Screenshot:
image

Copy link
Member

@aaronbrethorst aaronbrethorst left a 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.

@@ -66,21 +67,65 @@
}

function addMarker(s) {
const glyphImg = document.createElement('img');
glyphImg.src = busIcon;
const markerElement = document.createElement('div');
Copy link
Member

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.

Copy link
Collaborator Author

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?

Copy link
Member

@aaronbrethorst aaronbrethorst left a 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

@aaronbrethorst aaronbrethorst merged commit 7da99c5 into main Jul 12, 2024
2 checks passed
@aaronbrethorst aaronbrethorst deleted the stopmarkers branch July 12, 2024 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Give stop markers an arrow that points in the direction of travel
2 participants