-
Notifications
You must be signed in to change notification settings - Fork 8
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
event icon updated #2082
event icon updated #2082
Conversation
@jtmst a few comments/questions:
|
Added changes for the social links. I can also do event listings with this ticket if that makes most sense, but it needs to be a separate PR as event listings live in vets-website |
@jtmst The calendar icon shouldn't have been included here - because there is a calendar Otherwise, everything else looks good |
No I Lied above! The Twitter logo should be using the new X logo! Do we have that available in the new va-icon set? Would it also be possible - I know it's slightly out of scope - but could we update the link text to say "Share on X (formerly Twitter)" as well? Definitely say no if that's too much |
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.
Approved - noting that the slight misalignment of the Calendar icon and the other social icons should be resolved next week when the DST updates to use
Note: Delete the description statements, complete each step. None are optional, but can be justified as to why they cannot be completed as written. Provide known gaps to testing that may raise the risk of merging to production.
relates to department-of-veterans-affairs/vets-website#29769
Summary
Related issue(s)
Testing done
Screenshots
What areas of the site does it impact?
(Describe what parts of the site are impacted if code touched other areas)
Acceptance criteria
Quality Assurance & Testing
Error Handling
Authentication
#sitewide-public-websites
Slack channel for questionsRequested Feedback
(OPTIONAL) What should the reviewers know in addition to the above. Is there anything specific you wish the reviewer to assist with. Do you have any concerns with this PR, why?