Skip to content
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

Fix whitespace problem with pre tags containing brs #62

Merged
merged 1 commit into from Mar 16, 2016
Merged

Fix whitespace problem with pre tags containing brs #62

merged 1 commit into from Mar 16, 2016

Conversation

ghost
Copy link

@ghost ghost commented Mar 16, 2016

Consider this HTML:

<pre>
    <span>
void FillMeUp(char* in_string) {<br />  int i = 0;<br />  while (in_string[i] != \'\0\') {<br />    in_string[i] = \'X\';<br />    i++;<br />  }<br />}
    </span>
</pre>

In version 3 it was rendered like this:

void FillMeUp(char* in_string) {
  int i = 0;
  while (in_string[i] != \'\') {
    in_string[i] = \'X\';
    i++;
  }
}

But now in version 4 we get this:

void FillMeUp(char* in_string) {
int i = 0;
while (in_string[i] != \'\') {
in_string[i] = \'X\';
i++;
}
}

As best I can tell this has something to do with the changes to the callbackSearch array - specifically the
addition.

@mtibben
Copy link
Owner

mtibben commented Mar 16, 2016

@cameron1729 can you update the description with full details of the problem with examples? It's nice to have a clear record of changes like this in PRs.

@mtibben
Copy link
Owner

mtibben commented Mar 16, 2016

Ah right I found the description in #60

mtibben added a commit that referenced this pull request Mar 16, 2016
Fix whitespace problem with pre tags containing brs
@mtibben mtibben merged commit f55104b into mtibben:master Mar 16, 2016
@ghost
Copy link
Author

ghost commented Mar 17, 2016

Thanks for being so prompt with all these fixes 👍 :]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants