Skip to content

[BR-2114]: iOS generation + add list-view thumbnails - #524

Open
terrerox wants to merge 4 commits into
fix/android-upload-encoded-uri-thumbnailfrom
feature/drive-list-mode-thumbnails
Open

[BR-2114]: iOS generation + add list-view thumbnails#524
terrerox wants to merge 4 commits into
fix/android-upload-encoded-uri-thumbnailfrom
feature/drive-list-mode-thumbnails

Conversation

@terrerox

@terrerox terrerox commented Jul 8, 2026

Copy link
Copy Markdown
Contributor
  • fix(ios): toFileUri now emits absolute file:///… URIs. A relative path
    became file://var/… (parsed as host), so iOS thumbnail generation failed with
    CreateThumbnail Code=1 and uploads showed no preview.
  • feat: image thumbnails now render in Drive list view (previously grid
    only), via a shared useDownloadedThumbnail hook used by both grid and list.
    Falls back to the type icon for non-images/folders/download errors.

@terrerox terrerox changed the title Feature/drive list mode thumbnails [BR-2114]: iOS generation + add list-view thumbnails Jul 8, 2026
@terrerox
terrerox requested a review from CandelR July 8, 2026 04:43
@terrerox terrerox self-assigned this Jul 8, 2026
@terrerox
terrerox force-pushed the fix/android-upload-encoded-uri-thumbnail branch from b985e99 to a64d666 Compare July 15, 2026 20:58
@terrerox
terrerox requested a review from TamaraFinogina as a code owner July 15, 2026 20:58
terrerox added 3 commits July 15, 2026 22:39
…bnail paths

Uploads of files whose names contain spaces or non-ASCII characters failed
with ENOENT because percent-encoded cache URIs were passed unmodified to
RNFS.copyFile and to the native thumbnail generator. Decode the source URI
before copy (copyFileFromEncodedUri) and the native-returned thumbnail URI
(fromFileUri); drop the stale manual %20 encoding. Device-verified on Android.
@terrerox
terrerox force-pushed the feature/drive-list-mode-thumbnails branch from 00cec16 to 652e4e0 Compare July 16, 2026 02:46
@terrerox
terrerox removed the request for review from TamaraFinogina July 16, 2026 02:46
@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant