digitalcredentials / dcc-website

https://digitalcredentials.mit.edu
1 stars 7 forks source link

Lower resolution of background source images so they load faster #19

Closed fields37 closed 1 year ago

bmuramatsu commented 1 year ago

For the binary image, you could probably also make it 20-25% its current height. It's probably bigger than we need.

On Thu, Mar 2, 2023 at 1:01 PM fields37 @.***> wrote:

— Reply to this email directly, view it on GitHub https://github.com/digitalcredentials/dcc-website/issues/19, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFXVRVPCUWJOR3U3DZK7BTW2DN5PANCNFSM6AAAAAAVNWKW54 . You are receiving this because you are subscribed to this thread.Message ID: @.***>

fields37 commented 1 year ago

I realized the header image on the homepage was a png file, switching it to jpg reduced the size from ~3.5mb to 500kb. https://github.com/digitalcredentials/dcc-website-jekyll/commit/eafdf01084f7d7a8a8c86b1679f71c93a43ccd3e

bmuramatsu commented 1 year ago

I also suggest reducing the height of the image to something on the order of the same height as the top nav bar. At it's current height, over half of the height of the page on screen is lost to chrome meaning that less than 50% what an individual sees is content.

fields37 commented 1 year ago

Are you suggesting we squeeze text blocks with background images, something along the lines of the following?

Screen Shot 2023-04-13 at 7 45 00 PM Screen Shot 2023-04-13 at 7 48 41 PM

bmuramatsu commented 1 year ago

When the image at the top of each page just contains the page title, I'd make it much shorter. If it includes text then it should be big enough to give breathing room / whitespace. I'm ok with that block changing height as appropriate.

On Thu, Apr 13, 2023 at 7:55 PM fields37 @.***> wrote:

Are you suggesting we squeeze text blocks with background images, something along the lines of the following?

[image: Screen Shot 2023-04-13 at 7 45 00 PM] https://user-images.githubusercontent.com/20584379/231906496-05c72742-cef1-4bd1-8d1f-7a3dc75ec074.png [image: Screen Shot 2023-04-13 at 7 48 41 PM] https://user-images.githubusercontent.com/20584379/231906511-84df6086-de44-45bd-b276-90ad5e213222.png

— Reply to this email directly, view it on GitHub https://github.com/digitalcredentials/dcc-website/issues/19#issuecomment-1507739273, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFXVRQGBL7TVRKVFPTP7BDXBCGNVANCNFSM6AAAAAAVNWKW54 . You are receiving this because you commented.Message ID: @.***>

fields37 commented 1 year ago

Alright, I'll make some global adjustments to spacing based on this feedback. I honestly think a lot of the larger type could also be reduced a bit in size, I'll review that as well.

bmuramatsu commented 1 year ago

Great, thanks!

On Thu, Apr 13, 2023 at 8:20 PM fields37 @.***> wrote:

Alright, I'll make some global adjustments to spacing based on this feedback. I honestly think a lot of the larger type could also be reduced a bit in size, I'll review that as well.

— Reply to this email directly, view it on GitHub https://github.com/digitalcredentials/dcc-website/issues/19#issuecomment-1507760600, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFXVRT4O6DKLLYOFHBOVWTXBCJ2TANCNFSM6AAAAAAVNWKW54 . You are receiving this because you commented.Message ID: @.***>

fields37 commented 1 year ago

I made some of these size reductions to the header sections in commit https://github.com/digitalcredentials/dcc-website-jekyll/commit/e3ca36eb55687238b7ea5576fcab26ea94231dea, let me know if you would like things even smaller.

bmuramatsu commented 1 year ago

Please reduce it further. The header with the page title isn't the most important thing on the page, the content below it is.

My starting point would be the following, and I think it could be smaller still.

[image: Screen Shot 2023-04-19 at 9.45.39 AM.png]

On Mon, Apr 17, 2023 at 4:07 PM fields37 @.***> wrote:

I made some of these size reductions to the header sections in commit @.*** https://github.com/digitalcredentials/dcc-website-jekyll/commit/e3ca36eb55687238b7ea5576fcab26ea94231dea, let me know if you would like things even smaller.

— Reply to this email directly, view it on GitHub https://github.com/digitalcredentials/dcc-website/issues/19#issuecomment-1512011376, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFXVRXR2ALMP5HYIDZZJS3XBWPGHANCNFSM6AAAAAAVNWKW54 . You are receiving this because you commented.Message ID: @.***>

fields37 commented 1 year ago

Hey Brandon, I'm not able to see the image you included here.