Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Public Domain image link #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ghost
Copy link

@ghost ghost commented Feb 8, 2017

Added the href tag for theimage gallery from Public Domain Pictures to the click here
button. Task is to start working on issue: #4

Still need to work on adding categories that are relevant to the user.

Added the image gallery from Public Domain Pictures to the click here
button
@ghost ghost self-assigned this Feb 8, 2017
@ghost
Copy link
Author

ghost commented Feb 8, 2017

Should we try to link the categories to those of the poetry author section?

@@ -77,7 +77,7 @@
<div class="col-md-4">
<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 &raquo;</a></p>
<p><a class="btn btn-default" href="http://www.publicdomainpictures.net/browse-category.php?c=people&s=8" role="button">View details &raquo;</a></p>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Public domain link looks good! I agree that we should link categories to poetry author section.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, I'll work on that next week.

Copy link
Collaborator

@lariosw lariosw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That sounds like a good idea. The link looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants