search

rhnvrm / breeze-website

:sparkles: Official Website for Breeze 2016 :sparkles:
http://rhnvrm.github.io/breeze-website
Apache License 2.0
4 stars 6 forks source link

Compress images #48

Open arafsheikh opened 8 years ago

arafsheikh commented 8 years ago

Most of the recently added images here are huge. Please compress these images. I used https://imageoptim.com/api.html and saw a size reduction from 700kB to 50kB in one of the image.

FlameFractal commented 8 years ago

Compress all the images and send a PR, why don't you? Just leave the intro background, someone good with photoshop should reduce the size while maintaining quality for that. We can't afford that image to look bad.

arafsheikh commented 8 years ago

I would've done it but I'm really busy these days. :pensive:

FlameFractal commented 8 years ago

Okay. If you wanna go the back end way, then we can't do it here on github pages. @rhnvrm will have to create a php wherever we're hosting snu-breeze.com

arafsheikh commented 8 years ago

I don't think the setting up a backend server is worth the effort. We just need to compress a few images.

mr-karan commented 8 years ago

Hell, we're not using PHP for backend any-f-way.

ghost commented 8 years ago

@mr-karan +1

ghost commented 8 years ago

Okk...So, lemme take up this issue then..Noone's come for it till now But, Idk how should I get all the images...My fork is 'x' commits behind rhnvrm's gh-pages and even if I fork it again, it does not update....How to do that ?

arafsheikh commented 8 years ago

I think there's already a PR open for this. On Jul 6, 2016 10:21 AM, "Harshit Budhraja" notifications@github.com wrote:

Okk...So, lemme take up this issue then..Noone's come for it till now

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rhnvrm/breeze-website/issues/48#issuecomment-230670466, or mute the thread https://github.com/notifications/unsubscribe/AGtAZh981xJ93JOXHoVgtT0F2cXRGM6yks5qSzQ4gaJpZM4JEdHF .

arafsheikh commented 8 years ago

@harshitbudhraja git remote -v should show both upstream and origin remotes. If it doesn't you'll have to add upstream using git remote add upstream https://github.com/rhnvrm/breeze-website.git.

Then do git fetch upstream and git rebase upstream/gh-pages. (There might be typos in the commands as I wrote this on mobile)

FlameFractal commented 8 years ago

@arafsheikh Yes. I made the PR yesterday. @harshitbudhraja What you want to be doing is, "sync fork with upstream". So search for it and you'll get it.

ghost commented 8 years ago

@arafsheikh @FlameFractal , thanks both of you...Worked it out ! M sorry, but I'm not as experienced in using git as you might be....:) Still learning !