Agregar la opción `fromPrettyPrint` a `toJsSourceCode`

codenautas / js-from-html

js-to-html reverse

MIT License

0 stars 0 forks source link

Agregar la opción `fromPrettyPrint` a `toJsSourceCode` #8

Open emilioplatzer opened 8 years ago

emilioplatzer commented 8 years ago

Para que ignore los textos (cuando están solo conformados por blancos, tabs y fin de línea) entre elementos de display:'block' (el dato puede sacarse de jsToHtml.htmlTags)

diegoefe commented 8 years ago

En https://github.com/codenautas/js-from-html/blob/master/test/fixtures/from-pp.js#L13 (al menos hasta 2e41ba96472eea5935d3493bab2e4bb4d27d1aee) , sobraría esa línea, correcto? Por lo que hablamos en #6 y para este issue

emilioplatzer commented 8 years ago

Es cierto. Sobra.

diegoefe commented 8 years ago

Un par de cositas:

En los fixtures se llama "fromPretty". Es ese o fromPrettyPrint?
Cómo debe ignorar el texto? Debe bacer trim() (sólo principio y final) o Remover líneas vacías también?

Se entiende?

emilioplatzer commented 8 years ago

no recuerdo, que se llame de una sola manera. Si aparece de las dos buscá también en js-to-html para que sea coherente el nombre
no creo que haya que hacer trim en el medio. Para averiguarlo hacé como te dije, mirálo en el chrome a ver qué hace. Tiene que comportarse como el Chrome.