beautifier/js-beautify

HTML option "wrap-attributes": "preserve" does not preserve line break before closing angle bracket

Open

#1 707 ouverte le 10 sept. 2019

Voir sur GitHub
 (8 commentaires) (0 réactions) (0 assignés)JavaScript (8 342 stars) (1 460 forks)batch import
good first issuelanguage: htmltype: bug

Description

If you use "wrap-attributes": "force-expand-multiline", you get:

<tag
    attr1="val1"
    attr2="val2"
>content</tag>

The line break before the closing angle bracket is forced with this option.

Now, when you set "wrap-attributes": "preserve", I would expect this formatting to be preserved. However, you get:

<tag
    attr1="val1"
    attr2="val2">content</tag>

Please fix "preserved" or add a new option for keeping the line break before the closing angle bracket.

Guide contributeur

Stack technique
javascripthtmlcss
Domaine
toolingdeveloper experience
Type d'issue
feature
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.
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
JavaScriptjs beautify internalsHTML parsing
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Examine the HTML beautifier code in js beautify, likely in `js/lib/beautify html.js`, to understand how 'wrap attributes' options are handled. Look for the logic that distinguishes 'force expand multiline' from 'preserve' regarding line breaks before closing angle brackets. Consider adding a new option or modifying the preserve behavior to retain the line break. Check existing tests in the `test` directory for related test cases.