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

New lines should be interpreted. #46

Open
dchapman1988 opened this issue Jul 24, 2013 · 4 comments
Open

New lines should be interpreted. #46

dchapman1988 opened this issue Jul 24, 2013 · 4 comments

Comments

@dchapman1988
Copy link

I know there was this issue: #43

But should it not just interpret the new line?

Take my website for example: http://www.dchapman.io/ (here the code examples look wonky because the new line chars are getting replaced with spaces).

Here's what the post should look like: http://www.dchapman.io/posts/changing-column-type-in-postgresql-rails

Here's what the index looks like where truncate_html is being used: https://github.com/dchapman1988/dchapman.io/blob/master/app/views/posts/index.html.slim#L10

@hgmnz
Copy link
Owner

hgmnz commented Jul 24, 2013

Hi,

If you propose a pull request which implements the newline behavior you desire behind an option, where the default remains what it is today, that'd be a reasonable way forward.

@gcrieloue-main
Copy link

I can't wait for a truncate_html allowing line breaks. It's currently unusable with long

 code.

@jipiboily
Copy link

Did you end up fixing this for yourself? Getting the same problem for my new blog engine...

@hgmnz
Copy link
Owner

hgmnz commented Oct 8, 2014

@jipiboily nope, as far as I know, nobody has come up with a patch yet.

mbaev added a commit to mbaev/truncate_html that referenced this issue Jan 12, 2019
mbaev added a commit to mbaev/truncate_html that referenced this issue Jan 12, 2019
mbaev added a commit to mbaev/truncate_html that referenced this issue Jan 12, 2019
mbaev added a commit to mbaev/truncate_html that referenced this issue Jan 12, 2019
mbaev added a commit to mbaev/truncate_html that referenced this issue Jan 12, 2019
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

No branches or pull requests

4 participants