New contentHash parser breaks hash hrefs

[bermanjosh]

I think #198 breaks the following:
<a #logoutbutton href=#>

It returns the following error:

unexpected "h"
expecting ">"

[snoyberg]

Pinging @terickson001, I'm probably going to have to revert the change.

@bermanjosh interested in sending a PR for an additional test case to prevent this regression in the future?

[terickson001]

Sorry about that, overlooked Attributes a bit when I was testing. Should be fixed in this PR.

[joeyh]

I'm getting what seems like the same kind of reversion with 2.0.12

<a .btn .btn-default data-toggle="collapse" data-target="##{ident}">#{toggle}

unexpected "d"
expecting ">"

I think the 'd' is from data-target.

[terickson001]

@joeyh That is fixed in PR #201, I just tested it to be sure.

[bermanjosh]

@terickson001 Thanks!
@snoyberg Ok if I close the issue?

[snoyberg]

Yes, sounds good, I'll merge and release shortly.

…

On Wed, Dec 28, 2016, 9:56 AM Josh Berman ***@***.***> wrote: @terickson001 <https://github.com/terickson001> Thanks! @snoyberg <https://github.com/snoyberg> Ok if I close the issue? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#200 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AADBB9JrHWMXqWt3bLG5kkjRJH4NaArhks5rMhYggaJpZM4LWIPL> .