Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Image messages cause iOS (maybe android) to become unusable #2526

Closed
islathehut opened this issue May 13, 2024 · 1 comment
Closed

Image messages cause iOS (maybe android) to become unusable #2526

islathehut opened this issue May 13, 2024 · 1 comment
Assignees
Labels
bug Something isn't working dependencies Pull requests that update a dependency file images and files iOS mobile react-native

Comments

@islathehut
Copy link
Collaborator

Found while testing 2.2.0 on iOS.

Reproducing

  1. Connect a mobile client to a desktop client
  2. Send an image from the desktop client
  3. Wait for the image to download on the mobile client
  4. See errors

Errors

Errors were of a couple variety but generally what I saw on the device were:

Exception thrown while executing UI block: -[FFFastImageView setOnClick:]: unrecognized selector sent to instance 0x109b462c0

__44-[RCTUIManager flushUIBlocksWithCompletion:]_block_invoke
    RCTUIManager.m:1205
__44-[RCTUIManager flushUIBlocksWithCompletion:]_block_invoke.208
__RCTExecuteOnMainQueue_block_invoke
_dispatch_call_block_and_release
_dispatch_client_callout
_dispatch_main_queue_drain
_dispatch_main_queue_callback_4CF
3A5F992A-D1CD-312E-BD2E-F7C66343A417
3A5F992A-D1CD-312E-BD2E-F7C66343A417
CFRunLoopRunSpecific
GSEventRunModal
7BF01CFC-23F1-326A-AFD8-AD967FFECE28
UIApplicationMain
main
7BE2B757-3B3D-3E91-8CB7-74F3887660C7
Animated node with tag 501 does not exist

-[RCTNativeAnimatedNodesManager addAnimatedEventToView:eventName:eventMapping:]
    RCTNativeAnimatedNodesManager.mm:349
__73-[RCTNativeAnimatedModule addAnimatedEventToView:eventName:eventMapping:]_block_invoke
__56-[RCTNativeAnimatedModule uiManagerWillPerformMounting:]_block_invoke.49
__44-[RCTUIManager flushUIBlocksWithCompletion:]_block_invoke
__44-[RCTUIManager flushUIBlocksWithCompletion:]_block_invoke.208
__RCTExecuteOnMainQueue_block_invoke
_dispatch_call_block_and_release
_dispatch_client_callout
_dispatch_main_queue_drain
_dispatch_main_queue_callback_4CF
3A5F992A-D1CD-312E-BD2E-F7C66343A417
3A5F992A-D1CD-312E-BD2E-F7C66343A417
CFRunLoopRunSpecific
GSEventRunModal
7BF01CFC-23F1-326A-AFD8-AD967FFECE28
UIApplicationMain
main
7BE2B757-3B3D-3E91-8CB7-74F3887660C7

In the logs I saw:

Error setting property 'onClick' of FastImageView with tag #1139: Exception thrown while executing UI block: -[FFFastImageView setOnClick:]: unrecognized selector sent to instance 0x107845f40
Error setting property 'onClick' of FastImageView with tag #1139: Exception thrown while executing UI block: 'parentNode' is a required parameter
Error setting property 'onClick' of FastImageView with tag #1139: Animated node with tag 227 does not exist

The errors seemed to change over time and would pop up every time I tried to go into a channel or click any UI element (e.g. the meatball menu).

Potential Solutions

  1. Figure out why FastImage isn't working for us (if it ever has)
  2. Remove FastImage and use React Native components directly

It's important to note that the main use of FastImage is for loading/caching images from the web and we are just pulling images from file URIs. Also development on this repo seems to have stalled a couple of years ago and we are on the latest version so we can't try updating.

@islathehut islathehut added bug Something isn't working dependencies Pull requests that update a dependency file images and files iOS mobile react-native labels May 13, 2024
@islathehut islathehut added this to Quiet May 13, 2024
@islathehut islathehut moved this to Backlog - Mobile in Quiet May 13, 2024
@islathehut islathehut self-assigned this May 14, 2024
@islathehut islathehut moved this from Backlog - Mobile to In progress in Quiet May 14, 2024
@islathehut islathehut moved this from In progress to Waiting for review in Quiet May 16, 2024
@kingalg
Copy link
Collaborator

kingalg commented May 21, 2024

[email protected] | iOS 376

Fixed.

(additional info - this was only an issue on ios, android was fine)

@kingalg kingalg closed this as completed May 21, 2024
@kingalg kingalg moved this from Waiting for review to Done in Quiet May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dependencies Pull requests that update a dependency file images and files iOS mobile react-native
Projects
Status: Done
Development

No branches or pull requests

2 participants