Closed
Bug 997814
Opened 11 years ago
Closed 11 years ago
Make images responsive on front page of /explore
Categories
(Webmaker Graveyard :: webmaker.org, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kate, Assigned: akirose)
Details
(Whiteboard: explore, r2)
Attachments
(1 file)
Right now they are all scaled 2x
Comment 1•11 years ago
|
||
Aki, can you help with this?
All images are here: https://www.dropbox.com/sh/p2qynqx4ar0d4d0/sllHpYFyKa
Assignee: nobody → aki
Assignee | ||
Comment 2•11 years ago
|
||
So we're not going to be using SVGs then? Those would probably scale a little better—and keep page load down.
Status: NEW → ASSIGNED
Flags: needinfo?(cassie)
Comment 3•11 years ago
|
||
We can use SVGs but Kate suggested pngs. Plus I'd have to recreate the events illustration because I did that in photoshop (dumb, I know). Can we use what's there now for mvp?
Flags: needinfo?(cassie)
Assignee | ||
Comment 4•11 years ago
|
||
Definitely, I already started (I figured we'd need it for non-vector graphics anyway) but thought I'd check in. Definitely we should thing about adding that later though. (this is the kind of thing I can do too, I see it as the mundane, programmatic part of design. Recreating things in vector that is.)
Assignee | ||
Comment 5•11 years ago
|
||
Attachment #8410517 -
Flags: review?(kate)
Reporter | ||
Comment 6•11 years ago
|
||
Comment on attachment 8410517 [details] [review]
Adds retina image directive to make images respond to pixel ratio
This looks great except for 1 nit
Attachment #8410517 -
Flags: review?(kate) → review-
Assignee | ||
Updated•11 years ago
|
Attachment #8410517 -
Flags: review- → review?(kate)
Reporter | ||
Comment 7•11 years ago
|
||
Comment on attachment 8410517 [details] [review]
Adds retina image directive to make images respond to pixel ratio
1 style nit but R+
Attachment #8410517 -
Flags: review?(kate) → review+
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 8•11 years ago
|
||
Commits pushed to master at https://github.com/mozilla/webmaker.org
https://github.com/mozilla/webmaker.org/commit/fbdd749c5bcf7fdd52f2cc979846b96068e05472
Bug 997814 - Make images responsive on front page of /explore
https://github.com/mozilla/webmaker.org/commit/8279a936945d5f9720c710c71c1c0e72860e295a
Merge pull request #692 from gesa/gesa/997814
Bug 997814 - Make images responsive on front page of /explore
You need to log in
before you can comment on or make changes to this bug.
Description
•