@gaoyan To shorten the links, you can use a service like Tiny URL. I personally don’t mind when links are long, as long as the code is indented properly. I did notice that you have some extra spaces in the first few image elements that can be removed.
<p> element, you used 2 classes and then styled them separately in your CSS. Since they both apply to only one element, why not make it just one class?
I would suggest you add an additional class on the
<div>s of image rows, one that’s not unique to each row (perhaps
.box) and apply the padding - and additional styling if you want to build on top of this - to that non-unique class.
One advantage you will get from that is that you’ll be able to remove the class
.small-image from all the
<img> elements and style them by targeting
.box img instead.
Check out the discussion in the cohort’s topic for the Tribute Page project. It was suggested that you first try to pass the tests before moving on to the details to avoid later frustration. @ConnectExtend added this helpful comment which is a link to a document with all the test requirements.
If you need help adding the test suite to your project, here is how you do it: Click the gear icon in the JS section in CodePen and enter the following in the first external script field: