8000 代码问题 · Issue #314 · razerdp/BasePopup · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

代码问题 #314

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.

Sign up for GitHub

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

Jump to bottom
Closed
angcyo opened this issue Jul 15, 2020 · 2 comments
Closed

代码问题 #314

angcyo opened this issue Jul 15, 2020 · 2 comments
Labels
bug need more detail 不符合issue规范，低优先级 pending release 待发布正式版

Comments

@angcyo
Copy link
angcyo commented Jul 15, 2020

razerdp.basepopup.BasePopupHelper#applyToPopupWindow

源码如下:

 private void applyToPopupWindow() {
     if (mPopupWindow == null || mPopupWindow.mPopupWindowProxy == null) return;
     mPopupWindow.mPopupWindowProxy.setSoftInputMode(isAutoShowInputMethod() ? WindowManager.LayoutParams.SOFT_INPUT_ADJUST_RESIZE : WindowManager.LayoutParams.SOFT_INPUT_STATE_UNCHANGED
     mPopupWindow.mPopupWindowProxy.setSoftInputMode(mSoftInputMode);
     mPopupWindow.mPopupWindowProxy.setAnimationStyle(animationStyleRes);
 }

这里调用了2次setSoftInputMode, 是否有问题?
@razerdp razerdp added the need more detail 不符合issue规范，低优先级 label Jul 15, 2020
@razerdp
Copy link
Owner
razerdp commented Jul 15, 2020

确实有问题。。。我看看修改记录

image

5月7号时就已经遗留了。。。。蛋疼- -，我这边的锅，感谢指出~

@razerdp
Copy link
Owner
razerdp commented Jul 15, 2020 •
edited
Loading

相关提交：
2ee3227#diff-9cc906d83de4f0db339b77d70c78c7dfL715

当前修复未发布Candy版本，待其他功能完成后一同发布。

razerdp added a commit that referenced this issue Jun 29, 2021
@razerdp
fixed #314
f14daba 
提交issue 277测试用例
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug need more detail 不符合issue规范，低优先级 pending release 待发布正式版
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@angcyo @razerdp
0