EhViewer-NekoInverter / EhViewer

🥥 A fork of EhViewer, feature requests are not accepted. Forked from https://gitlab.com/NekoInverter/EhViewer
GNU General Public License v3.0
2.77k stars 129 forks source link

一旦勾選「記住常用收藏」後,便無法離開此狀態(無法變回每次詢問) #31

Closed ximellon closed 1 year ago

ximellon commented 1 year ago

复现步骤 / Step to reproduce

  1. 將任一畫廊加入收藏時,勾選「記住常用收藏」
  2. 點擊進入畫廊詳情
  3. 點擊收藏(愛心),取消收藏
  4. 長按收藏強制觸發收藏選單,這次不勾選「記住常用收藏」
  5. 完成加入流程後,再次嘗試任一非強制觸發收藏選單的加入收藏流程

预期行为 / Expected behaviour

以前在上述的 4 以後,就可以變回每次詢問的行為模式,但現在並沒有辦法。

实际行为 / Actual Behaviour

現在的狀況是:依舊維持「記住常用收藏」的狀態,不管從外面長按呼出選單後加入,或在裡頭(詳情)直接點擊愛心,都還是會直接加入前一次記住時的收藏夾。

备注 / Addition details

跟崩潰無關、且看 commit 沒有修正此類問題,就不測 CI 和附上 log 了。

我大致上看了一下,問題應該是出在

app/src/main/java/com/hippo/ehviewer/ui/CommonOperations.java

L113 部分的流程沒有被正常觸發而導致此結果(不熟 Android app 不確定是不是流程被後頭增加備註的部分 (L79 附近) 給跳過了)


我個人是覺得以前的流程也挺詭異的,看你如何改方便,在設定多加一個「預設收藏」之類的選項也是可以接受的(吧?)

EhViewer 版本号 / EhViewer version code

1.7.28.7

EhViewer CI 版本 / EhViewer CI Version

N/A

Android 系统版本 / Android version

One UI 3.1/Android 11

设备型号 / Device model

Galaxy Tab A 8.0 with S Pen (SM-P200)

SoC 型号 / Soc model

Exynos 7904

应用日志 / App logs

N/A

自查步骤 / Verify steps

xb2016 commented 1 year ago

以及正确做法 image

ximellon commented 1 year ago

我剛剛試了,確實可以。原來的 app 就有這個東西嗎?看來之前我用的是奇怪的方法。 =P

xb2016 commented 1 year ago

一直都有

ximellon commented 1 year ago

https://github.com/EhViewer-NekoInverter/EhViewer/commit/867bf43c68cc455a2e8311fd13f5f1a0c24b8e00

找到了,原來長按改收藏是上個作者加進去的,我沒用過最早的,所以一直都是這樣操作。 xD

剛才看不懂的 select 也是這裡加進去的。 OTL