-
Notifications
You must be signed in to change notification settings - Fork 51
Line breaks do not work #214
Comments
Yeah, a pull request would be great. Side note: both of those links in your issue are to the same page. |
Hey, I think I've fixed this, but I can't push my local branch to the git. Do I need permission from someone? |
The way pull requests work is that you fork the repo (essentially meaning you create a copy of it on GitHub), then you make your changes locally, then you push your local changes to your forked repo, then you submit a pull request so that your changes from your forked repo are requested to be integrated into the primary repo. I (or a fellow maintainer) would then review your pull request and either provide feedback or we'd merge in the pull request. |
Thanks, I've created a fork now - and hopefully a working pull request. #215 |
Also, minor code style changes. See: #214
This has been released in the latest version of Formulate. |
In brief, formulate v3.5.2 does not respect line breaks that are in a text area when submitting a contact form. This makes a carefully written email on the users end, e.g.
Turn into this instead:
I originally posted on the Umbraco forums about this issue: https://our.umbraco.com/packages/backoffice-extensions/formulate/formulate-questions//108202-formulate-line-breaks. But it also looks like other people have had similar concerns about large chunks of text becoming one line (and removing line breaks) in the comments on this other issue: #83
I've never done a pull request on somebody else's repo, but this has been bothering me for a while, so I'll certainly give it a shot at fixing it myself (but no promises).
The text was updated successfully, but these errors were encountered: