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

Improve mobile selection text menu #8829

Closed
3 tasks done
gdy106 opened this issue Jul 26, 2023 · 4 comments
Closed
3 tasks done

Improve mobile selection text menu #8829

gdy106 opened this issue Jul 26, 2023 · 4 comments
Assignees
Milestone

Comments

@gdy106
Copy link

gdy106 commented Jul 26, 2023

手机端选中文字后的快捷菜单会遮挡文字

Is there an existing issue for this?

  • I have searched the existing issues

Can the issue be reproduced with the default theme (daylight/midnight)?

  • I was able to reproduce the issue with the default theme

Could the issue be due to extensions?

  • I've ruled out the possibility that the extension is causing the problem.

Describe the problem

在手机端使用思源,可能由于扣除键盘后剩余屏幕区域较小,会出现选中文字后跳出的菜单遮挡选中的文字的情况。
建议这里选中文字还是常规的跳出横排复制粘贴剪切选项,添加一个更多选项,点击更多再跳出不同的复制选项。
这时候菜单也可以放到键盘区域而不是占用上方屏幕。

Expected result

复制粘贴复杂菜单不遮挡选中的文字。

Screenshot or screen recording presentation

Screenshot_2023-07-26-18-30-58-560_org b3log siyuan

Version environment

- Version: 2.9.7
- Operating System: 安卓
- Browser (if used):

Log file

More information

No response

@Vanessa219
Copy link
Member

Android 选中是横排,但是我们这个选项有点多,如果横排可能需要移除几个不常用的选项。

@gdy106
Copy link
Author

gdy106 commented Jul 26, 2023

这个简单。我做个一套横排用的效果图。

选中文字的菜单:
复制粘贴菜单

点击复制+后跳出复制为纯文本:
复制粘贴菜单-复制

点击粘贴+后跳出粘贴选项:
复制粘贴菜单-粘贴

Android 选中是横排,但是我们这个选项有点多,如果横排可能需要移除几个不常用的选项。

@Vanessa219
Copy link
Member

这两个加号合并为一个吧

@gdy106
Copy link
Author

gdy106 commented Jul 26, 2023

这两个加号合并为一个吧

都行,我就提供一个横排菜单的思路。我是觉得非必要的选项可以不出现,点+号选择了复制还是粘贴,接下来跳出的选项就可以少一点。
不过两行也不算多,反正塞得下,放一起也可以。
要进一步精简还可以把复制粘贴和剪切的文字删掉,仅保留图标。毕竟图标足够直观了。(当然,反过来也可以,删图标仅保留文字)

@Vanessa219 Vanessa219 self-assigned this Jul 26, 2023
@88250 88250 changed the title 手机端选中文字后的快捷菜单会遮挡文字 Improve mobile selection text menu Jul 27, 2023
@Vanessa219 Vanessa219 added this to the 2.9.9 milestone Aug 4, 2023
Vanessa219 added a commit that referenced this issue Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants