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

re-add papers/research section #1580

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

mschwaig
Copy link
Member

@mschwaig mschwaig commented Nov 14, 2024

I re-added this section, based on https://web.archive.org/web/20170518051949/http://nixos.org/docs/papers.html and recent works I am aware of.

This is just a draft, I still need to check if I got all of the right information.

Since the original page was at http://nixos.org/docs/papers.html and this is at http://nixos.org/research, it would also be nice to set up a redirect, or change the URL.

I'm happy to review PRs related to this list.
To make that process easier maybe we should

  • add a short notice with a link before the list which sends people here
  • introduce a label for those issues

Feel free to help me work on the text!

I think @fricklerhandwerk wanted to review this.

based on https://web.archive.org/web/20170518051949/http://nixos.org/docs/papers.html
and recent works I am aware of

with some help from Claude.ai
@fricklerhandwerk
Copy link
Contributor

Nice!

Closes #830

Off the top of my mind, a seminal paper that deeply concerns Nix is Build systems a la carte.

I have a few small nits on the wording, and will add a few comments. Other than that this is really really good, and we can add more papers as we go.

</div>
<div class="ml-2">
{paper.abstract && (
<button
Copy link
Contributor

@fricklerhandwerk fricklerhandwerk Nov 18, 2024

Choose a reason for hiding this comment

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

There's no reason whatsoever for this to be javascript-only. It could easily be a <details> element with the appropriate styling. Not a blocker, but I'd strongly prefer to (eventually) not have to use any scripting for static information

Comment on lines +907 to +909
<span class="ml-2 text-sm font-bold text-gray-400 move-button">
CITE
</span>
Copy link
Contributor

Choose a reason for hiding this comment

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

Without scripting this could lead to a BibTex raw file (not a blocker)

Copy link
Contributor

@fricklerhandwerk fricklerhandwerk left a comment

Choose a reason for hiding this comment

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

This is awesome, absolutely great stuff, and it seems to keep coming. I'd merge this with the prose suggestions and would help get fixups in if you're still interested in polishing the UX.

@fricklerhandwerk fricklerhandwerk marked this pull request as ready for review November 18, 2024 13:56
Copy link
Collaborator

@thilobillerbeck thilobillerbeck left a comment

Choose a reason for hiding this comment

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

One nitpick I have is that the buttons on the publications (Preprint, Download, etc.) have a contrast of 2.53:1 when comparing with the background, which is not that great. It'd be great if you could change this to a color that at least suffices AA contrast. :)

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.

3 participants