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

Threads: Improve fallback for clients without Threads #4972

Closed
daniellekirkwood opened this issue Jan 18, 2022 · 7 comments
Closed

Threads: Improve fallback for clients without Threads #4972

daniellekirkwood opened this issue Jan 18, 2022 · 7 comments
Assignees
Labels
A-Threads An issue for the Threads project Z-ThreadsP0 Issue to be released with the Threads MVP

Comments

@daniellekirkwood
Copy link
Contributor

daniellekirkwood commented Jan 18, 2022

Your use case

Instead of including the Root Message in the Reply view for clients without Threads, show the message previous to the response.

@daniellekirkwood to add more info

We know and accept that some clients will not update their versions and therefore will not have this new and improved fallback will not be available to everyone. Those users who do not update their version from the app/play store will still have the previous fallback of seeing the Root message.

@daniellekirkwood daniellekirkwood added A-Threads An issue for the Threads project Z-ThreadsP0 Issue to be released with the Threads MVP labels Jan 18, 2022
@daniellekirkwood
Copy link
Contributor Author

I think we also need content for what to show when the message has not yet been fetched, as we're currently showing "null". This also is the case on iOS: element-hq/element-ios#5394

@janogarcia

@ariskotsomitopoulos
Copy link
Contributor

This case would be rare while now we are using as fallback the previous event. However, I implemented that. You can suggest a better text resource:
Screenshot_20220124-140236_Element dbg

@daniellekirkwood
Copy link
Contributor Author

This is great! Let's go with that - I will update the iOS issue. Thanks, @ariskotsomitopoulos

@janogarcia
Copy link

@ariskotsomitopoulos Can you please confirm if we're not showing the "In reply to @name" part before the "Replied within a thread" placeholder message? Unsure if we're not rendering it at all when handling this edge case or it was just cropped in the screenshot you shared.

@daniellekirkwood If we're not rendering the "In reply to @name" part at all, should we consider following a similar copy for the placeholder message? In reply to a thread instead of Replied within a thread.

Same for iOS.

@ariskotsomitopoulos
Copy link
Contributor

@janogarcia When we don't have the parent relation or the event local in the device we removed completely the In reply to and it is like in the image above. In all the other cases that we have the event and the relation is like a normal reply. Is that clear?

@janogarcia
Copy link

@ariskotsomitopoulos Thanks for the clarification, Aris. 👍️ Then, I would suggest changing the copy to In reply to a thread, to keep it more consistent with similar messages, but let's give @daniellekirkwood the final word on that.

@daniellekirkwood
Copy link
Contributor Author

OK.

@janogarcia Can you update the necessary Figma and iOS issue here. Thanks

@daniellekirkwood daniellekirkwood removed their assignment Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Threads An issue for the Threads project Z-ThreadsP0 Issue to be released with the Threads MVP
Projects
None yet
Development

No branches or pull requests

3 participants