hackforla / website

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

Recreate an image for our homepage #4607

Open ExperimentsInHonesty opened 1 year ago

ExperimentsInHonesty commented 1 year ago

Overview

We need a new composite issue for our homepage to replace an image we don't know the origin of all the elements and a credit file to match

Action Items

Resources/Instructions

For the Civic Tech Infrastructure image:

The link for the base image is: https://unsplash.com/photos/unRkg2jH1j0

Click here to see the image from Unsplash man looking at analytics on a tablet device

We will need to recreate it to look like this image

Click here to see the image of the Civic Tech Infrastructure credit on the feature-homepage-launch branch

We will need to use the Unsplash image and get new icons to recreate the changed parts

images of


Main image

Name: Civic Tech Infrastructure Used in: Homepage Artist: Adeolu Eletu Provider: Unsplash Altered by: [YOUR NAME](YOUR LINKED OR PORTFOLIO LINK)

ExperimentsInHonesty commented 1 year ago

credit

1 composite image Name: Civic Tech Infrastructure Used in: Homepage Compiled by: [YOUR NAME](YOUR LINKED OR PORTFOLIO LINK) Provider: HfLA composite image

2 main image Name: man looking at marketing analytics Used in: Civic Tech Infrastructure (HfLA composite image) Artist: Adeolu Eletu Provider: Unsplash

3 Hammer & Wrench Name: Used in: Civic Tech Infrastructure (HfLA composite image) Artist: Provider:

4 Gears Name: Used in: Civic Tech Infrastructure (HfLA composite image) Artist: Provider:

5 Book Name: Used in: Civic Tech Infrastructure (HfLA composite image) Artist: Provider:

6 Magnifying glass Name: Used in: Civic Tech Infrastructure (HfLA composite image) Artist: Provider:

7 Bar chart Name: Used in: Civic Tech Infrastructure (HfLA composite image) Artist: Provider:

8 Hands Name: Used in: Civic Tech Infrastructure (HfLA composite image) Artist: Provider:

JessicaLucindaCheng commented 1 year ago

@JessicaLucindaCheng To Dos

JessicaLucindaCheng commented 1 year ago

@ExperimentsInHonesty During the Dev/PM meeting on May 8th, you asked what it would look like if the Provider field was removed or didn't exist from of the credits; below are my findings.

Test

On the Credits page, to test what it would look like if there was no Provider field for some of the credits, I temporarily removed the Provider field from the following credits:

Problem

The result was it recenters the text vertically and doesn't look good. Here are screenshots of the Provider field removed from some of the credits:

Click to see desktop view ![credits-no-provider](https://github.com/hackforla/website/assets/31293603/7af423eb-7aae-40c1-9b18-476341de8942)
Click to see mobile view ![credits-no-provider-mobile](https://github.com/hackforla/website/assets/31293603/8699fe1f-2839-47b7-ab83-af736feea4eb)

Possible Solution

To align the Credits text so it starts at the top of its flex container for each credit, I added align-self: flex-start; to the .icon-info class in _sass/components/_credit-items.scss.

Here are screenshots of adding align-self: flex-start; to the .icon-info class

Click here desktop view ![credits-align-self](https://github.com/hackforla/website/assets/31293603/0ddc3c78-ca77-423b-9b9c-4c5a393b682e)
Click here mobile view ![credits-align-self-mobile](https://github.com/hackforla/website/assets/31293603/4db4e8fb-0198-4fab-a009-56872bdd7292)

Edit:

github-actions[bot] commented 4 months ago

Hi @GRK1998, thank you for taking up this issue! Hfla appreciates you :)

Do let fellow developers know about your:- i. Availability: (When are you available to work on the issue/answer questions other programmers might have about your issue?) ii. ETA: (When do you expect this issue to be completed?)

You're awesome!

P.S. - You may not take up another issue until this issue gets merged (or closed). Thanks again :)