Update README.md#1469
Conversation
Merged the 'contribute' and 'feedback' sections as they're all part of getting actively involved. Added a 'contributors' header
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Copied some text over from https://github.com/UniversalViewer/universalviewer/wiki/About
More text tweaks designed to help newcomers to the UV understand what it is and how they can get involved
LanieOkorodudu
left a comment
There was a problem hiding this comment.
@mialondon and @erinburnand, Just a suggestion: In line 60 maybe we could include the word ‘collaborative’ to reflect our ongoing goal of fostering collaboration. ‘A collaborative community of developers, testers, and institutional users has grown around the Universal Viewer.’ Let me know what you both think before I approve. Otherwise, it looks good and well-defined to me if you’re both happy with the way it is. Thanks.
Updated after a suggestion from @LanieOkorodudu
|
@LanieOkorodudu great suggestion, thank you! I've made that tweak and added a little more context for the blog post |
|
Thanks so much @mialondon, We just need a final input from @demiankatz to confirm if everything looks good with the update. Always great to have more eyes on it, really appreciate it! Hope you don’t mind me tagging you here, @demiankatz. |
demiankatz
left a comment
There was a problem hiding this comment.
See below for one set of suggestions.
Also, since we're changing some headers around, might we want to consider adding "legacy" anchors for the old headings, in case people have linked directly to these sections from other pages?
Please let me know if you need assistance with technical implementation of anything.
| ## 📖 License | ||
|
|
||
| The Universal Viewer is released under the [MIT license](https://github.com/UniversalViewer/universalviewer/blob/master/LICENSE.txt). | ||
| <a href="https://github.com/UniversalViewer/universalviewer/graphs/contributors"><img src="https://opencollective.com/universalviewer/contributors.svg?width=890&button=false" /></a> |
There was a problem hiding this comment.
For consistency, we should probably use Markdown format rather than HTML to embed this image.
We should be sure to include alt text for consistency.
Maybe to clarify things, the above graphic should be "Institutional Contributors" and this graphic should be "Individual Contributors".
There was a problem hiding this comment.
I didn't want to muck with the page too much, but yes, it'd be good to refresh it all to make it more consistent at some point! In the meantime, Institutional/Individual Contributors is an easy change to make.
There was a problem hiding this comment.
I tried 
There was a problem hiding this comment.
How do you feel about [](https://github.com/UniversalViewer/universalviewer/graphs/contributors)?
...I think the one you didn't like was missing the arguments on the URL, which is why it was so tiny.
There was a problem hiding this comment.
Note that I just revised the Markdown in the above comment to make the link work correctly too. :-)
Added individual / institutional headers
Added anchors for legacy links
Updated Indiv contribs image thanks to @demiankatz
demiankatz
left a comment
There was a problem hiding this comment.
Thanks, @mialondon, I think this is in great shape now!
I'll merge once @LanieOkorodudu approves of the final version.
LanieOkorodudu
left a comment
There was a problem hiding this comment.
And thanks @mialondon you for updating the README. It’s all set to merge @demiankatz. I appreciate everyone's effort and time. Thanks everyone! :-)
In preparation for presenting the UV to new folk at the Digital Humanities conference in mid-July, some small changes: