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

[Bug]: mobile - visual - gray box above input box #1782

Closed
1 task done
MatissJanis opened this issue Oct 10, 2023 · 2 comments · Fixed by #1929
Closed
1 task done

[Bug]: mobile - visual - gray box above input box #1782

MatissJanis opened this issue Oct 10, 2023 · 2 comments · Fixed by #1929
Labels
bug Something isn't working good first issue Good for newcomers responsive Responsive/Mobile version

Comments

@MatissJanis
Copy link
Member

Verified issue does not already exist?

  • I have searched and found no existing issue

What happened?

Screenshot 2023-10-10 at 22 20 15

There's a weird gray box above the input field for some reason. I think this is new in v23.10.0

What error did you receive?

No response

Where are you hosting Actual?

None

What browsers are you seeing the problem on?

No response

Operating System

None

@MatissJanis MatissJanis added bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed responsive Responsive/Mobile version labels Oct 10, 2023
@miqh
Copy link
Contributor

miqh commented Oct 12, 2023

@MatissJanis, I briefly attempted reproducing this and I suspect what you're seeing there might be the result of an overflow scrollbar.

Oddly, I was only able to see it on Firefox under responsive conditions and not Chromium.

repro

I traced it to the following line below and found that if overflowX: 'hidden' is added then the scrollbars stop appearing on interaction for me.

<View style={{ overflowY: 'auto' }}>{input}</View>

There's some suspect animation code that's commented out, but I don't know enough to work out why the flicker happens for me.

Anyway, I'd be happy to dig further or provide that above suggested change based on what you think.

@joel-jeremy
Copy link
Contributor

@miqh Thank you for taking the time to investigate. Please feel free to create a PR with the fix. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers responsive Responsive/Mobile version
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants