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

optimize mobile experience #356

Closed
wants to merge 2 commits into from
Closed

optimize mobile experience #356

wants to merge 2 commits into from

Conversation

cyhhao
Copy link
Contributor

@cyhhao cyhhao commented Apr 2, 2023

优化移动端体验,有以下几点修改:

  • 移动端 chat list 点击一下即可进入对话界面。之前是要点击两下,第一下是 focus,第二下才是进入,很容易令人困惑。
  • 移动端 chat list 里每个 item 直接显示x删除按钮,不需要 focus 才显示。也去掉了底部的X按钮。
  • fix 输入文字时,右边的聊天气泡也会同步显示文字,改为了显示LoadingIcon。(不知道是否是有意设计还是bug,但真的很容易困惑,给几个朋友用都第一眼以为消息发出去了)
  • fix 会话界面滚动条 bug。之前在移动端下,输入键盘弹出然后输入内容,整个页面的滚动会错误地把输入框也滚没了,很恼人。改为只滚动 chat-body 这部分 DOM 的滚动条。
  • 移动端输入框,高度改小,发送按钮的文字去掉。降低输入框面积的占比,
  • 移动端输入框,在发送后自动失焦,把键盘收起来,尽可能多地呈现GPT的回复。(因为在这个ChatGPT场景下,用户不会连续发送消息,都是发一句等着回复)

以上遵循的体验原则主要是:

  • 移动端应该尽量简洁,交互路径短一点,能点一下的不点两下。
  • 不像pc端用户可以多线程操作,移动端屏幕小,尽量让用户单线程操作:打字的时候输入框和输入法占大头,看消息的时候输入法就应该自动收起,以回复消息显示最大化。

望采纳~

@vercel
Copy link

vercel bot commented Apr 2, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @Yidadaa on Vercel.

@Yidadaa first needs to authorize it.

@Yidadaa
Copy link
Collaborator

Yidadaa commented Apr 2, 2023

你说的应该是 ios 上的体验?

  1. 安卓上不存在该问题;
  2. 这是出于美观考虑,删除本来就不是高频操作,有这么个入口就行了,不然还会有人问为什么不做成长按删除的,哪这么多闲工夫去打磨这种边角操作;
  3. 有意设计,输入框支持 markdown,气泡内是 markdown 预览,不过的确需要增强一下“正在输入”提示,但是我寻思这个功能用两次不就知道是怎么个回事了吗?
  4. 也是 safari 的 bug,反正 safari 上自动滚动就是一堆 bug,谁爱修谁修吧;
  5. 输入框设计正在改版;
  6. 现在自动滚动逻辑与输入框焦点绑定,自动失焦会无法自动滚动,再说一遍 safari 就是依托 **,新时代 IE。

这个项目的移动端设计本来就是附带的功能,原则是“能用就行”,作者精力有限,很难面面俱到,还有其他更重要的功能等着开发呢,又不是不能用。

@cyhhao
Copy link
Contributor Author

cyhhao commented Apr 2, 2023

是的,是 iOS 上的体验。Safari 确实很多bug很烂😂,但它规定了各种 app 只能用它的内核,而且大多数人就是直接用 Safari 或者微信打开的。。。

  1. 个人感觉小x常显示对美观影响不大,也不会误触。
  2. 确实。不过个人觉得毕竟写md发给GPT的场景还比较小众吧,写简单的md语法不预览也没事,复杂的像表格这种在输入框写也够累的。

你看如果觉得可以就合一下PR,我都自测过没问题。或者挑你觉得可以的代码加进去也行。
如果不认同也没关系,我自己 fork 一版就是了。

我的初衷是想给父母和身边墙内小白朋友用,所以把自认为的小问题改了。仅供参考~
没有要求作者的意思哈,你做的已经很棒了👍

Yidadaa added a commit that referenced this pull request Apr 2, 2023
@cyhhao cyhhao closed this Apr 3, 2023
lileihappy123 pushed a commit to lileihappy123/ChatGPT-Next-Web that referenced this pull request Apr 22, 2023
yxl pushed a commit to yxl/ChatGPT-Next-Web that referenced this pull request May 4, 2023
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