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

fix: adjust network login and detail display #684

Merged
merged 1 commit into from
Feb 1, 2025

Conversation

SunnyCloudYang
Copy link
Contributor

What is the purpose of this PR?

  • Fixing a bug

Please describe your changes.

  • Network Login:
    • 上移了验证码区域,避免键盘遮挡
    • 在已经提交验证码且未返回结果时禁止输入
    • 提交验证码后显示refresh indicator,并增加下拉刷新
    • 微调验证码区域样式和说明文字
    • 格式化代码(是不小心的)
  • Network Details:
    • 改正了import路径
    • 修复了验证成功时页面不刷新的问题
    • 减小padding,尽量避免空间不足导致换行
    • 改正了typo
    • 格式化代码(是不小心的)

@SunnyCloudYang
Copy link
Contributor Author

主要修复的问题如下图所示:
detail
verify
但是忘记截改完的效果图了😿

@SunnyCloudYang SunnyCloudYang marked this pull request as ready for review January 31, 2025 15:43
@84634E1A607A
Copy link
Contributor

这个 KeyboardAvoidingView 毁灭吧👿

@SunnyCloudYang
Copy link
Contributor Author

这个 KeyboardAvoidingView 毁灭吧👿

但是神奇的是,它这次work了,虽然最后为了加refresh indicator我把它放在了上面,几乎不会有触发的场景,但是它真的work了(虽然但是,在线设备的kbdav依然是那个样子,绷)

@84634E1A607A
Copy link
Contributor

但是神奇的是,它这次work了,虽然最后为了加refresh indicator我把它放在了上面,几乎不会有触发的场景,但是它真的work了(虽然但是,在线设备的kbdav依然是那个样子,绷)

AbdAvView 的问题是在 Android 上面由于某些问题在 Keypad 显示的时候,它向上规避的距离不够长。所以在线设备那个由于输入在最底下,这个问题就很明显。其他的地方也是有问题的,只是有时候看不太出来。

@84634E1A607A
Copy link
Contributor

此外,个人建议不要把自己 Repo 的 master 分支请求合并到主线的 master 分支,还是单独开一个分支比较好。把 master 合并到 master 的时候我们本地不太方便测试和 review。(因为本地的 master track upstream 的 master,如果再来一个 master,要么重命名,要么改 remote tracking branch)

@84634E1A607A 84634E1A607A merged commit dc598bb into thu-info-community:master Feb 1, 2025
2 checks passed
@SunnyCloudYang
Copy link
Contributor Author

但是神奇的是,它这次work了,虽然最后为了加refresh indicator我把它放在了上面,几乎不会有触发的场景,但是它真的work了(虽然但是,在线设备的kbdav依然是那个样子,绷)

AbdAvView 的问题是在 Android 上面由于某些问题在 Keypad 显示的时候,它向上规避的距离不够长。所以在线设备那个由于输入在最底下,这个问题就很明显。其他的地方也是有问题的,只是有时候看不太出来。

原来如此,不过在我的设备上它向上的距离刚好但是keypad收起的时候输入框就回不到最下面了🤣

@84634E1A607A
Copy link
Contributor

原来如此,不过在我的设备上它向上的距离刚好但是keypad收起的时候输入框就回不到最下面了🤣

向上的距离刚好是因为调了 keyboardVerticalOffset={headerHeight}. 然后这玩意有个副作用就是上去之后回不去了 ()

我去 RN 的官方 Demo 上试了半天也有这个问题。不知道 RN 能不能修好 (小声)。 然后其他的解决方案疑似也不起作用

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.

2 participants