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

shows above keyboard in dialog box #26

Closed
ghost opened this issue May 2, 2016 · 8 comments · Fixed by #97
Closed

shows above keyboard in dialog box #26

ghost opened this issue May 2, 2016 · 8 comments · Fixed by #97

Comments

@ghost
Copy link

ghost commented May 2, 2016

I have a dialog box in which user can enter text and emoji.
But when i open emoji panel it shows above the keyboard instead of top of the keyboard.
I have given root view of the dialog box custom xml root view

@vanniktech
Copy link
Owner

Hey there which version are you using and also can you provide a screenshot of the issue?

@ghost
Copy link
Author

ghost commented May 2, 2016

screenshot_2016-05-02-17-22-14
_20160502_173530

0.1

@vanniktech
Copy link
Owner

Yup that's true. I just verified it and there's no support for dialogs. Since I'm not using this library with dialogs and right now I don't have any particular need for that, I won't fix this for now though I'd be open for PRs.

@ghost
Copy link
Author

ghost commented Jul 30, 2016

I successfully resolved this issue by changing the value of y axis.

private void showAtBottom() { popupWindow.showAtLocation(rootView, Gravity.BOTTOM, 0, -700); }

@vanniktech
Copy link
Owner

Where is that -700 coming from? It might fit your device but not different devices.

@ghost
Copy link
Author

ghost commented Jul 30, 2016

Yes, for my device, But we can calculate the distance between bottom of dialog to the bottom of mobile screen.

@vanniktech
Copy link
Owner

That number could also be a parameter of the builder so when you construct your EmojiPopUp you can pass the value you want and then as the consumer you'd need to calculate it. Default value would still be 0 though.

@ghost
Copy link
Author

ghost commented Aug 7, 2016

Yeah it will be good.
Kindly add this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant