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

updated project image sizes and resolutions #1798

Closed

Conversation

mgodoy2023
Copy link
Member

@mgodoy2023 mgodoy2023 commented Jun 21, 2021

Fixes #1165

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

  • Adjusted the image size and resolution for the following project cards on the homepage to 600 pixels x 400 pixels and a standard resolution of 72dpi as was requested in the issue:
  1. 100 Automations
  2. Home Unite Us
  3. Open Community Survey
  4. VRMS
  5. RailStats LA
  • Placed updated images in here
  • For the Home Unite Us image the dimensions of the original image were 600 x 600 so I left it as is. Changing it to 600 x 400 would ruin the circular logo they had created.

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

Edit_This_To_Add_Image_Description

Paste_Your_Image_Link_Here

@daniellex0
Copy link
Member

daniellex0 commented Jun 21, 2021

Thank you @mgodoy2023 !

VRMS and 100 Automations are perfect 👍

And I am so sorry this issue was a few months old, so it looks like Railstats and Open Community Survey had already been fixed if I'm not mistaken. So you can remove those from your PR.

As for Home Unite Us, it actually shouldn't mess up the circle if the image is just cropped to a height of 400px. As long as the circle is still centered, it should still look fine. We need to fix this because otherwise it looks wonky next to the other standardized project cards:

Screen Shot 2021-06-21 at 2 09 28 PM

is it possible to crop it to the same width, but 400px height (and circle vertically centered)?

@mgodoy2023 mgodoy2023 closed this Jun 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers P-Feature: Home page https://www.hackforla.org/ role: design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adjust file size of some homepage project card images
5 participants