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

Blank strip between compose box and keyboard #3370

Closed
gnprice opened this issue Feb 27, 2019 · 3 comments · Fixed by #5564
Closed

Blank strip between compose box and keyboard #3370

gnprice opened this issue Feb 27, 2019 · 3 comments · Fixed by #5564
Labels
a-compose/send Compose box, autocomplete, camera/upload, outbox, sending a-iOS a-layout

Comments

@gnprice
Copy link
Member

gnprice commented Feb 27, 2019

In this video taken by @armaanahluwalia to demonstrate #3369, there's another symptom present:

A blank white band between the compose box and the keyboard.

The video is from iOS. (@armaanahluwalia , would you confirm what version of the app it's from?) I don't reproduce this issue on Android 9 on 23.1.110.

Possibly-relevant recent history is in #3273 and its fix, c3b4560.

@gnprice gnprice added a-iOS a-layout a-compose/send Compose box, autocomplete, camera/upload, outbox, sending labels Feb 27, 2019
@jackrzhang
Copy link
Contributor

jackrzhang commented Feb 28, 2019

I don't see this on an iPhone 7 with iOS 12.1, though it does reproduce on a simulated iPhone XR - I think that this bug is related to the fact that the shape of the touch screen on the newer iPhone models (as of the writing of this comment) is no longer a rectangle.

The application could be utilizing some values for the height of the screen (and by relation, the height of the keyboard) that are no longer represented the same way across the iPhone X's and earlier iPhone models.

@gnprice
Copy link
Member Author

gnprice commented Jun 4, 2019

Thanks @jackrzhang for those repro details! That's helpful -- I agree it looks like evidence that this happens just on iPhones with non-rectangular screens.

I think this is likely an indirect symptom of #3066. We have some code that attempts to handle the notch / handle the display cutout / limit to the safe area, and I suspect this is caused by that code going awry.

@gnprice
Copy link
Member Author

gnprice commented Oct 17, 2019

As of v26.9.132, this is still present though the symptom is a little subtler: the extra space is (in light mode) now the same gray color as the compose area, rather than white.

So basically it looks like a bad design choice rather than a bug. (Though in reality it's still a bug.)

chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Jul 16, 2021
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Jul 29, 2021
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Aug 13, 2021
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Nov 18, 2022
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Nov 19, 2022
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Nov 21, 2022
chrisbobbe added a commit to chrisbobbe/zulip-mobile that referenced this issue Nov 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-compose/send Compose box, autocomplete, camera/upload, outbox, sending a-iOS a-layout
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants