-
-
Notifications
You must be signed in to change notification settings - Fork 2
Updating formatting in ABOUT.md for Nextercism #84
Comments
|
Can you please choose a markdown parser that supporst this? I strongly disagree with single lines that span thousands of thousands of characters which will probably blow many editors as well! If you have paragraphs that are longer than only 2 or 3 sentences its quite aburden to find the correct place when there's a need to update something. |
Paragraphs should be characterized by a separation of 2 new lines. (to clarify, a new line, and then a line with only a new line on it) Any single lines that are not separated by two new lines should be considered one paragraph. Since early years in writing this has been the case. This is just how plain text works, even on paper. |
For what it's worth, GitHub issues don't work like this. this |
Not even talking about generated diffs… |
But rendering Just take a look at https://raw.githubusercontent.com/exercism/cli/master/README.md and compare how it is rendered on github or when you post it as a comment. In a comment the paragraphs appear split up while when beeing rendered multiline texts appear as a single paragraph. |
Ah, yepp. I see that now. OK, cool. Yes, I would suggest that we see if we can tell the markdown rendering engine to deal with paragraphs in this way. |
So we can change this but two things to raise:
As long as we consider those two things to be ignorable, then I'll change it. |
My 2 cents on this:
My vote is to enforce no breaks within a paragraph for easier maintenance. I haven't dealt with hard-wrapped markdown documents outside of Exercism and I have not enjoyed the re-formatting required when doing so. |
I wholeheartedly agree with @ryanplusplus and that's my vote too. |
I think the rendered line wrappings is normal for the medium to take care of, the only thing it should understand is "a paragraph is words separated by two new line characters" and wrap as it sees fit. Are there examples of deliberately specified line-breaks? Do we need them? Do we need them to not be considered a new paragraph? |
Ah, @NobbZ when I hear "paragraph" I see it as Vim sees it. I see it as on paper. Your example with the shorter lines, I see as common to prose, a poem with a shape... that is a paragraph. (Even as rendered by HTML, with the br tags). In HTML, it does not honor the breaks, like that on the editor it is tagged as such, in markdown, it is plain text,. I am starting to see the difference, perhaps harder to know the intent for display. I did not consider writing prose. |
For what it's worth, I always find it a huge pain to edit files with hard lines, because my editor settings are not typical. So basically, what @ryanplusplus says. That said, I will happily do whatever we end up deciding, for the sake of consistency. |
As someone who uses MD on a daily basis, I find it even more confusing when…
We don't. Even if I do not like it how commonmark handles it, but common mark does render a trailing space as a line break (
Yupp, even emacs does. But working with overly long lines in emacs and vim (at least when used on CLI and not extended via plugins) is a pain, since cursor up/down means skipping a line, not "advance as much as necessary, to reach the next soft-wrapped line on screen". We should consider such tools as well, some folks are still using them, eg. @kotp, @NobbZ
Doesn't your editor support clicking the preview and setting the cursor appropriately? Even if I have a wide screen, and often use it fullwidth, I prefer to not have much more than about 120 characters in a line. Its just hard on the eyes to follow those lines…
Editing manually broken lines? Why? Common sense is to have a sentence per line. Easy to read, easy to edit and nicely
I'm not sure what vim thinks a paragraph were, but for me a paragraph is all the stuff I'd put no empty line inbetween when writing markdown, LaTeX or plaintext (email), or enclose in I accept though, that one sometimes wan't to have extra linebreaks inbetween a paragraph. Thats when I use trailing spaces in MD (or use The closer we stick to common-mark, the less bug reports will be filed by experienced markdown users. |
Can you explain to me how someone would insert a manual linebreak please? For example, how would I make the top three lines split?
|
In case this wasn't clear:
Because a lot of people break at 80 columns whether or not it's the end of the sentence. Then when I change the paragraph in any significant way I have to combine the lines into one, and rebreak it to respect the 80 columns limit.
I would be happy with this approach.
You'd explicitly add |
I use search to jump around in long, wrapped lines. It's not ideal, but it's better than joining all the lines and rebreaking them, as I often end up having to do. |
I think clearly defining a paragraph as any text delimited with a new line that has only a new line character. I think the sentence per line tactic is sensible. Using a space at the end of a line that you want broken will work. I prefer using the html tag Like @kytrinyx I also would join the paragraph when reflowing it, it ends up being two characters to press in my vim, it translates to "join and reflow this paragraph". I map to |
Either use
The semantic meaningful trailing whitespace has some problems though:
In emacs it's In fact, I try to avoid reflowing completely:
|
It is true, for sure, during reflow, it does create some noise. This would indicate that single sentence per line, paragraph separated by empty new line, makes things easier in many aspects. |
Crystal follows this spec. EDIT: Crystal's |
I do not see a spec here, only a living discussion about how to treat paragraphs and lines. Also you have at least 2 places that are rendered with an additional linebreak according to the rules presented in the OP: **Local:**
The problem-specifications repo must reside at the same level as the developer's crystal directory, and must contain a canonical-data.json file for the given exercise.
**Remote:**
If the local problem-specifications repo doesn't exist, a request is made to GitHub to download (and cache) the test's canonical-data.json file. New:
Current:
|
It feels like the best of both worlds here is the following:
This will avoid the noisy (and painful) reflows, and also allows fairly reasonable navigation by line for editors where that is a thing. For editors that soft-wrap, each sentence will wrap, but you won't get those really weird things where part of a sentence is broken up across multiple lines. Sound acceptable? |
Ah... I was just looking at the
Well... not an exact spec but I see what you're saying.
+1 for that. |
The OP has been updated to reflect what @kytrinyx has proposed. @mhelmetag - Crystal currently renders with paragraphs inside bullet points due to the newlines between points. Could you remove the spacing between those pls? |
@kytrinyx In vim, do you know you can visually highlight the paragraph and then press |
@robphoenix I did not! Going to go try that right now. |
Oh. My. Goodness. |
@robphoenix I love that technique. I use Actually, I guess what I use is |
Pretty much my same reaction.
I know! I can't believe I used to manually reformat every line in a paragraph after changing one word! |
Ok, it sounds like there's no great pushback on the suggested solution, so I'm going to call it. This is the plan:
@iHiD Would you implement this into the prototype? |
@iHiD Also: let's open a new, clean issue to keep track of things—this one is going to be hard for people to read through and parse. |
Hello @exercism/track-maintainers.
In preparation for the new look and feel of nextercism I'm trying to get all of the ABOUT.md files reformatted. This means removing things like line-breaks and markdown that we're not supporting.
As an example from the C++ Track, I've proposed going from:


to:
In summary, ABOUT.md should now only use:
used to split a paragraph into lines without spacing between them (discouraged).
This is a checklist of the tracks that need doing. I am trying to send PRs to tracks that need updating, but if maintainers are happy to do it themselves that will save me hours of time. If your ABOUT.md already complies with these formatting rules, could you please comment and and I will tick it off the list. Thank you 💙
The text was updated successfully, but these errors were encountered: