Adding a max-width property to images in units

[dottorblaster]

Building a page with Gridism, I saw it did not render the way I wanted. I have put these three lines in my style.css but I believe these should be part of the framework, providing a good responsiveness for pics and "huge" graphic elements.

[cobyism]

:+1: Yeah, I’m fine with this change, however I’d like to suggest a couple of small tweaks:

• I’d like to move this rule further down in the file—probably just after the "clearfix" rules and above the "utility classes" section. The rules that govern the width of grid units and so forth are much more important than rules like this one, which I see more as a helpful convenience than anything else, so if people go to read the gridism CSS file, I want all of that core stuff to come first.
• The .grid .unit img selector could probably just be changed to .unit img, since the extra specificity of having .grid in there might not really be doing much.
• It would be nice to add a quick one-line comment explaining what this rule does. Something like "make images match unit width by default" but worded a bit better.

Once you make those changes, I’ll merge this in. Thanks for the pull! :grinning:

[dottorblaster]

Committed the changes. You were right at all :smile: And, of course, sorry for submitting you code without a line of comment.

[cobyism]

:metal: Thanks! No worries for not having a comment originally—there’s other things in there that I didn’t put comments on, but for things like this I think it’s worth providing a bit of background for people who read the code :+1: Thanks again!