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

Elements overlap each other on Android (profile selection page; chat view) #16693

Closed
qoqobolo opened this issue Jul 14, 2023 · 16 comments · Fixed by #16832
Closed

Elements overlap each other on Android (profile selection page; chat view) #16693

qoqobolo opened this issue Jul 14, 2023 · 16 comments · Fixed by #16832

Comments

@qoqobolo
Copy link
Contributor

qoqobolo commented Jul 14, 2023

Bug Report

Problem

Screenshot 2023-07-14 at 17 16 25 Screenshot 2023-07-14 at 17 16 10

Reproduction

We were able to reproduce these scenarios:

1.

  1. Create several accounts
  2. Add at least one community to each of them
  3. Log out
  4. Open the profile selection page

At some point the cards start overlaying each other

2.

  1. Create a profile, create a community
  2. Log out
  3. Create another profile, join a community

3.

Preconditions: at least 1 profile is already created

  1. Create one more fresh profile - User A
  2. UserA: send CR to UserB
  3. UserB: accept CR
  4. UserA: log out right after User B accepts CR
  5. Open the profile selection screen by User A

Additional Information

  • Status version: RC
  • Operating System: Android
@Parveshdhull
Copy link
Member

Parveshdhull commented Jul 14, 2023

Hi @qoqobolo, Are you able to reproduce this issue on multiple Android devices?

@qoqobolo
Copy link
Contributor Author

Hi @qoqobolo, Are you able to reproduce this issue on multiple Android devices?

Yes, we reproduced this on at least Pixel, Huawei and Samsung

@Parveshdhull
Copy link
Member

Hi @qoqobolo, Are you able to reproduce this issue on multiple Android devices?

Yes, we reproduced this on at least Pixel, Huawei and Samsung

Thank you for confirming

@Parveshdhull
Copy link
Member

Parveshdhull commented Jul 17, 2023

(defn view
  []
  (let [show-profiles?    (reagent/atom false)
        set-show-profiles #(reset! show-profiles? true)
        set-hide-profiles #(reset! show-profiles? false)]
    (fn []
      [:<>
       [background/view true]
       (if @show-profiles?
         [profiles-section {:set-hide-profiles set-hide-profiles}]
         [login-section {:set-show-profiles set-show-profiles}])])))

Profile screen is controlled by a regent atom and only one view should be visible at a time. Don't know why both are visible. Maybe you have some idea @flexsurfer

Both profiles and chat screens force updates view at runtime, and removes old view and renders new one
First using reframe subscription and second using reagent atom. But for some reason old view is not getting removed and both views getting rendered simultaneously 🤔

@Parveshdhull
Copy link
Member

Parveshdhull commented Jul 17, 2023

@qoqobolo, I tried creating several accounts with communities and navigating to profiles, but I am not able to reproduce the issue.
Please can you confirm the frequency of occurrence, if it is quite rare and non-blocker, Please can we move this to the next release?

@Parveshdhull
Copy link
Member

Parveshdhull commented Jul 17, 2023

Don't know why they are visible both

In #16433 we implemented layout animation, so if a toast is shown it enables them. Just a guess, but that might be breaking layout while animating it.

Because once enabled, it starts to affect other places too: #14752

@Parveshdhull
Copy link
Member

Parveshdhull commented Jul 17, 2023

Should I create a pr for disabling layout animations for QA, to check if it fixes the issue?

@qoqobolo
Copy link
Contributor Author

@qoqobolo, I tried creating several accounts with communities and navigating to profiles, but I am not able to reproduce the issue.
Please can you confirm the frequency of occurrence, if it is quite rare and non-blocker, Please can we move this to the next release?

Thank you very much for investigating the issue!
It's quite severe and also occurs in other places (screens) of the app, but it's really not that easy to reproduce, and since we don't have a simple and quick solution, let's move it to 1.25 👍

Should I create a pr for disabling layout animations for QA, to check if it fixes the issue?

Yes, please do this so we can test the hypothesis after testing the release.

@Parveshdhull
Copy link
Member

Yes, please do this so we can test the hypothesis after testing the release.

Sounds good, thank you very much

@Parveshdhull
Copy link
Member

Parveshdhull commented Jul 20, 2023

Hi @qoqobolo, I created a pr with disabled layout animations. Whenever you have some spare time, please give it a try and let me know if the issue is still reproducible for you. Thank you very much for looking into this issue.

#16729

@qoqobolo
Copy link
Contributor Author

thank you @Parveshdhull, we will check!
Added the request-manual-qa label just for better visibility.

@qoqobolo
Copy link
Contributor Author

Reopening the issue since it's reproducible again.
Device: Pixel 7a

Steps:

  1. Create a profile
  2. Log out from the profile screen
  3. Quickly tap the Profiles button (at the top right)

Sometimes you need to repeat this a couple of times.

video_2023-08-31_10-37-15.mp4

@qoqobolo
Copy link
Contributor Author

qoqobolo commented Sep 1, 2023

Unable to reproduce it again.

@qoqobolo qoqobolo closed this as completed Sep 1, 2023
@pavloburykh
Copy link
Contributor

pavloburykh commented Sep 21, 2023

@Parveshdhull unfortunately still reproducible, so reopening the issue. Steps are yet unknown.

Reproduced on Android 12, Samsung Galaxy A52
Build nightly (21 Sep, 2023)

Status-debug-logs (63).zip

photo_2023-09-21 13 19 40

telegram-cloud-document-2-5215358463747569775.mp4

@Parveshdhull
Copy link
Member

It looks like issue is most probably with reanimated library version and will be fixed in reanimated v3. More info #16832

@churik
Copy link
Member

churik commented Oct 19, 2023

closing this for now, reopen if reproducible

@churik churik closed this as completed Oct 19, 2023
@ilmotta ilmotta removed the bug label Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants