8
16
Fork 22

Fix #637 : Markdown footnotes #700

Zusammengeführt
papey hat 2 Commits von fix-markdown-notes nach master vor 4 Jahren zusammengeführt
papey hat vor 4 Jahren kommentiert (Migriert von github.com)

Here is fix for issue #637 that :

  • Ensure footnotes classes are not filtered by html sanitizer
  • Add some footnote style rules

An example of footnotes using this fix :

Markdown :

Here is something[^a]


Here is another thing[^b]

[^a]: something

[^b]: another thing

Footnotes rendeering

Note that there is some cpull-markdown related syntax : a carriage return is needed between each note definition

Here is fix for issue #637 that : - Ensure footnotes classes are not filtered by html sanitizer - Add some footnote style rules An example of footnotes using this fix : **Markdown :** ```md Here is something[^a] Here is another thing[^b] [^a]: something [^b]: another thing ``` ![Footnotes rendeering](https://user-images.githubusercontent.com/17273325/69247036-0d144700-0baa-11ea-83bd-bbd9fa6564c4.png) Note that there is some cpull-markdown related syntax : **a carriage return is needed between each note definition**
papey hat vor 4 Jahren kommentiert (Migriert von github.com)

Note that back reference are not supported (yet) by cpull-markdown : https://github.com/raphlinus/pulldown-cmark/issues/142

Note that back reference are not supported (yet) by cpull-markdown : https://github.com/raphlinus/pulldown-cmark/issues/142
codecov[bot] hat vor 4 Jahren kommentiert (Migriert von github.com)

Codecov Report

Merging #700 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #700   +/-   ##
=======================================
  Coverage   39.25%   39.25%           
=======================================
  Files          72       72           
  Lines        9498     9498           
  Branches     2265     2265           
=======================================
  Hits         3728     3728           
  Misses       4713     4713           
  Partials     1057     1057
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/700?src=pr&el=h1) Report > Merging [#700](https://codecov.io/gh/Plume-org/Plume/pull/700?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/7f296b2ef2fe17482f24fe84b4ecb3ae0947601f?src=pr&el=desc) will **not change** coverage. > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## master #700 +/- ## ======================================= Coverage 39.25% 39.25% ======================================= Files 72 72 Lines 9498 9498 Branches 2265 2265 ======================================= Hits 3728 3728 Misses 4713 4713 Partials 1057 1057 ```
elegaanz (Migriert von github.com) hat die Änderungen vor 4 Jahren genehmigt
elegaanz (Migriert von github.com) hat einen Kommentar hinterlassen

Thanks!

Thanks!

Reviewer

Der Pull Request wurde als 3bd2305115 gemergt.

Schritt 1:

Wechsle auf einen neuen Branch in deinem lokalen Repository und teste die Änderungen.
git checkout -b fix-markdown-notes master
git pull origin fix-markdown-notes

Schritt 2:

Führe die Änderungen zusammen und aktualisiere den Stand online auf Forgejo.
git checkout master
git merge --no-ff fix-markdown-notes
git push origin master
Anmelden, um an der Diskussion teilzunehmen.
Keine Reviewer
Kein Meilenstein
Niemand zuständig
1 Beteiligte
Nachrichten
Fällig am
Das Fälligkeitsdatum ist ungültig oder außerhalb des zulässigen Bereichs. Bitte verwende das Format „jjjj-mm-tt“.

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Keine Abhängigkeiten gesetzt.

Referenz: Plume/Plume#700
Laden…
Hier gibt es bis jetzt noch keinen Inhalt.