CommonMark testing started here!

Shouldn’t those render as test?.. and test!.. (i.e. three dots rather than two)?

Strongly depends on which color you like your bike shed :smile:

8 Likes

I guess the idea is that there are indeed three dots, but one of them is part of the question mark (or exclamation mark) :wink:

?.. !..

4 Likes

Not quite. Typographically, three dots are an ellipsis and and ellipsis is always three dots. There is no two to ellipsis and I am not aware of any style guide that suggests that the question mark or exclamation mark can replace one of the dots. By contrast, there are plenty of recommendations that suggest the use of a punctuation mark plus three dots (and, while we’re at it: to separate them from the punctuation mark with a space):

1 Like

I suggest you open this on the markdown it Github issues page cause we are simply consuming the functionality

6 Likes

Woo markdown tables!

1 Like

I did. They are using Russian typography and they’re basically sending the ball back to you by answering:

you can disable replacer and add your own if you don’t like existing one.

https://github.com/markdown-it/markdown-it/issues/376#issuecomment-311588034

We do not care about this specific issue and will not be working on it.

1 Like

The hide details feature doesn’t work properly on meta:

Summary

This text will be hidden

Now we just need a nifty composer option to insert a table and we’ll be golden. :pray:

You need to write it

[details=Summary]
This text will be hidden
[/details]
Summary

This text will be hidden

1 Like

Makes sense. Then the composer options menu needs a tweak to fix how it inserts it.

7 Likes

Thanks… fixed per:

https://github.com/discourse/discourse/commit/c1560d8195f40dc048b8ad605ef8a67a516c64d6

Should be deployed here shortly.

6 Likes

It looks like the right align in the table is not working:

|  Tables  |      Are      | Cool |
|---|:---:|---:|
| col 1 is |  left-aligned | $100 |
| col 2 is |    centered   |  $52 |
| col 3 is | right-aligned |   $9 |
Tables Are Cool
col 1 is left-aligned $100
col 2 is centered $52
col 3 is right-aligned $9
3 Likes

Yeah, our sanitizer needs a fix there, added to my list

5 Likes

And table alignment stuff is fixed :sparkles:

I am now almost done with core and core plugin changes.

Only little bit left is some extra BBCode support, which I will work on tomorrow.

To celebrate I have exposed the site setting to enable CommonMark

https://github.com/discourse/discourse/commit/fbb5600c8e68f59fba0bcf89708fdcfff5ca18a9

9 Likes

It seems that footnotes are not working yet. :frowning:

Footnote[^easy]

[^easy]: One footnote

Option Description
data path to data files to supply the data that will be passed into templates.
engine engine to be used for processing templates. Handlebars is the default.
ext extension to be used for dest files.
neu Footnote 1 link[^first]

[^first]: Footnote can have markup

It is a plugin per: https://github.com/markdown-it/markdown-it-footnote

After I am done fixing this BBCode stuff I am going to have to clean up existing plugins. Once that is done I will do some posts that demonstrate how to re-package plugins, it is really easy.

8 Likes

Here’s a weird :bug:

This is [a link with a #bug](). 

will render as :arrow_down_small:

This is a link with a #bug.

5 Likes

Cool, nice report, fixed per:

https://github.com/discourse/discourse/commit/c6b83f47140b4685c4a8f7a466d1e5673f11f898

2 Likes