Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Cards #44

Closed
chadluo opened this issue Mar 27, 2016 · 5 comments · Fixed by #260
Closed

Cards #44

chadluo opened this issue Mar 27, 2016 · 5 comments · Fixed by #260
Labels

Comments

@chadluo
Copy link
Contributor

chadluo commented Mar 27, 2016

The current version of the page occupies by default the whole width of the page, making the cards too wide to scan through and leaving too much space between them.

I managed to reduce the width of the cards without squeezing them too tight, and you can refer to this screenshot:

image

I wanted to change the background color of the page to light gray since white cards with drop shadows does not look good on the white background, but a gray background does not cope well with the beige banner. I am still working on that, and I shall come with some other solutions later.

If you are happy with this, I can just put this on.

@chadluo chadluo added the design label Mar 27, 2016
@keplersj
Copy link
Contributor

Puurrrrrty.

@keplersj
Copy link
Contributor

Mind doing it in a PR so I can see it before merge?

@keplersj
Copy link
Contributor

Direct posting to master scares me.

@chadluo
Copy link
Contributor Author

chadluo commented Mar 29, 2016

It's fine doing a PR. I said I won't push without your approval.

@keplersj
Copy link
Contributor

Yay! Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants