facebook/lexical

Feature: support empty line in markdown

Open

#4.838 geöffnet am 28. Juli 2023

Auf GitHub ansehen
 (4 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)JavaScript (16.523 Stars) (1.349 Forks)batch import
contribution-weekenhancementgood first issuemarkdown

Beschreibung

In https://playground.lexical.dev/ if you enter multiple line breaks and toggle markdown twice. The empty lines will be eaten.

I checked the code https://github.com/facebook/lexical/blob/main/packages/lexical-markdown/src/MarkdownImport.ts#L80-L81 and I guess it's not implemented?

Contributor Guide

Tech Stack
javascripttypescript
Domain
frontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
basic JavaScriptLexical markdown plugin basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75
Research-Richtung
The issue is in MarkdownImport.ts at lines 80-81 where empty lines are not handled. Investigate how the markdown import parser currently processes line breaks and why empty lines are being eaten. Check the markdown export logic in MarkdownExport.ts for consistency. The solution likely involves adding support for empty lines in the import regex or parsing logic to preserve them as separate paragraphs.