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

Bugs (Gravity + Drag inversion) #57

Open
sidetraxaudio opened this issue Apr 4, 2023 · 0 comments · May be fixed by #63
Open

Bugs (Gravity + Drag inversion) #57

sidetraxaudio opened this issue Apr 4, 2023 · 0 comments · May be fixed by #63

Comments

@sidetraxaudio
Copy link

sidetraxaudio commented Apr 4, 2023

Good work. Thanks for your contribution.

Alignment enum bugs exist where drag direction is inverted on horizontal, vertical or both axis depending on alignment:

TopLeft=OK
CenterLeft=OK
BottomLeft = Vertical drag axis inverted.
TopCenter = OK
Center = OK
BottomCenter = Vert drag axis inverted
TopRight = Horizontal drag inverted
CenterRight = Horizontal drag inverted
BottomRight = Both drag axis inverted.

I'm not a Java programmer but your ACTION_MOVE switch statement seems to look fine, and obviously works fine gravity set to center etc. It is a weird gravity issue, possibly a bug between window manager and the flutterview? Out of my depth sorry.

To rectify in the short term in your onTouch method if you can't find the cause :
int xx = params.x + (int) dx; (subtract dx instead on problematic right gravity)
int yy = params.y + (int) dy; (subtract dx instead on problematic bottom gravity)

Also there is a screen dims / vertical centering issue. If you pass a full screen colored container from flutter you might see the screen centering too far north, off the screen and padded at the bottom. NavBar size calculation issue??

@sidetraxaudio sidetraxaudio changed the title Bugs Bugs (Gravity + Drag inversion) Apr 4, 2023
@radomir9720 radomir9720 linked a pull request May 18, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant