hackforla / website

Hack for LA's website
https://www.hackforla.org
GNU General Public License v2.0
323 stars 774 forks source link

Remove unused `alt-hero` field from 311-data.md project file #2892

Closed JessicaLucindaCheng closed 2 years ago

JessicaLucindaCheng commented 2 years ago

Overview

As a developer, I want to make sure unused code is removed from the codebase in project cards so that

Details

In project md files, image-hero is a decorative background image and alt-hero is the alt text associated with image-hero. However, alt-hero text is not used anywhere in our codebase since the image-hero is used as an SCSS background image.

Moreover, if the image-hero were to be used in an img HTML tag in the future, the alt text for all the image-hero images would be alt="", which could be set programmatically for all the alt properties associated with each image-hero.

Action Items

Resources/Instructions

File and code links you will need to work on this issue

Information about WCAG

github-actions[bot] commented 2 years ago

Hi @JessicaLucindaCheng.

Good job adding the required labels for this issue. The merge team will review the issue and add a "Ready for Milestone" label once it is ready for prioritization.

Additional Resources:

trishajjohnson commented 2 years ago

Availability for this week: 6hrs My estimated ETA for completing this issue: end of day Thursday

JessicaLucindaCheng commented 2 years ago

@trishajjohnson Please give a progress report in a comment in this issue that includes Progress, Blockers, Availability, and ETA. See more detailed information in section 2.5 Reporting progress on your issue in CONTRIBUTING.md. Also, if you are taking a leave of absence or a vacation, please also let us know in your progress report.

github-actions[bot] commented 2 years ago

@trishajjohnson

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures: "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the developer meeting discussion column and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel.

You are receiving this comment because your last comment was before Tuesday, May 3, 2022 at 12:20 AM PST.

trishajjohnson commented 2 years ago

Apologies, I got swallowed whole by some interviews/take-homes.

Progress: I need to push changes and create pull request to be reviewed/approved. Blockers: None Availability: 6hrs ETA: pushed EOD 5/6, reviewed hopefully end of week Pictures: None

trishajjohnson commented 2 years ago

@JessicaLucindaCheng I have 2 questions: when fetching upstream, do I need to be in the issue branch or gh-pages branch? Also, when preparing to push changes to my fork, do I add/commit/push from the issue branch or gh-pages?

JessicaLucindaCheng commented 2 years ago

@trishajjohnson

when fetching upstream, do I need to be in the issue branch or gh-pages branch?

Answer: For fetching upstream, if you mean updating your repo, you would want to be in gh-pages

Also, when preparing to push changes to my fork, do I add/commit/push from the issue branch or gh-pages?

Answer: Your issue branch, which would be the branch you made for the issue you are working on.