Js-beautify: Le formateur HTML envoie des commentaires sur une nouvelle ligne lorsque le commentaire n'inclut pas d'espace comme premier caractère

Créé le 29 juil. 2020  ·  4Commentaires  ·  Source: beautify-web/js-beautify

La description

Le formateur HTML envoie des commentaires à une nouvelle ligne lorsque le commentaire n'inclut pas d'espace comme premier caractère. Cela fonctionne correctement lorsque le commentaire HTML commence par n'importe quel caractère d'espace blanc. Je crois que le problème est ici : https://github.com/beautify-web/js-beautify/blob/master/js/src/html/beautifier.js#L698

Problèmes liés : #1337 #426

Contribution

Le code ressemblait à ceci avant l'embellissement :

<!--start button--><button>Click here</button><!--end button-->

Production attendue

Le code aurait dû ressembler à ceci après embellissement :

<!--start button--><button>Click here</button><!--end button-->

Sortie réelle

Le code ressemblait en fait à ceci après l'embellissement :

<!--start button--><button>Click here</button>
<!--end button-->

Pour que cette fonction soit correctement ajoutée, ajoutez un espace au début du deuxième commentaire.

Contribution

Le code ressemblait à ceci avant l'embellissement :

<!--start button--><button>Click here</button><!-- end button-->

Sortie réelle

Le code ressemblait en fait à ceci après l'embellissement :

<!--start button--><button>Click here</button><!-- end button-->

Environnement

J'ai testé ça sur beautifier.io

Paramètres

{
  "indent_size": "2",
  "indent_char": " ",
  "max_preserve_newlines": "-1",
  "preserve_newlines": false,
  "keep_array_indentation": false,
  "break_chained_methods": false,
  "indent_scripts": "normal",
  "brace_style": "collapse",
  "space_before_conditional": false,
  "unescape_strings": false,
  "jslint_happy": false,
  "end_with_newline": false,
  "wrap_line_length": "0",
  "indent_inner_html": true,
  "comma_first": false,
  "e4x": false,
  "indent_empty_lines": false
}
good first issue html bug

Commentaire le plus utile

J'aimerais travailler sur ce @avigoldman @aashigupta19

Tous les 4 commentaires

Cela ressemble à #1301, mais ce n'est pas la même chose.

Ce problème est encore plus apparent si pour quelque chose comme :

<span><!--start button--><button>Click here</button><!--end button--></span>

Qui se transforme en :

<span>
    <!--start button--><button>Click here</button>
    <!--end button-->
</span>

Cela ajoute un espace entre <span> et <button> et un espace à l'intérieur du <button> . :sanglot:

J'aimerais travailler sur ce @avigoldman @aashigupta19

@cfya
Je vous en prie!

Cette page vous a été utile?
0 / 5 - 0 notes