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

[TCL30-70] Update Pins on Map #37

Merged
merged 1 commit into from
Sep 4, 2021
Merged

Conversation

jssckbl
Copy link
Contributor

@jssckbl jssckbl commented Sep 4, 2021

Description

This updates the styling of the pins associated with available locations on current map view. It also updates the styling of the pin representing the user when they are sharing their location.

Related Issue

Type of Changes

Type
🐛 Bug fix
✨ New feature
🔨 Refactoring
💯 Add tests
🔗 Update dependencies
📜 Docs

Updates

Before

BeforeUpdatePins

After

AfterUpdatePins

Testing Steps / QA Criteria

  1. v2-whats-near-me
  2. jw-update-pins
  3. npm start
  4. While on /, share your location. Your icon should be pulsing shades of green. Pin locations should have a green border.

@github-actions
Copy link

github-actions bot commented Sep 4, 2021

Visit the preview URL for this PR (updated for commit dab39d5):

https://tcl-30-whats-near-me-d2e34--pr37-jw-update-pins-vcsd4dq2.web.app

(expires Sat, 11 Sep 2021 02:34:39 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@jssckbl jssckbl requested a review from alxmcr September 4, 2021 02:36
@@ -265,8 +265,9 @@ a.active {
width: 24px;
height: 24px;
cursor: pointer;
border: 2px solid #1c2334;
background: grey;
border: 2px solid var(--primary-color);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Our primary color 😄 !

@@ -307,15 +308,16 @@ a.active {
.user-location-icon {
width: 180px;
height: 180px;
fill: #a020f0;
fill: var(--primary-color);
Copy link
Contributor

@alxmcr alxmcr Sep 4, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

@alxmcr alxmcr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great improvements! brand colors are present 😄 !

@jssckbl jssckbl merged commit e1b0968 into v2-whats-near-me Sep 4, 2021
@jssckbl jssckbl deleted the jw-update-pins branch September 4, 2021 03:23
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.

2 participants