Add citation and references

[rollaball]

• Please check if the PR fulfills these requirements

• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

• What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  Adds Components for Citation and References.

• What is the current behavior? (You can also link to an open issue here)

• What is the new behavior (if this is a feature change)?
  Add [Cite/] anywhere on the page with props authors, title and source.
  Then Add [Cite.References /] and it will get the list of all citations on the page

• Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
  No

• Other information:
  Issue: Add reference component #509

[rollaball]

Idyll is a markup language and toolkit for writing interactive articles. [Cite 
  url:"https://idl.cs.washington.edu/files/2018-Idyll-UIST.pdf" 
  authors:"Matthew Conlen and Jeffrey Heer"
  title:"Idyll: A Markup Language for Authoring and Publishing Interactive Articles on the Web"
/]

[Cite.References/]

[mathisonian]

This looks great @rollaball!

One use case this doesn't quite cover yet is a way to cite via an id. For example it is common to cite the same work multiple times, it would be nice if there were a way to provide a shortcut if you want to reference something again, without typing in all the information. Can you think of a clean way to do that?

[rollaball]

[Cite author:"..." title:"..." url:"..." referenceName:"..." /]
[Cite referenceName:"..." /]

@mathisonian does this seem like a good way to do this?

[mathisonian]

That works for me! Only comment is that referenceName is a little verbose, what about just name or id? Of those two id maybe is clearer than name since one might reasonable think name is referring to the author name or paper title. @fredhohman you have any comment on this?

We should probably print a warning if you provide a referenceName but it doesn't have any associated data yet.

I like this API because it doesn't seem like it would need any changes if we also add bibtex support in the future.

[fredhohman]

Nice!

I agree id is more clear, because name could be referring to a number of different fields of a citation/reference/publication.

Other examples:

• I think BibTeX calls it a "citation-key": http://www.bibtex.org/Format/
• Distill calls it a key as well. Here's link from their example article template with multiple citations as well: https://github.com/distillpub/template/blob/master/examples/article-test.html#L95

[mathisonian]

It should probably accept a date property as well that takes an arbitrary string.

[mathisonian]

Thanks for the links @fredhohman!

@rollaball would calling it key mess with react's special key property?

[rollaball]

@mathisonian
Tried it and it is messing with the special key. Will have to call it something else.

[rollaball]

@mathisonian
so what should I go ahead with? id or citation-key?

[mathisonian]

@rollaball let's go with id

[rollaball]

Hey, @mathisonian
I have made the changes

[mathisonian]

Just tested locally and everything seems to work great, thanks @rollaball! I've just left a couple small update requests and then I'm good to merge 👍 .

[mathisonian]

Updates look great - thanks @rollaball 🎉