Skip to content

Fix sharing issues#792

Merged
microbit-matt-hillsdon merged 1 commit intoappsfrom
fix-share
Mar 17, 2026
Merged

Fix sharing issues#792
microbit-matt-hillsdon merged 1 commit intoappsfrom
fix-share

Conversation

@microbit-matt-hillsdon
Copy link

@microbit-matt-hillsdon microbit-matt-hillsdon commented Mar 16, 2026

Omit the text parameter from Share.share() on iOS where UIActivityViewController treats it as a second shareable item. Android still receives it as EXTRA_TEXT for email body pre-fill. Previously you got the intended file and a file called "text".

Dismiss the save progress dialog as soon as the hex is ready on all platforms, before entering platform-specific save/share logic. Previously the Download path on iOS left SaveStep.SaveProgress active while the share sheet was open. Nothing then dismissed it if you cancelled.

Omit the `text` parameter from Share.share() on iOS where
UIActivityViewController treats it as a second shareable item.
Android still receives it as EXTRA_TEXT for email body pre-fill.

Dismiss the save progress dialog as soon as the hex is ready on all
platforms, before entering platform-specific save/share logic. Previously
the Download path on iOS left SaveStep.SaveProgress active while the
share sheet was open. Nothing then dismissed it if you cancelled.
@github-actions
Copy link

Preview build will be at
https://review-createai.microbit.org/fix-share/

Copy link

@microbit-grace microbit-grace left a comment

Choose a reason for hiding this comment

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

LGTM, tested downloading hex on iOS via share and there's no text file. Also, dismissed share dialog and did not see the saving progress dialog.

@microbit-matt-hillsdon microbit-matt-hillsdon merged commit 5b4ae53 into apps Mar 17, 2026
4 checks passed
@microbit-matt-hillsdon microbit-matt-hillsdon deleted the fix-share branch March 17, 2026 11:08
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.

2 participants