-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Incorrectly removed whitespaces #48
Comments
potential fix choojs/nanohtml#74 |
So what's the solution here? The "fix" applied to bel contains a whole bunch of things and it's a bit hard to extract what is needed to fix the issue in yo-yoify. Can you help @yoshuawuyts? |
Gotta resolve this; it's a priority of mine rn
choojs/nanohtml#77
…On Tue, May 30, 2017 at 10:02 AM Maximilian Antoni ***@***.***> wrote:
So what's the solution here? The "fix" applied to bel contains a whole
bunch of things and it's a bit hard to extract what is needed to fix the
issue in yo-yoify. Can you help @yoshuawuyts
<https://github.com/yoshuawuyts>?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#48 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ACWlegtZg5X_HpX1u9rl5-Wa31WKXWZTks5r-80YgaJpZM4NZDXG>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As commented on PR #45, some layouts are screwed up now (I should have opened an issue in the first place). Consider this:
This should render as "Yes yo". With the change from #45 it becomes "Yesyo".
You can replace multiple whitespace characters with a single blank though. That'd be fine.
The text was updated successfully, but these errors were encountered: