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

Update Project page's GitHub icon to icon-github.svg #6973

Merged

Conversation

ha-bach
Copy link
Member

@ha-bach ha-bach commented Jun 8, 2024

Fixes #6909

What changes did you make?

  • Updated _includes/resource-card.html to use _includes/svg/icon-github.svg instead of _includes/svg/icon-github-color.svg
  • Updated _sass/components/_project-page.scss so that the new icon is displayed appropriately
  • Removed _includes/svg/icon-github-color.svg

Why did you make the changes (we will use this info to test)?

  • To remove redundant GitHub icon file _includes/svg/icon-github-color.svg

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

image

Visuals after changes are applied

image

Copy link

github-actions bot commented Jun 8, 2024

Want to review this pull request? Take a look at this documentation for a step by step guide!


From your project repository, check out a new branch and test the changes.

git checkout -b muninnhugin-update-project-resource-card-6909 gh-pages
git pull https://github.com/muninnhugin/hfla-website.git update-project-resource-card-6909

@github-actions github-actions bot added role: front end Tasks for front end developers Complexity: Medium P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) Feature: Refactor HTML size: 1pt Can be done in 4-6 hours labels Jun 8, 2024
@ha-bach ha-bach changed the title update Project page's GitHub icon to icon-github.svg Update Project page's GitHub icon to icon-github.svg Jun 8, 2024
@terrencejihoonjung terrencejihoonjung self-requested a review June 8, 2024 07:07
@terrencejihoonjung
Copy link
Member

Review ETA: 6/11/24, 11:59pm PST
Availability: 4pm - 12am, Mon-Fri

@aadilahmed aadilahmed self-requested a review June 10, 2024 22:02
@aadilahmed
Copy link
Member

Review ETA: EOD 6/11/24
Availability: 9-5 PM M-F

Copy link
Member

@terrencejihoonjung terrencejihoonjung left a comment

Choose a reason for hiding this comment

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

Hi @muninnhugin,

  • branches are correctly specified
  • title is clear, changes made and why they were made are clearly stated, and visual changes are appropriately shown
  • removed {% include svg/icon-github-color.svg %} and added {% include svg/icon-github.svg %}
  • deleted icon-github-color.svg file
  • added appropriate styling to _project-page.scss and visual changes look consistent

Great job! Thanks for your help!

Copy link
Member

@aadilahmed aadilahmed left a comment

Choose a reason for hiding this comment

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

Hi @muninnhugin , you correctly updated the github icon to use the new svg file, and deleted the original icon-github-color.svg file. Styling looks consistent with the original icon and your code is clean. Everything looks good, great job!

Copy link
Contributor

@mSharifHub mSharifHub left a comment

Choose a reason for hiding this comment

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

code is correct

_includes/resource-card.html Show resolved Hide resolved
@jphamtv jphamtv merged commit 7cc396a into hackforla:gh-pages Jun 13, 2024
12 checks passed
@ha-bach ha-bach deleted the update-project-resource-card-6909 branch June 19, 2024 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Medium Feature: Refactor HTML P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: front end Tasks for front end developers size: 1pt Can be done in 4-6 hours
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Resource card on Project pages to use icon-github.svg
5 participants