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

新增直播间开窗增加cookie输入框,方便粘贴cookie,避免手动修改配置文件产生错误. #812

Closed
wants to merge 1 commit into from

Conversation

murphysking
Copy link
Contributor

新增直播间开窗增加cookie输入框,方便粘贴cookie,避免手动修改配置文件产生错误. #806 #810

@kira1928
Copy link
Collaborator

kira1928 commented Dec 4, 2024

我有几个想法:

  1. 添加直播间的窗口默认显示输入 cookie 的输入框可能让大多数不需要了解 cookie 相关知识的用户感到困惑。
    我觉得可以在添加界面下面加一个 高级设置 的按钮,点击后对话框下面可以展开 cookie 等等更多设置。
    但这样也可能让用户误以为这个cookie是每个直播间单独设置的,让 UI 的引导有点混论。

  2. 添加的时候最好能在前端粗略验证一下用户输入的 cookie 格式正确与否,进一步减少用户填错的可能。

  3. 添加的地方有输入框,但添加完成后对应直播间没有查看和修改刚才填入的 cookie 的界面。
    可能用户手滑填错了或者想检查一下刚才填的内容就不知道在哪里看了 (针对不知道 cookie 怎么填,也不知道在 config 里可以看到填入的 cookie 内容的用户)
    这样用户犯错后自己改正的难度比较大,可能带来更多重复 issue 和提问。

也许我们可以把添加删除查看全局 cookie 设置的功能单独做一个设置 UI,而不是把它和添加直播对话框绑定?

@murphysking
Copy link
Contributor Author

这个想法很棒,我会试着修改,不过前端部分的修改可能超出我的能力范围了,我对react不太了解

@murphysking murphysking closed this Dec 5, 2024
@kira1928
Copy link
Collaborator

kira1928 commented Dec 5, 2024

大佬对不起😣

@murphysking
Copy link
Contributor Author

基于上面的建议,我的解决方法是:
1、添加直播间界面增加高级设置,默认折叠,将cookie设置以及后续可能新增的设置项放在里面
2、cookie格式验证仅限于必选项name=value,其他部分没法验证
3、现有监控列表改为多页签显示,增加cookie管理页签,将直播间列表host去重后与cookies合并,可以添加和删除cookie
如果有其他更换的建议,欢迎提出。如果没有其他建议的话,我会着手开始修改

@kira1928
Copy link
Collaborator

kira1928 commented Dec 5, 2024

大佬nb!
关于第三点,我有点没理解多页签是什么效果。可以贴一下类似ui的截图吗?
感谢!

@kira1928
Copy link
Collaborator

kira1928 commented Dec 5, 2024

另外大佬可以新开一个issue,我们在那边继续聊。
在一个closed的pr下面聊我怕后面找不到了。。

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