Null Class value (class="") messes up the translation about html2jade

[soumendra]

Is

tr.

valid jade?

[donpark]

not sure if "tr." is valid or not.

regardless, this is a bug.

[donpark]

Fixed in version 0.1.18 and published to NPM.

[donpark]

Fix was to ignore empty class attributes.

If you have a use-case that needs output to be something like below, let me know:

tr(class="")

[soumendra]

Wow, that was fast.

This fixes everything I needed for the time being. Thanks. :)

On 21 June 2012 03:55, Don Park
[email protected]
wrote:

Fix was to ignore empty class attributes.

If you have a use-case that needs output to be something like below, let me know:

   tr(class="")

---

Reply to this email directly or view it on GitHub:
#33 (comment)

Fix was to ignore empty class attributes.

Which leads to problems on my specific case, where empty attributes won't be set to some default but should remain empty. Since these are ignored from html2jade, these are set default.

[donpark]

sorry about that.
a selfish-suggestion: just filter class="" out using sed or awk.

EDIT: Oops. just realized your issue is exact opposite. It's been too long.

Never mind, I troubleshot this by setting some placeholder class-name temporarily.
However, I suggest supporting empty attribute-values.

Thank you for your quick response!