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 new cite.html #2195
add new cite.html #2195
Conversation
As discussed during the meeting today. I will follow up with PRs for linking to it in readme/manual/etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! Looks good except for the few small changes. Feel free to merge yourself once you've addressed them.
cite.html
Outdated
</p> | ||
|
||
<p><b>3. Citation</b><br> | ||
Please cite one of all of the following documents. The canonical reference for this software: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one or all
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And can you put the sentence "The canonical reference for this software" into a new line?
index.html
Outdated
@@ -111,6 +111,18 @@ <h2> | |||
</td> | |||
</tr> | |||
|
|||
<tr> | |||
<td style="vertical-align:top;height:4em"> | |||
<a class="btn btn-success" href="cite.html">Cite! »</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The other green boxes all have a different size. I think that's because they have the style="width:100%"
code? Can you fix that?
fixed. I am merging so we can take a look at it... |
I think it looks good. |
No description provided.