PrismJS/prism

Keep Markup and zero-length tags

Open

#1 640 ouverte le 30 nov. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (11 488 stars) (1 296 forks)batch import
bughelp wantedplugins

Description

The current implementation of the Keep Markup plugin does not handle zero-length tags (tags with textContent.length == 0) correctly.

It will move any nested zero-length tag at the end of its parent outside and after its parent, effectively flattening the markup structure. Examples:

  • <x><y></y></x> -> <x></x><y></y>
  • <x>a<y></y></x> -> <x>a</x><y></y>
  • <x>a<y></y><z></z></x> -> <x>a</x><y></y><z></z>
  • <x>a<y><z></z></y></x> -> <x>a</x><y></y><z></z>

Guide contributeur

Stack technique
javascript
Domaine
tooling
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
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.
clear
Prérequis
Familiarity with PrismJSJavaScript knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Investigate the Keep Markup plugin in the PrismJS repository (likely in plugins/keep markup/). Look at the code that traverses the DOM and handles zero length tags. The issue describes the expected behavior: zero length tags should remain nested. To fix, identify where the plugin incorrectly detaches empty tags and adjust the logic to preserve their position. Consider adding test cases for the examples provided.