When on a post comments I used to be able to swipe to the right to go back to the community but now it bugs out and opens the sidebar so I have to use the Android back to return to the community

  • idunnololzM
    link
    fedilink
    arrow-up
    2
    ·
    3 months ago

    Yeah I will probably need to disable the main screen on transition. Annoying because there isn’t a built in way to do this. You sort of just have to do it manually T_T

      • idunnololzM
        link
        fedilink
        arrow-up
        1
        ·
        3 months ago

        It’s technically frowned upon for accessibility reasons (eg. accessibility users can technically select elements under the transparent layer). But yes I’m too lazy to do things properly so I will go this route. Also this is such an edge case.

        Adding this transparent layer doesn’t even fix the issue if you double tap really fast because events are handled asynchronously, converted to a serial queue and then set to the UI.

        This means that a double tap can go off even if you show a transparent layer because by the time the UI is notified of the first tap, both taps are already in the touch event queue. So even if you show the transparent layer now, the 2nd tap will go off for sure. it will not block the 2nd tap. The 100% correct way is to have a flag to handle this. However this is extremely tricky because you need to check this flag everywhere and also disable each and every possible interaction with every single view manually.

        As a middle ground, I fixed the issue where tapping two posts will show a black screen so at least in the next version, users who tap too fast will not end up in a weird state.

      • idunnololzM
        link
        fedilink
        arrow-up
        1
        ·
        3 months ago

        Ok after using the app with this change for a day ai have to say that I HATE IT. It makes thing so much slower. I have to wait for the stupid close animation to finish before i drag. I’m removing this change immediately lmao.

        • Redjard@lemmy.dbzer0.com
          link
          fedilink
          arrow-up
          2
          ·
          3 months ago

          Did you put it on top of everything? I meant between the opening post and the posts list. Such that if it had 50% opacity the post would look identical and the post list would be dimmed, and you wouldn’t even have to remove it when the post is done opening (tho ofc you should for when it is swiped away).

          • idunnololzM
            link
            fedilink
            arrow-up
            2
            ·
            3 months ago

            no its only on top of the post list. However the way i browse is I tap a post, quickly read it then swipe and then while the post is closing I am already trying to scroll the post list. This wont work since interactions with the post list is blocked until the post sheet is closed fully.

            • Redjard@lemmy.dbzer0.com
              link
              fedilink
              arrow-up
              2
              ·
              3 months ago

              Oh yeah that makes sense. And if you fix all the issues when closing the post, opening it wouldn’t be very different and should also mostly be fixed.

              Then I’ll report back if I ever see other issues related to it or have the accidental postlist scroll happen organically.