Update Homepage - Styles

[RedSparr0w]

Closes #1206

IMO it looks a little strange, but makes sense to display it the real size

Added social & for-the-badge style to badge creation dropdown list
& close <option> tags

[paulmelnikow]

Nice!

Could you confirm (on here) that Red$parr0w on Discord is you? And then DM me your email address? I'd like to set you up to deploy review apps from Heroku.

[paulmelnikow]

Huh. Why is this needed? Are the images fixed height? Would height:auto work?

[RedSparr0w]

pretty much all the images are height:20px

I could change this badge to height:auto if you like though.

From memory the change to height:20px was due to the page used to jump around a lot while loading.

Edit: Reduce page jumping while all the badges are loading #967 & #813

[RedSparr0w]

Could update the css in try-header to remove .badge-img img which would only affect the following sections:

table.badge > tbody > tr > td > img, #copyImg {
  height: 20px;
  vertical-align: middle;
}

[paulmelnikow]

Nah, thanks for explaining. I remember that change and it makes sense now.

[paulmelnikow]

👍

[RedSparr0w]

👍 Red$parr0w on discord is me

[tooomm]

Thanks for taking care of this @RedSparr0w ! 👍

Added social & for-the-badge style to badge creation dropdown list

Is the new for the badge style option there also previewed in its real size?
There is no heroku review app available for this pr to verify for myself.

[RedSparr0w]

Oh did not even think about that, thanks!
Will update it soon with those changes.

[paulmelnikow]

@RedSparr0w I invited you to the Heroku app so you should be able to deploy review apps now! Let me know how you make out with that.

[RedSparr0w]

Thanks, think that worked, although it says you deployed it so not 100% sure?

Couple quick questions,
Do I have to redeploy each time a new commit is pushed through or will it automatically update the deployment?
Once a PR is merged will it automatically stop or does it have to be done manually?

[paulmelnikow]

Haha, that's funny! No, I did not deploy it.

It will automatically re-deploy when you update the PR.

And it'll turn itself off when the PR closes, or after I think 30 days.

[RedSparr0w]

updated the css to be min-height: 20px; for the badges instead of fixed height: 20px this way all the for-the-badge style badges should be shown at full size without any inline css and the page shouldn't jump around too much

[RedSparr0w]

Should be ready for merge now:

Unless there is anything else i've missed.

[paulmelnikow]

Looks good to me!