Skip to content

Conversation

@Skillmon
Copy link
Member

Just the basics of the tabular environment. Might need extensive rework.

@Skillmon Skillmon mentioned this pull request Mar 22, 2020
en/lesson-8.md Outdated
to as table preamble – to the `tabular` environment, in which you specify
the columns by using single letter names. The available column types are:

`l`
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we have Frank's new w type column here?

Copy link
Member Author

Choose a reason for hiding this comment

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

@josephwright IIRC, it is contained in array.sty not in the kernel.

Copy link
Contributor

Choose a reason for hiding this comment

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

Even though the array extension requires using a package I think it is is so much core that it should be recommended upfront (caveat: only commenting on the level of comments above, not yet looked at what or how somehting is described)

Copy link
Member Author

Choose a reason for hiding this comment

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

@josephwright something else: The used MD-Syntax in this is for descriptions (\n: ), but it's not part of core-markdown and I have no idea whether it'll be supported on learnlatex. It might be necessary to reformulate this as a markdown table, which are also not part of core-markdown, or an inline HTML table if absolutely necessary.

Copy link
Contributor

Choose a reason for hiding this comment

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

@Skillmon We are using GitHub Pages behind-the-scenes: whatever works in that regard is fine. (I've no desire to run Jekyll locally)

@josephwright
Copy link
Contributor

@Skillmon Sure but I think we should just say 'load array in all cases': really it's one of those 'would have been in the kernel if the team were not short of tokens in 1994' things

@PhelypeOleinik
Copy link
Contributor

I think the lesson should also cover \multirow, shouldn't it? It's not
in the kernel, yes, but it's a fairly common thing to use in tables.

@josephwright
Copy link
Contributor

Do we really want \multirow? It's usually a sign of a badly-constructed table

@Skillmon
Copy link
Member Author

I'd vote against \multirow as well. In most cases, working the other way around gives better and more predictable results. \multirow always feels like a poor man's solution, imho. I'd rather include instructions of \makecell or tell the users to use a nested tabular.

@PhelypeOleinik
Copy link
Contributor

Well, yes, abominations can be created there, but it is useful
sometimes. Though perhaps not often enough to be worth the effort.
I yield :-)

@josephwright
Copy link
Contributor

@PhelypeOleinik We all get to agree the curriculum; it's complicated :)

@Skillmon
Copy link
Member Author

Is \newcolumntype in or out?

@josephwright
Copy link
Contributor

@Skillmon Does it fall within strictly beginners? I'm flexible on that, but I'd want an example that's common enough

@Skillmon
Copy link
Member Author

@josephwright I did not include it in that commit. I frequently use it :) I think one of the "classics" is something like \newcolumntype{L}[1]{>{\RaggedRight\arraybackslash}p{#1}} (or using \raggedright) if you need a fixed width column that'll be too narrow for full justification.

@FrankMittelbach
Copy link
Contributor

Well, yes, abominations can be created there, but it is useful
sometimes. Though perhaps not often enough to be worth the effort.

there has to be something left for TLC :-) else all the work there is in vain :-) But no I don't think we want anything more than the basics initially. My point was that I think "array" is basics. It is only not in the kernel because it fixes the rules andso does change the table width slightly.

And no, I don't think \newcolumntype should be in.

@josephwright
Copy link
Contributor

@Skillmon I guess it depends on how you look at it; for me, I use tables for pretty limited cases, and they are all one-liners. As @FrankMittelbach says, the aim of the site is (for me) to get people over the very first day or so of usage. Lots and lots of people need some form of table, enough to include them, but beyond that it's more complex.

@FrankMittelbach
Copy link
Contributor

FrankMittelbach commented Mar 23, 2020 via email

@josephwright
Copy link
Contributor

We could consider a set of 'extension lessons' for each main lesson, so going-further-8.md or similar here. That would give us a predictable structure and let us have a link on each main lesson: keep the basics clear but also give us more scope for content. I think that deserves a separate issue: I'll create one.

@Skillmon
Copy link
Member Author

If no one objects, I'll merge this into master today.


For more information on the history of TeX and LaTeX see:

*
Copy link
Contributor

Choose a reason for hiding this comment

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

what is this *doing?

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't know, I didn't add it. I merged with master because the filenames changed, I guess it was added there.

Copy link
Member Author

Choose a reason for hiding this comment

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

@FrankMittelbach can confirm that this was added to this branch in e3861e2 which was the merge of master into this branch. Didn't search for when it was added to master though. I'd guess that the author wanted to add bullet points there with other resources and used this as a placeholder.

@josephwright josephwright merged commit 55021f2 into master Mar 29, 2020
@josephwright josephwright deleted the lesson-8-basics branch April 20, 2020 07:15
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.

4 participants