I completed my first project for FreeCodeCamp and I’d welcome feedback. Here is my site:
I read on another thread that br element is deprecated, which I saw on MDN later. So I plan to change that. It appears that p elements or margins are recommended instead. If I were to use p element, just an empty one to make additional vertical space I guess? Interesting.
When I was creating the page I had some trouble getting the photos that were lined up 2 in a row to be the right size vertically. I tried various things such as fussing with max height, max width, hard coded image sizes, and hiding overflow. The issue was, I think, tied to the fact that the images were not the same size when I was starting out. I didn’t want to sacrifice responsiveness or scaling, so in the end I cropped the photos and hosting in a location apart from their original source. There must be a better way… but after much Googling and a question to the Help part of Gittr I just couldn’t find it.
Also, although I used Bootstrap for the site I didn’t enclose everything in “rows” and “columns”. I just used those for the parts of the site I thought might need it. If you’re coding a Bootstrap site do you put everything in rows and columns? Or just sections of the site? I guess I erred on the side of not because when I was looking at my code it looked like an awful lot of div-row’s and div-columns for things that seemed simple, like an h1 in one line or a full-width img. What is the recommended path?
Thanks for any feedback or guidance you might have.