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

doc: TeX Live to CommonMark #105270

Merged
merged 6 commits into from Dec 3, 2020
Merged

doc: TeX Live to CommonMark #105270

merged 6 commits into from Dec 3, 2020

Conversation

lucasew
Copy link
Contributor

@lucasew lucasew commented Nov 28, 2020

There was no hint about the date and the original author in the XML so I just put the title in the markdown frontmatter

Signed-off-by: GitHub <noreply@github.com>
Copy link
Member

@ryantm ryantm left a comment

Choose a reason for hiding this comment

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

Checked the HTML output and it looks good! Just some Markdown style things to take care of. Thank you.

doc/languages-frameworks/texlive.section.md Outdated Show resolved Hide resolved
doc/languages-frameworks/texlive.section.md Outdated Show resolved Hide resolved
doc/languages-frameworks/texlive.section.md Outdated Show resolved Hide resolved
doc/languages-frameworks/texlive.section.md Outdated Show resolved Hide resolved
lucasew and others added 4 commits December 1, 2020 13:44
Co-authored-by: Ryan Mulligan <ryan@ryantm.com>
Co-authored-by: Ryan Mulligan <ryan@ryantm.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
```ShellSession
$ nix repl
nix-repl> :l <nixpkgs>
nix-repl> texlive.collection-[TAB]
Copy link
Contributor

Choose a reason for hiding this comment

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

Previously, we had a tab key symbol here (though it does not seem to be properly styled in the current stylesheet). Not sure if we should try to add it.

Otherwise, the diff looks fine and should be ready to merge after squash.

Copy link
Member

Choose a reason for hiding this comment

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

I think that [TAB] is an improvement over that obscure character.

Copy link
Contributor

@jtojnar jtojnar Dec 3, 2020

Choose a reason for hiding this comment

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

The issue is not the textual representation but rather the graphical one. Compare

nix-repl> texlive.collection-→|

with

nix-repl> texlive.collection-[TAB]

Copy link
Member

Choose a reason for hiding this comment

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

Oh yeah, it looks a lot better with that styling.

@ryantm ryantm added this to In progress in Conversion to CommonMark Docs Dec 3, 2020
@ryantm ryantm linked an issue Dec 3, 2020 that may be closed by this pull request
@ryantm ryantm merged commit 543f084 into NixOS:master Dec 3, 2020
@ryantm ryantm moved this from In progress to Done in Conversion to CommonMark Docs Dec 3, 2020
@ryantm ryantm moved this from Done (add to next Discourse update) to Completely done in Conversion to CommonMark Docs Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Conversion to CommonMark Docs
Done (mentioned on Discourse)
Development

Successfully merging this pull request may close these issues.

Convert Texlive from Docbook to Commonmark
4 participants