Skip to content

Cancel the read-only mode message notification after startup #9100

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

Closed
HopeLHP opened this issue Sep 2, 2023 · 10 comments
Closed

Cancel the read-only mode message notification after startup #9100

HopeLHP opened this issue Sep 2, 2023 · 10 comments
Assignees
Milestone

Comments

@HopeLHP
Copy link

HopeLHP commented Sep 2, 2023

只读模式取消启动后弹窗提醒

In what scenarios do you need this feature?

为了避免误操作改变文档内容,只读模式是我日常保持打开的模式。
但是只读模式启动后弹窗,让我没办法启动后立马点右上角的插件或按钮(包括退出只读),必须要点击打钩或者等它消失,如图
image


不清楚当初为什么要设置只读模式打开后弹窗提醒?
如果只是为了提醒人们目前在只读模式无法进行文档操作,那么右上角红色的退出只读按钮完全能做到提醒的作用,更持久更美观。
我相信大家都讨厌广告弹窗,之所以广告弹窗令人讨厌,我想就是它没有意义,现在只读模式的弹窗就像个狗皮膏药,每天打开siyuan都要撕一遍,非常影响整体感。

Describe the optimal solution

取消弹窗提醒即可,如有必要加强“退出只读”按钮的面积或颜色

Describe the candidate solution

No response

Other information

No response

@88250
Copy link
Member

88250 commented Sep 2, 2023

没有提示不行的,之前太多用户来问为什么无法编辑,不是这么多的反馈我们也不会画蛇添足的,感谢反馈。

@88250 88250 closed this as completed Sep 2, 2023
@HopeLHP
Copy link
Author

HopeLHP commented Sep 2, 2023

首先,我认为退出只读的按钮比强制弹窗提醒效果好。不知道当时右上角有没有退出只读的按钮,我认为红色的按钮已经能起到提示的作用了,如果他们还无法看到这个一直存在的且颜色鲜明的按钮,难道还能期望他们能注意到只在启动时才出现的弹窗吗?
其次,真的是因为启动的弹窗让他们立马知道还在只读模式吗?我每次发现想要编辑内容但无法编辑,都停顿一下,然后习惯性的想起或瞟到右上角的红色按钮,而不是因为启动时给我的那个弹窗。
再次,启动时的弹窗或许对第一次打开siyuan或刚开始使用的人有那么一点强制性提醒作用,但是要为此而在以后的每次都弹窗,我想这有点因噎废食。
最后,我不否认弹窗的强制提醒作用的效果,但我认为应该把这种弹窗用在例如同步失败、内核崩溃等等重要的有意义的场景上,而不是这种提示效果边际递减的场景。

@88250
Copy link
Member

88250 commented Sep 2, 2023

默认主题不是红色的……而是这样的:

image

@HopeLHP
Copy link
Author

HopeLHP commented Sep 2, 2023

默认主题不是红色的

所以我说我们可以用更好的办法解决用户忘记只读模式的问题,就比如写未主题的按钮就显得很自然不突兀,至少我认为强制弹窗不是一个好的解决办法。
只读模式刚发布时或许用户还不明白,但是人总是有记忆力的或者养成习惯的,所以现在大家提无法编辑的问题变少了,会不会只是因为习惯了,而不是这个强制提醒的弹窗。所以我说提示效果的边际递减的,但是他给别人造成的麻烦是一直存在或递增的。

@88250
Copy link
Member

88250 commented Sep 2, 2023 via email

@88250
Copy link
Member

88250 commented Sep 9, 2023

#9145#8417 中调整了只读模式界面设计,加了较为明显的图标:

image

现在可以取消这个通知了 #7700

@88250 88250 reopened this Sep 9, 2023
@88250 88250 self-assigned this Sep 9, 2023
@88250 88250 added this to the 2.10.5 milestone Sep 9, 2023
@88250 88250 changed the title 只读模式取消启动后弹窗提醒 Cancel the read-only mode message notification after startup Sep 9, 2023
88250 added a commit that referenced this issue Sep 9, 2023

Verified

This commit was signed with the committer’s verified signature.
@88250 88250 closed this as completed Sep 9, 2023
EightDoor added a commit to EightDoor/siyuan that referenced this issue Sep 9, 2023
* commit '7a7af86b11b6ded088bc3db6769dd63f1ce6229f': (26 commits)
  🎨 aria label
  🎨 Pandoc template for exporting docx Fix siyuan-note#8740
  🎨 Pandoc template for exporting docx Fix siyuan-note#8740
  🎨 Cancel the read-only mode message notification after startup Fix siyuan-note#9100
  🎨 siyuan-note#8417
  🐛 An issue that caused cloud data corruption Fix siyuan-note#9144
  🐛 fix siyuan-note#9142
  🐛 fix siyuan-note#9141
  🐛 fix siyuan-note#9141
  🎨 Set Adaptive Width for individual documents siyuan-note#9107
  🎨 siyuan-note#9107
  🎨 siyuan-note#9134
  ♻️
  🎨 siyuan-note#9134
  🎨 Physically delete a workspace on the mobile siyuan-note#9134
  🎨 Physically delete a workspace on the mobile siyuan-note#9134
  🎨 fix siyuan-note#9134
  🎨 fix siyuan-note#9140
  🎨 Physically delete a workspace on the mobile siyuan-note#9134
  🔖 Release v2.10.4
  ...
@mozhux
Copy link

mozhux commented Sep 9, 2023

#9145#8417 中调整了只读模式界面设计,加了较为明显的图标:

image

现在可以取消这个通知了 #7700

图标换成原来顶栏全局的那个吧,眼睛和笔的,不然这个锁还以为是笔记加锁了呢

@88250
Copy link
Member

88250 commented Sep 9, 2023

@mozhux 用锁图标好像较为通用,比如 Notion 和 IDE

@88250
Copy link
Member

88250 commented Sep 10, 2023 via email

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

4 participants
@88250 @mozhux @HopeLHP and others