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

Add feature: hide bookmarked pictures #62

Merged
merged 5 commits into from
Apr 27, 2020
Merged

Conversation

ultranity
Copy link
Contributor

Add feature: hide bookmarked pictures

@ultranity
Copy link
Contributor Author

隐藏已经收藏过的图在收图之类场景还挺有用的,分别在搜索页面的筛选对话框中和用户界面的右上角菜单中增加隐藏功能

@ultranity
Copy link
Contributor Author

网页上操作结果手滑把另一个关于缓存删除的也提交到这里了
目前的cache策略是否包括定期清除?我没有找到相关内容。只是file.copyto()的话 #34 问题就又回来了-_-,所以增加了file.delete()

@Notsfsssf
Copy link
Owner

根据 Aria的文档描述,app在onCreate内有调用Aria.download(this).removeAllTask(true),已经包括删除下载缓存的效果了
这个feature确实挺实用的,我会在确认以后合并👍

@Notsfsssf
Copy link
Owner

搜索页toggleButton改变后并没有立刻刷新列表,导致需要人工的刷新一次才会生效,这个能不能优化一下

@ultranity
Copy link
Contributor Author

ultranity commented Apr 27, 2020

搜索页 toggleButton 改变后并没有立刻刷新列表,导致需要人工的刷新一次才会生效,这个能不能优化一下

之前没注意这个问题,但搜索筛选对话框在点击 确定 的时候应该会触发刷新?
不过现在确实有不点击确定却保存toggle状态的bug
但使每次进入筛选对话框都能显示当前的筛选内容更符合交互直觉(标签一致和按照顺序也应该作相似改动)

@Notsfsssf Notsfsssf merged commit 74f87f8 into Notsfsssf:master Apr 27, 2020
@Notsfsssf
Copy link
Owner

搜索页还是有问题,我处理一下吧

@Notsfsssf
Copy link
Owner

搜索页还是有问题,我处理一下吧

应该是Preview和普通搜索的接口不同影响到了,没什么大碍,挺实用的feature👍

@ultranity
Copy link
Contributor Author

搜索页还是有问题,我处理一下吧

能具体点描述下哪里有问题或者怎么复现吗,其实我目前使用中没发现哪里有异常:joy:

@Notsfsssf
Copy link
Owner

搜索页还是有问题,我处理一下吧

能具体点描述下哪里有问题或者怎么复现吗,其实我目前使用中没发现哪里有异常😂

当左上角点热门以后,在热门的列表点收藏,再点dialog的toggle切换到隐藏收藏以后确认,会回调刷新一下列表,但是会刷回‘默认’返回的插画,这应该是我的锅...

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