Closed AndrewNatoli closed 8 years ago
I will take a look and push the change
@AndrewNatoli they seem a bit big when I change my chrome window size, is this expected
@RowdyChildren What size are you changing to (Phone, Tablet, Desktop, Super high desktop resolution)?
On tablets they can be a little large as they fall under the left sidebar and the container takes up the full width. On mobile they collapse to one per row so a single image will take up the full width.
yes, I see what you mean. It looks pretty shitty when there is one small image per row. Change has been pushed.
On Fri, Dec 18, 2015 at 8:23 PM Andrew Natoli notifications@github.com wrote:
@RowdyChildren https://github.com/RowdyChildren What size are you changing to (Phone, Tablet, Desktop, Super high desktop resolution)?
On tablets they can be a little large as they fall under the left sidebar and the container takes up the full width. On mobile they collapse to one per row so a single image will take up the full width.
— Reply to this email directly or view it on GitHub https://github.com/CodeForCharlotte/codeforcharlotte.org/issues/162#issuecomment-165932207 .
Changed pushed and approved, change is now in production
So... it's fine as-is then as it's one full-width image per row? Or should I look into getting two per row on mobile?
yeah, it is fine. There is one size in which the images are huge and super pixelly, but the likely hood of that exact really strange window size occurring is not high...so don't worry about it
Revised /templates/team/leadership.tpl
Photos now stay circular and scale for different window sizes.