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

Removed ending slash from img tag #4517

Merged
merged 2 commits into from
Apr 19, 2023

Conversation

pdimaano
Copy link
Member

Fixes #4448

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

  • Removed ending slash from img tag to keep syntax consistent throughout codebase

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

No visual changes to the website.

@vanessavun vanessavun self-requested a review April 15, 2023 06:05
@vanessavun
Copy link
Member

Availability: weekday evenings
ETA: 04/14/2023

Copy link
Member

@vanessavun vanessavun left a comment

Choose a reason for hiding this comment

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

Strange. Based on #4448 it asks specifically for
<img class="card-image" src="{{ item.image }}" alt=""/>
to be changed to
<img class="card-image" src="{{ item.image }}" alt="">

but the file you have looks different with it as
<img class="card-image" src="{{ item.card-image-src }}" alt="{{ item.card-image-alt }}"/>

To double check, can you ensure that you updated the gh-pages branch?

I hope a second reviewer can check this as well.

@cmedina-dev cmedina-dev self-requested a review April 15, 2023 15:30
@cmedina-dev
Copy link
Contributor

Availability - All day today
ETA - 4/15/2023

Copy link
Contributor

@cmedina-dev cmedina-dev left a comment

Choose a reason for hiding this comment

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

Hi @pdimaano! Looks like your PR has a merge conflict that you'll need to resolve before the PR can be merged. You did modify the correct line, but if you take a look here, that same line was updated: https://github.com/hackforla/website/blob/gh-pages/_includes/program-area-pages-cards.html

My only other suggestion is making sure your PR is added to the project board on the right-hand side of your PR. You can add it really easily by just clicking the little cog wheel and selecting the HfLA Project Board option.

Everything else looks good! You linked the proper issue, created the PR from your feature branch, and explained why the change was made. Good work!

Copy link
Member

@vanessavun vanessavun left a comment

Choose a reason for hiding this comment

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

Thanks for fixing the merge issue. Now the changes you made in _includes/program-area-pages-cards.html reflects the changes needed mentioned in the issue you're working on.

This looks good, and you linked the proper issue, updated the PR from your feature branch, and explained why the change was made. Thanks!

Copy link
Contributor

@cmedina-dev cmedina-dev left a comment

Choose a reason for hiding this comment

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

Looks good to me, good job fixing the merge conflict! 👍

@chrismenke45 chrismenke45 merged commit 21c549e into hackforla:gh-pages Apr 19, 2023
@pdimaano pdimaano deleted the img-tag-refactor-4448 branch August 31, 2023 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

img Tag Refactor: Citizen Engagement page
4 participants