Techtonica / techtonica.org

This repo is for the techtonica.org website for Techtonica, a nonprofit tech training program that helps women and non-binary adults with low incomes overcome barriers into tech careers.
https://techtonica-org.vercel.app
29 stars 47 forks source link

Adjust large screen sponsor page images and chart #374

Open daaimah123 opened 5 days ago

daaimah123 commented 5 days ago

This PR addresses issue #352 whereby the sponsors page chart image and the pie chart are very large on larger screen sizes. The following section's images still need to be addressed:

[✅ Working] For the chart: To adjust the chart to have limited sizing of the image after 2880px by 1800px screen, we'll modify both the HTML and JavaScript parts; use CSS media queries to control the canvas size and update the Chart.js configuration to ensure proper scaling.

[✅ Working] For the images: accomplished the change by limiting a width: 100% CSS rule from growing after 2880px

Before

Screenshot 2024-10-30 at 2 37 07 PM Screenshot 2024-10-30 at 5 05 13 PM Screenshot 2024-10-30 at 5 05 21 PM

After

Screenshot 2024-10-30 at 2 38 31 PM Screenshot 2024-10-30 at 5 03 23 PM Screenshot 2024-10-30 at 5 03 41 PM
vercel[bot] commented 5 days ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
techtonica-org ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2024 0:02am
daaimah123 commented 4 days ago

⏸️ Making a note for revisting, the current work to date, does not address the image resizing for the following sections on the page, that also have a .full-width-img class:

MaggieFero commented 4 days ago

⏸️ Making a note for revisting, the current work to date, does not address the image resizing for the following sections on the page, that also have a .full-width-img class:

  • Techtonica Consulting
  • How Hiring Works

For Hiring, we should still do it here. Do you want to just skip the consulting section for now, though, since we're in the process of redoing that content? Could make more sense to put it out of scope for this and just address those images on the new page.