Closed narthur closed 3 weeks ago
Fix deployed
After:
Are we sure we disliked the wrapping? Maybe it's nice? I don't know, I'm bad at aesthetics, but maybe Bee could weigh in unless you're sure it's better this way.
@dreeves Hmm, that definitely is how I intended it to work at small screen widths. I guess I should reconsider that choice?
@dreeves @bsoule Closing this now, since the original bug is fixed. If there's a desire to reconsider the design intention here let's create a new issue.
Set me straight if I'm creating bad workflows but since this one has the screenshots and stuff I'd rather leave it open till Bee chimes in. She may say it's fine and we can just close this. But with this closed we have nothing to remind us to get a new gissue created if we do want to change something about this.
after mobile / small scale is good:
after for the desktop / large scale is fine:
I maybe have a slight preference for the "before" version above, with the text wrapping around the image, except maybe because of the variablility of the length of the excerpts, it might be better the way it currently is?
like if some excerpts are long and wrap below the image, and then others are shorter and don't wrap, maybe that would look worse than the status quo? i have a slight, though not strong, prediction that it'd be ugly for some to have text below the image and others without (jsut cuz the excerpt is too short).
Text wrapping around the images like this is a regression.