Email Rendering in Desktop App - Githubissues

invoiceninja / admin-portal

Invoice Ninja: Desktop/mobile admin portal built with Flutter

https://demo.invoiceninja.com

Other

1.62k stars 548 forks source link

Email Rendering in Desktop App #635

Open Axolord opened 8 months ago

Axolord commented 8 months ago

Setup

Version: v5.8.27-C155 and v5.8.27-L155
Environment: Docker

Interface

Flutter: [x]
React: [ ]
Both: [ ]

Checklist

[x] Can you replicate the issue on our v5 demo site https://demo.invoiceninja.com or https://react.invoicing.co/demo?
[x] Have you searched existing issues?
[ ] Have you reported this to Slack/forum before posting?
[ ] Have you inspected the logs in storage/logs/laravel.log for any errors?

Describe the bug

Before sending a Mail, it is not properly rendered in the preview in the desktop app. Webapp and Android app are working fine. <br/> tags dont get respected and the theme "light" also does not work, showing only text with no styling.

Steps To Reproduce

Include the following email template and preview it in the desktop app. Optionally enable the "light" theme.

<div class="center">$view_button</div>
<p>Bei Fragen stehe ich jederzeit zur Verfügung.</p>
<p>WWW<br />XXX<br />YYY <br />ZZZ</p>

Expected Behavior

The following image (I removed privacy-sensitive information like my logo and the footer, containing my phone number):

Screenshots

Email from the Desktop app looks like this (here no further information is displayed. No logo, no footer and the <br/> tags do not work. Email template is exactly the same):

turbo124 commented 8 months ago

@hillelcoren

is the flutter AP transforming the data here? I cannot replicate using React or the flutter interface will the text/html displayed

hillelcoren commented 8 months ago

The Flutter web and desktop apps have different email previews. Flutter doesn't yet support using the browser in a desktop app so we aren't able to show the full HTML preview.