Skip to content

[Bugfix] Mobile ChatMyMessage attachment download#5753

Merged
jimchou-dev merged 7 commits intomainfrom
jimchou/bugfix-mobile-mymessage-attachment-download
Mar 28, 2025
Merged

[Bugfix] Mobile ChatMyMessage attachment download#5753
jimchou-dev merged 7 commits intomainfrom
jimchou/bugfix-mobile-mymessage-attachment-download

Conversation

@jimchou-dev
Copy link
Copy Markdown
Member

What

  • Fix issue for mobile ChatMyMessage attachment download need to press twice before download action happens (the hovering was preventing the first click action to go through on mobile)
    • Previous behaviour: When clicking first time nothing happen (tooltip shows up), second click actual download action
attachment-download-before.mov
  • After fix: First click will be download action
attachment-download-after.mov

Why

How Tested

Process & policy checklist

  • I have updated the project documentation to reflect my changes if necessary.
  • I have read the CONTRIBUTING documentation.

Is this a breaking change?

  • This change causes current functionality to break.

@github-actions
Copy link
Copy Markdown
Contributor

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot.

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 28, 2025

@azure/communication-react jest test coverage for stable.

Lines Statements Functions Branches
Base 28146 / 44726
62.92%
28146 / 44726
62.92%
788 / 1439
54.76%
2350 / 3728
63.03%
Current 28092 / 44728
62.8%
28092 / 44728
62.8%
788 / 1439
54.76%
2338 / 3716
62.91%
Diff -54 / 2
-0.12%
-54 / 2
-0.12%
0 / 0
0%
-12 / -12
-0.12%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 28, 2025

@azure/communication-react jest test coverage for beta.

Lines Statements Functions Branches
Base 58530 / 94478
61.95%
58530 / 94478
61.95%
1180 / 2698
43.73%
3577 / 5866
60.97%
Current 58473 / 94481
61.88%
58473 / 94481
61.88%
1180 / 2698
43.73%
3536 / 5836
60.58%
Diff -57 / 3
-0.07%
-57 / 3
-0.07%
0 / 0
0%
-41 / -30
-0.39%

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

Calling bundle size is not changed.

  • Current size: 12401036
  • Base size: 12401036
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

Chat bundle size is increased❗.

  • Current size: 1775263
  • Base size: 1775237
  • Diff size: 26

@github-actions
Copy link
Copy Markdown
Contributor

CallWithChat bundle size is not changed.

  • Current size: 12401048
  • Base size: 12401048
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

@jimchou-dev jimchou-dev merged commit a5a59b7 into main Mar 28, 2025
41 checks passed
@jimchou-dev jimchou-dev deleted the jimchou/bugfix-mobile-mymessage-attachment-download branch March 28, 2025 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants