3pp-mirror/penpot
1
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2025-06-01 01:51:38 +02:00
Code Issues Projects Releases Packages Wiki Activity

🐛 Properly dispose rx subscription on grid thumbnail component

Browse source
This commit is contained in:
Andrey Antukh 2025-04-22 21:39:57 +02:00
parent 05fac41534
commit 1194e40222
1 changed files with 10 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

18
frontend/src/app/main/ui/dashboard/grid.cljs
View file

@ -89,14 +89,16 @@
(mf/with-effect [file-id revn visible? thumbnail-id] (mf/with-effect [file-id revn visible? thumbnail-id]
(when (and visible? (not thumbnail-id)) (when (and visible? (not thumbnail-id))
(->> (ask-for-thumbnail file-id revn) (let [subscription
(rx/subs! (fn [thumbnail-id] (->> (ask-for-thumbnail file-id revn)
(st/emit! (dd/set-file-thumbnail file-id thumbnail-id))) (rx/subs! (fn [thumbnail-id]
(fn [cause] (st/emit! (dd/set-file-thumbnail file-id thumbnail-id)))
(log/error :hint "unable to render thumbnail" (fn [cause]
:file-if file-id (log/error :hint "unable to render thumbnail"
:revn revn :file-if file-id
:message (ex-message cause))))))) :revn revn
:message (ex-message cause)))))]
(partial rx/dispose! subscription))))
[:div {:class (stl/css :grid-item-th) [:div {:class (stl/css :grid-item-th)
:style {:background-color bg-color} :style {:background-color bg-color}