In a LiveView, compact: true does not seem to work

[Bastes]

Describe the bug
I'm using compact: true to avoid whitespaces in the .alert tags showing the flashes. The tags still get spaces inside in a liveview.

To Reproduce

# in a non-liveview template
p compact: true, do: nil
p compact: true, do: ""

# in a liveview template
p compact: true, do: nil
p compact: true, do: ""

In the non-liveview template, the paragraphs will have no whitespaces inside in the browser.
In a liveview template, the paragraphs will have two newline chars in the browser.

Expected behavior
All examples should keep no whitespace inside

Desktop (please complete the following information):

• OS: Ubuntu 20:04 ; in a docker container (elixir:latest at the time of writing)
• Temple Version: master branch (at the time of writing)
• Elixir Version: 1.10.4
• Erlang Version: Erlang/OTP 22 [erts-10.7.2.3]

Additional context
Reading the code, I don't believe this is a purely temple-centric problem ; it might be LiveView adding whitespaces while massaging the dom or something.

[mhanberg]

Can you try this with a normal leex template to see if you get the same behavior?

[Bastes]

Good idea. Will do as soon as possible.

[Bastes]

Tested in pure html .leex templates: the html shows no complementary whitespaces.

[mhanberg]

Thanks for checking that. In the mean time, you can use elixir to conditionally show your flash message instead of css ```elixir if get_flash(@conn, :info) do # my markup end ```

…

On Oct 30, 2020, at 10:16 AM, Michel Belleville ***@***.***> wrote: Tested in pure html .leex templates: the html shows no complementary whitespaces. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#100 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKEUECV6RVKTHGWJ4VWH3TSNLDDHANCNFSM4TD6R4HQ>.

[Bastes]

Sure, I'll be trying that ; I take it that will work with live_flash?

[mhanberg]

Yes

…

On Oct 30, 2020, at 11:14 AM, Michel Belleville ***@***.***> wrote: Sure, I'll be trying that ; I take it that will work with live_flash? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#100 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKEUEFRQFVFHVO5TGSQUWLSNLJ5NANCNFSM4TD6R4HQ>.

[Bastes]

Hmm... I don't quite know what happened, but after removing the .leex templates I used to test that they did not add unnecessary spaces, it appears the compact: true attribute is now working as intended.

I'm completely puzzled as to why it did not do that yesterday, but maybe there was some cache keeping the previous templates (without the compact: true) in memory and now I've reloaded the server it doesn't have them? Anyways, I suggest we close this issue and sorry for the noise.