facebook/lexical

Feature: support empty line in markdown

Open

#4 838 ouverte le 28 juil. 2023

Voir sur GitHub
 (4 commentaires) (2 réactions) (0 assignés)JavaScript (16 523 stars) (1 349 forks)batch import
contribution-weekenhancementgood first issuemarkdown

Description

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?

Guide contributeur

Stack technique
javascripttypescript
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
basic JavaScriptLexical markdown plugin basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75
Direction de recherche
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.