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

img Tag Refactor: Toolkit page #4374

Merged
merged 1 commit into from
Apr 4, 2023

Conversation

cng008
Copy link
Member

@cng008 cng008 commented Apr 3, 2023

Fixes #4366

What changes did you make and why did you make them ?

  • change an img HTML tag ending with a slash (<img.../>) to an img tag without an ending slash (<img...>) so that the codebase is consistent with how we use img HTML tags

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

No visual changes to the website.

@github-actions
Copy link

github-actions bot commented Apr 3, 2023

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 cng008-img-tag-refactor-4366 gh-pages
git pull https://github.com/cng008/website.git img-tag-refactor-4366

@github-actions github-actions bot added good first issue Good for newcomers role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers P-Feature: Toolkit https://www.hackforla.org/toolkit/ size: 0.5pt Can be done in 3 hours or less labels Apr 3, 2023
@cng008 cng008 requested a review from arghmatey April 3, 2023 00:33
@drakenguyen4000 drakenguyen4000 self-requested a review April 3, 2023 05:00
@drakenguyen4000
Copy link
Member

Review ETA: 4/3/23
Availability: M-F 4-8PM

@janice87 janice87 self-requested a review April 3, 2023 16:57
@janice87
Copy link
Member

janice87 commented Apr 3, 2023

Review ETA: 4/3/23
Availability: M-W 5-8 PM

Copy link
Member

@janice87 janice87 left a comment

Choose a reason for hiding this comment

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

The pull request is done in the correct branch and the issue is linked correctly. In the source code, I see the change was made to the tag to remove the slash at the end. There are no visual changes made. Good job on your first issue, this is approved!

@cng008 cng008 removed the request for review from arghmatey April 4, 2023 00:47
@cng008 cng008 changed the title img Tag Refactor img Tag Refactor: Toolkit page Apr 4, 2023
@github-actions github-actions bot added Feature: Refactor HTML size: 0.25pt Can be done in 0.5 to 1.5 hours and removed size: 0.5pt Can be done in 3 hours or less labels Apr 4, 2023
@drakenguyen4000
Copy link
Member

drakenguyen4000 commented Apr 4, 2023

The tag is fixed, the issue is linked, your pull request description is helpful, changes look good in the local environment, and branches are set up correctly.

However, I suggest adding why the change was needed. Per the issue description as to why the change was needed.
We want to change an img HTML tag ending with a slash (<img.../>) to an img tag without an ending slash (<img...>) so that the codebase is consistent with how we use img HTML tags.

Thank you for taking this issue @cng008! Great job!

@cng008
Copy link
Member Author

cng008 commented Apr 4, 2023

@drakenguyen4000 I appreciate the feedback! I've taken your advice and made the necessary adjustments. Thank you for your time

@jdingeman jdingeman merged commit 1abc580 into hackforla:gh-pages Apr 4, 2023
@cng008 cng008 deleted the img-tag-refactor-4366 branch April 5, 2023 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature: Refactor HTML good first issue Good for newcomers P-Feature: Toolkit https://www.hackforla.org/toolkit/ role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours
Projects
None yet
Development

Successfully merging this pull request may close these issues.

img Tag Refactor: Toolkit page
4 participants