Add Image Search Section #16
Conversation
Add image search section to main page
ghost
self-assigned this
ghost
requested review from
kevinsheen,
hlkessner,
KMcWeeney22,
lariosw and
lukevpoel
|
Any changes needed for the main image section? |
|
We could add a preview section where users could see some images to entice them to explore more. |
There was a problem hiding this comment.
Good job creating the image galleries section. I think the only change that I would suggest for the next step on this task would be adding a photo of something below the h2 title to visually help the user locate this section quickly. I think for now these changes are sufficient and it can be pushed.
There was a problem hiding this comment.
The only additional thing I would change before merging would be from galleries to gallery since it is one gallery or archive to keep that consistent. Also as I commented before making this added change I would add a photo to help the user easily identify this section. Great work!
| @@ -4,7 +4,9 @@ | |||
| Author's custom styles | |||
| ========================================================================== */ | |||
|
|
|||
|
|
|||
| #image-galleries .btn { | |||
There was a problem hiding this comment.
I would make the change to image-gallery instead of galleries since it is a singular gallery or archive.
| <h2>Heading</h2> | ||
| <p>Donec sed odio dui. Cras justo odio, dapibus ac facilisis in, egestas eget quam. Vestibulum id ligula porta felis euismod semper. Fusce dapibus, tellus ac cursus commodo, tortor mauris condimentum nibh, ut fermentum massa justo sit amet risus.</p> | ||
| <p><a class="btn btn-default" href="#" role="button">View details »</a></p> | ||
| <div id="image-galleries" class="col-md-4"> |
There was a problem hiding this comment.
<div id="image-gallery" class="col-md-4">
Added image search section to main page. Closed the following issue:
#3