BetterUntis icon indicating copy to clipboard operation
BetterUntis copied to clipboard

[Bug Report] #370 doesn't work in the latest beta version (4.0.2)

Open Lauritz-Tieste opened this issue 1 year ago • 10 comments

Unfortunately the error still occurs even though it was fixed by the PR.

Android Version

Android 13

BetterUntis Version

4.0.2 (28227720)

Installation Source

GitHub Release

Lauritz-Tieste avatar Sep 02 '23 21:09 Lauritz-Tieste

Did you test the changes when you made the PR? I myself haven't been able to replicate the issue at all, so I can't really help...

SapuSeven avatar Sep 02 '23 21:09 SapuSeven

Yes, I tested it and it worked perfectly

Lauritz-Tieste avatar Sep 02 '23 21:09 Lauritz-Tieste

That's weird. Maybe we need an entirely different approach to control when the drawer can be opened.

SapuSeven avatar Sep 02 '23 21:09 SapuSeven

I'll take a look at it

Lauritz-Tieste avatar Sep 02 '23 21:09 Lauritz-Tieste

The changes are not included in the source code on the dev branch!

Lauritz-Tieste avatar Sep 02 '23 21:09 Lauritz-Tieste

Can't confirm. I looked at the source code attached to the 4.0.2 release, as well as the current dev branch. They both include the PR changes.

SapuSeven avatar Sep 02 '23 22:09 SapuSeven

The changes of 28968905a0922921316149c42be4c3ede2428cdb caused the error again

Lauritz-Tieste avatar Sep 02 '23 22:09 Lauritz-Tieste

I myself haven't been able to replicate the issue at all, so I can't really help...

To reproduce it: You have to quickly swipe in one direction on the timetable several times. Then the drawer should open

Lauritz-Tieste avatar Sep 02 '23 22:09 Lauritz-Tieste

Given my comment in #408 - is it possible that this error happens when you (accidentally) start the swipe too far to the left edge?

SapuSeven avatar Sep 02 '23 22:09 SapuSeven

Detailed description for #370, #407 and #408

Steps to reproduce the error:

  1. Open BetterUntis
  2. QUICKLY swipe through the timetable (right/left), as if you wanted to quickly go 8 weeks into the future.

After a while, the NavigationDrawer accidentally opens on all of my devices.

For reproducing the error, it might be useful to test this on a real phone, rather than an emulator.

  • Tablet (Samsung S6 Lite, BetterUntis Version 4.0.2, Android 13)
  • Smartphone (Samsung A32, BetterUntis Version 4.0.2, Android 13)

Given my comment in #408 - is it possible that this error happens when you (accidentally) start the swipe too far to the left edge?

@SapuSeven No, this error also occurs if you only swipe in the middle of the screen and leave 2cm between swiping and the edge of the screen

Lauritz-Tieste avatar Nov 07 '23 16:11 Lauritz-Tieste

I was no longer able to cause the error in the latest version (4.1.0-beta-01 28334728).

Lauritz-Tieste avatar Apr 08 '24 12:04 Lauritz-Tieste