-
Notifications
You must be signed in to change notification settings - Fork 0
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
Conversation
… sharing location.
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 🌎 |
@@ -265,8 +265,9 @@ a.active { | |||
width: 24px; | |||
height: 24px; | |||
cursor: pointer; | |||
border: 2px solid #1c2334; | |||
background: grey; | |||
border: 2px solid var(--primary-color); |
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.
Our primary color 😄 !
@@ -307,15 +308,16 @@ a.active { | |||
.user-location-icon { | |||
width: 180px; | |||
height: 180px; | |||
fill: #a020f0; | |||
fill: var(--primary-color); |
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.
👍
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 improvements! brand colors are present 😄 !
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
Updates
Before
After
Testing Steps / QA Criteria
v2-whats-near-me
jw-update-pins
npm start
/
, share your location. Your icon should be pulsing shades of green. Pin locations should have a green border.