Add Image Search Section #16
Conversation
Add image search section to main page
ghost
self-assigned this
ghost
requested review from
KMcWeeney22,
hlkessner,
kevinsheen,
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. |
lariosw
left a comment
lariosw
left a comment
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.
lariosw
left a comment
lariosw
left a comment
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!
| ========================================================================== */ | ||
|
|
||
|
|
||
| #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">
2 participants
Added image search section to main page. Closed the following issue:
#3