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

When changing the message to be replied from image message to text/audio message the old image belonging to previously selected message should go away #4343

Closed
ryg-git opened this issue Oct 26, 2021 · 3 comments
Labels
A-Timeline O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Tolerable Low/no impact on users T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@ryg-git
Copy link
Contributor

ryg-git commented Oct 26, 2021

Steps to reproduce

  1. Open a room screen or DM screen.

  2. Select a image message to reply either by swiping or from long press menu
    IMG_20211026_195908

  3. Then while the image message is selected for the reply select another message (text or audio)

  4. Now you will see that only text is replaced i.e image name is changed to selected message or file but the previously selected image doesn't go away.
    IMG_20211026_200610

  5. Also if you select text message first and then you change message to be replied to image the selected image is not displayed only image file name is displayed.
    IMG_20211026_200651
    IMG_20211026_200712

Intended result and actual result

What did you expect?

When changing the message to be replied from image to text the old image should go away and only text belonging to newly selected message should be displayed,
Also if you select text message first to reply and then select image the image should also be displayed not only file name.

What happened instead?

If you select image first to reply and then change the message to be replied to text the image doesn't go away.
If you select text message first to reply and then select image the image is not displayed only file name is displayed.

Your phone model

Xiaomi redmi note 8 pro

Operating system version

Android 11

Application version and app store

Element android app version 1.3.3

Homeserver

matrix.org

Will you send logs?

No

@ryg-git ryg-git added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Oct 26, 2021
@ryg-git
Copy link
Contributor Author

ryg-git commented Oct 26, 2021

If it is okay I would like to work on this issue

@ryg-git ryg-git changed the title Image doesn't disappear when changing the message to be replied from image to text/audio When changing the message to be replied from image message to text/audio message the old image belonging to previously selected message should go away Oct 26, 2021
@ouchadam
Copy link
Contributor

@ryg-git thanks for raising this and all the reproduction steps! A fix would be very welcome:+1:

I was also able to reproduce the same scenarios mentioned

IMAGE - TEXT TEXT - IMAGE
reply-image reply-text-image

@ouchadam ouchadam added A-Timeline O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Tolerable Low/no impact on users labels Oct 26, 2021
@ryg-git ryg-git mentioned this issue Nov 17, 2021
9 tasks
@bmarty
Copy link
Member

bmarty commented Nov 19, 2021

Fixed in #4490 and Element 1.3.9

@bmarty bmarty closed this as completed Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Timeline O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Tolerable Low/no impact on users T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Development

No branches or pull requests

3 participants