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

朋友,看一下这样是否更好 #235

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

THEONE10211024
Copy link

我们在测试的时候发现使用之前的方式设置模式切换在魅族部分手机WebView全屏的情况下UI显示不正确,如此修改后测试无问题!

@laobie
Copy link
Owner

laobie commented Aug 27, 2018

那个 FULL_SCREE 的 flag 在某些场景下是需要的,我后面提供一个新的方法来处理这个,至于 light mode 和 dark mode 的定义,比较倾向 Google 官方的定义,即 白底黑字为 light mode ,黑底白字为 dark mode

@THEONE10211024
Copy link
Author

"至于 light mode 和 dark mode 的定义,比较倾向 Google 官方的定义,即 白底黑字为 light mode ,黑底白字为 dark mode",这个我改的比较冲忙,没有看官方的定义,就依你的处理就可以。但是如果用FULL_SCREE来设置在我们的魅族手机上确实是有问题

@laobie
Copy link
Owner

laobie commented Aug 27, 2018

FULL_SCREEN 的问题会改一下,这个确实存在问题,会多添加一个方法来处理

@THEONE10211024
Copy link
Author

好的,朋友!

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 this pull request may close these issues.

None yet

2 participants