Skip to content

Fix: Sync Media Fail Notifications are expandable and copyable#20976

Open
LUwUcifer wants to merge 2 commits intoankidroid:mainfrom
LUwUcifer:syncmediaworker-notif
Open

Fix: Sync Media Fail Notifications are expandable and copyable#20976
LUwUcifer wants to merge 2 commits intoankidroid:mainfrom
LUwUcifer:syncmediaworker-notif

Conversation

@LUwUcifer
Copy link
Copy Markdown
Contributor

Purpose / Description

SyncMediaWorker fail throwable notification could not be expanded or opened

Fixes

Approach

  1. Adds attribute .bigText to make text expandable
  2. Adds addAction for a button that copies error message and instantly closes the message upon the action
  3. Uses a custom receiver for serving pendingIntent required for addAction

How Has This Been Tested?

Tested on local device

Learning (optional, can help others)

  1. https://stackoverflow.com/questions/14291436/copy-to-clipboard-by-notification-action#14294660
  2. https://developer.android.com/develop/ui/views/touch-and-input/copy-paste#Copying
  3. https://developer.android.com/develop/ui/views/notifications/build-notification#Actions

Checklist

  • You have a descriptive commit message with a short title (first line, max 50 chars).
  • You have commented your code, particularly in hard-to-understand areas
  • You have performed a self-review of your own code
  • UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
  • UI Changes: You have tested your change using the Google Accessibility Scanner

@LUwUcifer
Copy link
Copy Markdown
Contributor Author

Second commit may not be fully optimised, requesting review

@LUwUcifer LUwUcifer force-pushed the syncmediaworker-notif branch from b4b1ea1 to aacfdcb Compare May 7, 2026 14:39
@BrayanDSO
Copy link
Copy Markdown
Member

How Has This Been Tested?

Tested on local device

Describe that appropriately and/or send a video or screenshot

@BrayanDSO BrayanDSO added Needs Author Reply Waiting for a reply from the original author and removed Needs Review labels May 8, 2026
@LUwUcifer
Copy link
Copy Markdown
Contributor Author

IMG_20260508_164622.jpg

@LUwUcifer
Copy link
Copy Markdown
Contributor Author

LUwUcifer commented May 8, 2026

bigText allows for display of the entire error message, copy to clipboard - copies to clipboard

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

Labels

Needs Author Reply Waiting for a reply from the original author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SyncMediaWorker fail throwable notification cannot be expanded or opened

2 participants