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: 资源收藏 #4933

Open
wants to merge 51 commits into
base: main
Choose a base branch
from
Open

Conversation

tangge233
Copy link
Contributor

@tangge233 tangge233 commented Oct 11, 2024

前言

好了好了,不再多开坑了()

ToDo

  • 实现加入收藏功能
  • 实现收藏查看
  • 实现搜索功能
  • 使用更小的可多选组件
  • 实现批量操作

预览

image

image

杂项

要造的轮子比较多,开发稍慢 Orz
加载器不是很会使用,所以会写出高血压代码,请多见谅 Orz

Close #487 (关联投票 #1646

@JingHai-Lingyun JingHai-Lingyun added the 社区处理中 社区正在调查或处理该项 label Oct 11, 2024
@LTCatt
Copy link
Member

LTCatt commented Oct 11, 2024

umm 简单说下我的预期……

收藏的内容技术上应该是可以放到同一个 Page 的,顶多不同类型的分到不同的 Card 下面,没必要每种类型开一个新页面。
image

所有收藏内容的获取可以只通过一次加载器。两个网站的 API 都可以从工程 ID 的数组定向获取对应内容。

@tangge233
Copy link
Contributor Author

tangge233 commented Oct 12, 2024

基础功能已完成,需要人员帮忙测试稳定性

社区成员已帮忙进行测试
感谢 @Ad-closeNN 帮忙测试

@tangge233 tangge233 marked this pull request as ready for review October 12, 2024 14:01
@tangge233 tangge233 requested a review from LTCatt October 12, 2024 16:43
@tangge233 tangge233 marked this pull request as ready for review October 18, 2024 11:45
@tangge233 tangge233 requested a review from LTCatt October 19, 2024 13:41
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 暂无计划 短期内没有制作该项的计划 and removed 社区处理中 社区正在调查或处理该项 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 labels Oct 19, 2024
@tangge233
Copy link
Contributor Author

tangge233 commented Nov 11, 2024

感觉可以对 MyListItem 做点修改,这样就可以抛弃 MyMiniCompItem 了,过几天看看()

内容完成

If TypeOf (value) Is List(Of String) Then
list = CType(value, List(Of String))
End If
PanTags.Children.Clear()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

如果这样整的话,PanTags 最好也不要直接加载:
不要把它放进 XAML
如果 Tags 不是空列表,才新建这个 StackPanel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· 新功能 暂无计划 短期内没有制作该项的计划
Projects
None yet
Development

Successfully merging this pull request may close these issues.

增加Mod收藏夹功能
5 participants