Skip to content

feat: Integrate contact card feature - #2997

Open
Elouan1411 wants to merge 11 commits into
mainfrom
virtual-card
Open

feat: Integrate contact card feature#2997
Elouan1411 wants to merge 11 commits into
mainfrom
virtual-card

Conversation

@Elouan1411

@Elouan1411 Elouan1411 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Copilot AI review requested due to automatic review settings July 23, 2026 16:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Integrates the Core contact-card experience into the account switcher, including navigation, theming, accessibility, persistence, and QR presentation.

Changes:

  • Adds contact-card navigation and account-switcher entry points.
  • Integrates the Core Compose contact-card UI and preserves card data during profile refreshes.
  • Adds localized accessibility text, visual assets, theming, and dependency licensing.

Reviewed changes

Copilot reviewed 31 out of 32 changed files in this pull request and generated no comments.

Show a summary per file
File Description
LICENSES.md Documents the QRose dependency.
app/build.gradle.kts Adds the Core contact-card module.
app/src/main/java/com/infomaniak/mail/ui/bottomSheetDialogs/AccountBottomSheetDialog.kt Opens contact cards from the account sheet.
app/src/main/java/com/infomaniak/mail/ui/components/compose/MailTopAppbar.kt Adds title and container-color support.
app/src/main/java/com/infomaniak/mail/ui/main/contactcard/ContactCardFragment.kt Hosts the contact-card Compose screen.
app/src/main/java/com/infomaniak/mail/ui/main/folder/ThreadListFragment.kt Enables contact cards in the main account sheet.
app/src/main/java/com/infomaniak/mail/ui/main/user/SwitchUserAdapter.kt Adds QR interaction and selected-account styling.
app/src/main/java/com/infomaniak/mail/ui/noValidMailboxes/NoValidMailboxesFragment.kt Disables contact cards in the invalid-mailbox flow.
app/src/main/java/com/infomaniak/mail/ui/theme/MailThemeColors.kt Defines Mail-specific Compose colors.
app/src/main/java/com/infomaniak/mail/ui/theme/Theme.kt Provides the new theme colors.
app/src/main/java/com/infomaniak/mail/utils/AccountUtils.kt Preserves locally stored card data.
app/src/main/res/drawable/bg_reminder_icon.xml Adds the QR icon background.
app/src/main/res/drawable/ic_pencil_filled.xml Adds the filled compose icon.
app/src/main/res/drawable/ic_qrcode.xml Adds the QR vector icon.
app/src/main/res/layout/fragment_thread_list.xml Uses the filled compose icon.
app/src/main/res/layout/item_switch_user_account.xml Adds the contact-card QR control.
app/src/main/res/navigation/main_navigation.xml Adds contact-card navigation.
app/src/main/res/navigation/no_valid_mailboxes_navigation.xml Adds account-sheet configuration.
app/src/main/res/values/strings.xml Adds English QR accessibility text.
app/src/main/res/values-da/strings.xml Adds Danish localization.
app/src/main/res/values-de/strings.xml Adds German localization.
app/src/main/res/values-el/strings.xml Adds Greek localization.
app/src/main/res/values-es/strings.xml Adds Spanish localization.
app/src/main/res/values-fi/strings.xml Adds Finnish localization.
app/src/main/res/values-fr/strings.xml Adds French localization.
app/src/main/res/values-it/strings.xml Adds Italian localization.
app/src/main/res/values-nb/strings.xml Adds Norwegian localization.
app/src/main/res/values-nl/strings.xml Adds Dutch localization.
app/src/main/res/values-pl/strings.xml Adds Polish localization.
app/src/main/res/values-pt/strings.xml Adds Portuguese localization.
app/src/main/res/values-sv/strings.xml Adds Swedish localization.

@github-actions github-actions Bot added the dependent This MR depends on another PR label Jul 24, 2026
@github-actions

Copy link
Copy Markdown

This PR/issue depends on:

@Elouan1411
Elouan1411 force-pushed the virtual-card branch 2 times, most recently from b47d8bb to 400e448 Compare August 3, 2026 15:08
@Elouan1411
Elouan1411 force-pushed the virtual-card branch 8 times, most recently from d0cb0d8 to 23990d1 Compare August 12, 2026 13:25
@Elouan1411
Elouan1411 force-pushed the virtual-card branch 5 times, most recently from 506898d to d49b772 Compare August 14, 2026 11:41
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependent This MR depends on another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants