Merge pull request #10076 from rafaeltonholo/fix/3172/email-html-css-bad-rendering

fix(message-reader): complex html email rendering incorrectly

Author: rafaeltonholo

app-common/build.gradle.kts

@@ -49,6 +49,8 @@ dependencies {
 
     implementation(projects.feature.mail.message.export.api)
     implementation(projects.feature.mail.message.export.implEml)
+    implementation(projects.feature.mail.message.reader.api)
+    implementation(projects.feature.mail.message.reader.impl)
 
     implementation(projects.mail.protocols.imap)
     implementation(projects.backend.imap)

app-common/src/main/kotlin/net/thunderbird/android/feature/mail/message/reader/api/css/DefaultCssClassNameProvider.kt

@@ -0,0 +1,26 @@
+package net.thunderbird.android.feature.mail.message.reader.api.css
+
+import com.fsck.k9.message.html.EmailTextToHtml
+import net.thunderbird.core.featureflag.FeatureFlagProvider
+import net.thunderbird.feature.mail.message.reader.api.MessageReaderFeatureFlags
+import net.thunderbird.feature.mail.message.reader.api.css.CssClassNameProvider
+
+class DefaultCssClassNameProvider(
+    featureFlagProvider: FeatureFlagProvider,
+    override val defaultNamespaceClassName: String,
+) : CssClassNameProvider {
+    override val rootClassName: String = "${defaultNamespaceClassName}__message-viewer"
+    override val mainContentClassName: String = "${defaultNamespaceClassName}__main-content"
+    override val plainTextMessagePreClassName: String =
+        if (featureFlagProvider.provide(MessageReaderFeatureFlags.UseNewMessageReaderCssStyles).isEnabled()) {
+            "${defaultNamespaceClassName}__plain-text-message-pre"
+        } else {
+            EmailTextToHtml.K9MAIL_CSS_CLASS
+        }
+    override val signatureClassName: String =
+        if (featureFlagProvider.provide(MessageReaderFeatureFlags.UseNewMessageReaderCssStyles).isEnabled()) {
+            "${defaultNamespaceClassName}__signature"
+        } else {
+            "k9mail-signature"
+        }
+}

app-common/src/main/kotlin/net/thunderbird/app/common/feature/AppCommonFeatureModule.kt

@@ -4,6 +4,7 @@ import app.k9mail.feature.launcher.FeatureLauncherExternalContract
 import app.k9mail.feature.launcher.di.featureLauncherModule
 import net.thunderbird.app.common.feature.mail.appCommonFeatureMailModule
 import net.thunderbird.feature.mail.message.composer.inject.featureMessageComposerModule
+import net.thunderbird.feature.mail.message.reader.impl.inject.featureMessageReaderModule
 import net.thunderbird.feature.navigation.drawer.api.NavigationDrawerExternalContract
 import net.thunderbird.feature.notification.impl.inject.featureNotificationModule
 import org.koin.android.ext.koin.androidContext
@@ -14,6 +15,7 @@ internal val appCommonFeatureModule = module {
     includes(featureNotificationModule)
     includes(featureMessageComposerModule)
     includes(appCommonFeatureMailModule)
+    includes(featureMessageReaderModule)
 
     factory<FeatureLauncherExternalContract.AccountSetupFinishedLauncher> {
         AccountSetupFinishedLauncher(

app-k9mail/build.gradle.kts

@@ -142,6 +142,7 @@ dependencies {
     implementation(projects.core.ui.legacy.theme2.k9mail)
     implementation(projects.feature.launcher)
     implementation(projects.feature.mail.message.list)
+    implementation(projects.feature.mail.message.reader.api)
 
     implementation(projects.legacy.core)
     implementation(projects.legacy.ui.legacy)

app-k9mail/src/debug/kotlin/app/k9mail/featureflag/K9FeatureFlagFactory.kt

@@ -7,6 +7,7 @@ import net.thunderbird.core.featureflag.FeatureFlagFactory
 import net.thunderbird.core.featureflag.FeatureFlagKey
 import net.thunderbird.core.featureflag.toFeatureFlagKey
 import net.thunderbird.feature.account.settings.AccountSettingsFeatureFlags
+import net.thunderbird.feature.mail.message.reader.api.MessageReaderFeatureFlags
 
 class K9FeatureFlagFactory : FeatureFlagFactory {
     override fun createFeatureCatalog(): List<FeatureFlag> {
@@ -20,6 +21,7 @@ class K9FeatureFlagFactory : FeatureFlagFactory {
             FeatureFlag(MessageListFeatureFlags.UseComposeForMessageListItems, enabled = false),
             FeatureFlag(MessageViewFeatureFlags.ActionExportEml, enabled = true),
             FeatureFlag(AccountSettingsFeatureFlags.EnableAvatarCustomization, enabled = false),
+            FeatureFlag(MessageReaderFeatureFlags.UseNewMessageReaderCssStyles, enabled = true),
         )
     }
 }

app-k9mail/src/main/kotlin/app/k9mail/feature/FeatureModule.kt

@@ -5,8 +5,11 @@ import app.k9mail.feature.funding.featureFundingModule
 import app.k9mail.feature.migration.launcher.featureMigrationModule
 import app.k9mail.feature.onboarding.migration.onboardingMigrationModule
 import app.k9mail.feature.telemetry.telemetryModule
+import com.fsck.k9.BuildConfig
+import net.thunderbird.android.feature.mail.message.reader.api.css.DefaultCssClassNameProvider
 import net.thunderbird.feature.account.settings.featureAccountSettingsModule
 import net.thunderbird.feature.mail.message.list.featureMessageListModule
+import net.thunderbird.feature.mail.message.reader.api.css.CssClassNameProvider
 import org.koin.dsl.module
 
 val featureModule = module {
@@ -18,4 +21,11 @@ val featureModule = module {
     includes(featureMessageListModule)
 
     single<FundingSettings> { K9FundingSettings() }
+
+    single<CssClassNameProvider> {
+        DefaultCssClassNameProvider(
+            featureFlagProvider = get(),
+            defaultNamespaceClassName = BuildConfig.APPLICATION_ID.replace(".", "-"),
+        )
+    }
 }

app-k9mail/src/release/kotlin/app/k9mail/featureflag/K9FeatureFlagFactory.kt

@@ -7,6 +7,7 @@ import net.thunderbird.core.featureflag.FeatureFlagFactory
 import net.thunderbird.core.featureflag.FeatureFlagKey
 import net.thunderbird.core.featureflag.toFeatureFlagKey
 import net.thunderbird.feature.account.settings.AccountSettingsFeatureFlags
+import net.thunderbird.feature.mail.message.reader.api.MessageReaderFeatureFlags
 
 /**
  * Feature flags for K-9 Mail (release)
@@ -23,6 +24,7 @@ class K9FeatureFlagFactory : FeatureFlagFactory {
             FeatureFlag(MessageListFeatureFlags.UseComposeForMessageListItems, enabled = false),
             FeatureFlag(MessageViewFeatureFlags.ActionExportEml, enabled = false),
             FeatureFlag(AccountSettingsFeatureFlags.EnableAvatarCustomization, enabled = false),
+            FeatureFlag(MessageReaderFeatureFlags.UseNewMessageReaderCssStyles, enabled = false),
         )
     }
 }

app-k9mail/src/test/kotlin/app/k9mail/DependencyInjectionTest.kt

@@ -15,15 +15,18 @@ import com.fsck.k9.job.MailSyncWorker
 import com.fsck.k9.job.SyncDebugWorker
 import com.fsck.k9.mailstore.AttachmentResolver
 import com.fsck.k9.message.html.DisplayHtml
-import com.fsck.k9.message.html.HtmlSettings
+import com.fsck.k9.message.html.DisplayHtmlFactory
 import com.fsck.k9.ui.changelog.ChangeLogMode
 import com.fsck.k9.ui.changelog.ChangelogViewModel
+import com.fsck.k9.ui.helper.DisplayHtmlUiFactory
 import com.fsck.k9.view.K9WebViewClient
 import com.fsck.k9.view.MessageWebView
 import net.openid.appauth.AppAuthConfiguration
+import net.thunderbird.core.common.mail.html.HtmlSettings
 import net.thunderbird.core.preference.storage.Storage
 import net.thunderbird.feature.account.AccountId
 import net.thunderbird.feature.mail.message.list.ui.dialog.SetupArchiveFolderDialogContract
+import net.thunderbird.feature.mail.message.reader.api.css.CssClassNameProvider
 import org.junit.Test
 import org.koin.core.annotation.KoinExperimentalAPI
 import org.koin.test.verify.definition
@@ -53,7 +56,13 @@ class DependencyInjectionTest {
             injections = injectedParameters(
                 definition<AccountRemoverWorker>(WorkerParameters::class),
                 definition<ChangelogViewModel>(ChangeLogMode::class),
-                definition<DisplayHtml>(HtmlSettings::class),
+                definition<DisplayHtml>(
+                    HtmlSettings::class,
+                    CssClassNameProvider::class,
+                    List::class,
+                ),
+                definition<DisplayHtmlFactory>(List::class),
+                definition<DisplayHtmlUiFactory>(List::class),
                 definition<K9WebViewClient>(AttachmentResolver::class, MessageWebView.OnPageFinishedListener::class),
                 definition<MailSyncWorker>(WorkerParameters::class),
                 definition<SyncDebugWorker>(WorkerParameters::class),

app-thunderbird/build.gradle.kts

@@ -222,6 +222,7 @@ dependencies {
 
     implementation(projects.feature.account.settings.impl)
     implementation(projects.feature.mail.message.list)
+    implementation(projects.feature.mail.message.reader.api)
 
     implementation(projects.feature.widget.messageList)
     implementation(projects.feature.widget.messageListGlance)

app-thunderbird/src/beta/kotlin/net/thunderbird/android/featureflag/TbFeatureFlagFactory.kt

@@ -7,6 +7,7 @@ import net.thunderbird.core.featureflag.FeatureFlagFactory
 import net.thunderbird.core.featureflag.FeatureFlagKey
 import net.thunderbird.core.featureflag.toFeatureFlagKey
 import net.thunderbird.feature.account.settings.AccountSettingsFeatureFlags
+import net.thunderbird.feature.mail.message.reader.api.MessageReaderFeatureFlags
 
 /**
  * Feature flags for Thunderbird Beta
@@ -23,6 +24,7 @@ class TbFeatureFlagFactory : FeatureFlagFactory {
             FeatureFlag(MessageListFeatureFlags.UseComposeForMessageListItems, enabled = false),
             FeatureFlag(MessageViewFeatureFlags.ActionExportEml, enabled = false),
             FeatureFlag(AccountSettingsFeatureFlags.EnableAvatarCustomization, enabled = false),
+            FeatureFlag(MessageReaderFeatureFlags.UseNewMessageReaderCssStyles, enabled = false),
         )
     }
 }