removeUnnecessaryWhitespace should remove leading and trailing spaces

[sgara]

Hi!
First of all, thanks for this lib!
I'm just seeing a minor issue where leading (and trailing) spaces are not being removed although - as I understand it - they should be.

Simple example:
When parsing indented html document such as:

<div>
  content
</div>

Equivalent string rendered in flutter:

Html(data: "<div>\n  content\n</div>");

The resulting string gives me " content " but should be "content"

Edit: forgot to mention version! I'm using "1.0.0"

[hotlittlewhitedog]

I also have the save problem of extra leading space (v1.0.0).
Please solve it guys :)

Examples:
<h1>My header</h1>- line 1<br>- line 2<br>

Look also at your screenshot, you will see there is an extra space in front of all options 1.

[LucasJunioRG]

Same problem here (v1.0.0)

I'm using the html widget on a ListView builder and its adding spaces which breaks my list :/

[Pherne]

I have the same issue. I'm just using a p-tag. Using the whiteSpace attribute of the Style class didn't help.

Why are there no replies, yet? Is this an easily solved issue on our part? Because I couldn't find anything helpful online (maybe I just looked for the wrong thing) and I don't know how to solve this.

[hotlittlewhitedog]

I reverted my code to ^0.11.1

…

On Thu, 3 Sep 2020, 15:09 Pherne, ***@***.***> wrote: I have the same issue. Using the whiteSpace attribute of the Style class didn't help. Why are there no replies, yet? Is this an easily solved issue on our part? Because I couldn't find anything helpful online (maybe I just looked for the wrong thing) and I don't know how to solve this. [image: Screenshot_1599138322] <https://user-images.githubusercontent.com/48560128/92118802-40e9f180-edf7-11ea-8158-53199c807da7.png> — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#347 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AFMEE56CMYNN25I4XTPWQJLSD6IRDANCNFSM4OUPED4A> .

[Pherne]

@hotlittlewhitedog Thanks, that worked.

I'm just not very satisfied with this solution because it seems I'm missing some newer features now. I just hope the devs can get to this in the not too far future.

[glowinthedark]

@sgara:

With the current master branch a block such as

<p>    The objective of this..</p>

is rendered with the first character in the paragraph indented by one character as can be seen in the screenshot:

As you can see above The is not left aligned with the text on the second line and has a preceding space character.

I used the following declaration in pubspec.yaml:

dependencies:
  flutter_html:
    git:
      url: https://github.com/Sub6Resources/flutter_html.git
      ref: master