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

feat: Add a new grid mode for message display. #1626

Merged
merged 9 commits into from
Feb 17, 2025
Merged

Conversation

cl1107
Copy link
Contributor

@cl1107 cl1107 commented Feb 14, 2025

新增网格模式的消息展示方式#1597

CleanShot 2025-02-14 at 11 36 25@2x
CleanShot 2025-02-14 at 11 36 54@2x

目前实现是 popover hover 就展示,然后宫格内我直接overflow:hidden了, 另一种是 popover click 再展示,宫格就算小也让用户可以滚动查看, 这两种实现可以抉择一下

@cl1107 cl1107 changed the title 新增网格模式的消息展示方式 feat: Add a new grid mode for message display. Feb 14, 2025
@eric-gitta-moore
Copy link

我天,大佬这么快,我还想着没人搞我自己搞

@eric-gitta-moore
Copy link

目前实现是 popover hover 就展示,然后宫格内我直接overflow:hidden了, 另一种是 popover click 再展示,宫格就算小也让用户可以滚动查看, 这两种实现可以抉择一下

是不是可以在 # 按钮上,弄个右键,里面设置列数和 popover 方式会比较好

@kangfenmao
Copy link
Collaborator

不错,大家多测试测试,准备放到下下个版本发

@cl1107
Copy link
Contributor Author

cl1107 commented Feb 14, 2025

目前实现是 popover hover 就展示,然后宫格内我直接overflow:hidden了, 另一种是 popover click 再展示,宫格就算小也让用户可以滚动查看, 这两种实现可以抉择一下

是不是可以在 # 按钮上,弄个右键,里面设置列数和 popover 方式会比较好

做成配置的话,我觉得放设置-消息设置里更合理

@kangfenmao
Copy link
Collaborator

稍微等一等,我们需要一段时间来测试有没有隐藏的问题还没有发现

@kangfenmao
Copy link
Collaborator

kangfenmao commented Feb 16, 2025

image

我在网格视图展开的情况下,想再加一个模型进来的时候出现了这个

@kangfenmao
Copy link
Collaborator

网格左侧需要和这条线对齐

image

@kangfenmao
Copy link
Collaborator

这里也需要同步增加一个状态

image

当用户选择网格的时候,下方才能出现下面这个触发方式的选项

@kangfenmao
Copy link
Collaborator

其他好像都挺好的,用着感觉非常不错,而且完成度很高

@cl1107
Copy link
Contributor Author

cl1107 commented Feb 16, 2025

上述3个都处理了

@kangfenmao kangfenmao merged commit bad2f15 into CherryHQ:main Feb 17, 2025
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.

3 participants