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

Adjust file size of some homepage project card images #1165

Closed
8 tasks
daniellex0 opened this issue Mar 7, 2021 · 7 comments · Fixed by #1819
Closed
8 tasks

Adjust file size of some homepage project card images #1165

daniellex0 opened this issue Mar 7, 2021 · 7 comments · Fixed by #1819
Assignees
Labels
good first issue Good for newcomers P-Feature: Home page https://www.hackforla.org/ role: front end Tasks for front end developers size: missing To Update ! No update has been provided

Comments

@daniellex0
Copy link
Member

daniellex0 commented Mar 7, 2021

Overview

We need to decrease the size of some of the newer homepage project card images because unnecessarily large PNG files can slow down page loading time (per our image guide PNG section)

The images should have a 3:2 ratio and a standard resolution of 600x400 pixels.

Action Items

  • Review the standard file size/dimensions of the correct project card images (for ex. Ballotnav, Food Oasis sizes are correct). The images should have a 3:2 ratio and a standard resolution of 600x400 pixels.
  • Project card image sizes to decrease and standardize:
    • 100 Automations
    • Home Unite Us
    • VRMS
  • Add the new standardized versions in comments in this issue below

Development

  • Add the decreased/standardized images to their project cards
    • (*note that only the homepage project card image file needs to be adjusted- not the large project page "splash" version image file)

Resources/Instructions

Homepage project cards

A project card image for reference:

Screen Shot 2021-03-06 at 9 19 41 PM

@daniellex0 daniellex0 added role: front end Tasks for front end developers Complexity: Medium labels Mar 7, 2021
@Linda-OC Linda-OC self-assigned this May 11, 2021
@qiqicodes
Copy link
Member

@Linda-OC Can you let us know your progress in regards to this issue? Thank you =]

Progress:
Blockers:

@qiqicodes qiqicodes added the To Update ! No update has been provided label May 15, 2021
@Linda-OC
Copy link
Member

Progress: None yet
Blockers: Locating the code that needs to be changed
Availability: 4 hrs
ETA: By 5/19

@paulsvh paulsvh added role: UI and removed role: front end Tasks for front end developers To Update ! No update has been provided labels May 19, 2021
@Sihemgourou Sihemgourou added role: design good first issue Good for newcomers and removed role: UI good first issue Good for newcomers labels Jun 12, 2021
@Linda-OC Linda-OC assigned Linda-OC and unassigned Linda-OC Jun 14, 2021
@sayalikotkar sayalikotkar added the Feature Missing This label means that the issue needs to be linked to a precise feature label. label Jun 20, 2021
@Sihemgourou Sihemgourou added P-Feature: Home page https://www.hackforla.org/ and removed Feature Missing This label means that the issue needs to be linked to a precise feature label. labels Jun 20, 2021
@mgodoy2023 mgodoy2023 self-assigned this Jun 21, 2021
@mgodoy2023
Copy link
Member

mgodoy2023 commented Jun 21, 2021

Here are the new project image files that have been adjusted to 600pixels x 400pixels. I set all of the resolutions to a standard 72dpi, as that is the standard used for online images.

100 automations:
100automations

Home Unite Us:
home-heart

VRMS:
vrms

@mgodoy2023
Copy link
Member

mgodoy2023 commented Jun 21, 2021

Hi @daniellex0, please review these images and advise on next steps. Thank you!

@mgodoy2023
Copy link
Member

mgodoy2023 commented Jun 21, 2021

Hi again @daniellex0, I updated the images in a new pull request and adjusted the Home Unite Us image the way you said to. Please review the images and let me know about any next steps. Thank you!

@Sihemgourou Sihemgourou added To Update ! No update has been provided role: front end Tasks for front end developers and removed role: design labels Jun 28, 2021
@github-actions
Copy link

@mgodoy2023 Please add update

  1. Progress
  2. Blockers
  3. Availability
  4. ETA

@Sihemgourou
Copy link
Contributor

@mgodoy2023 why this issue is still open ? The PR seems closed
Please add uptades on this issue before taking another one (I unassigned you from this one but Please feel free to take it back once you finish working on this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment