Penpot/penpot
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2025-02-13 18:48:37 -05:00
Code Issues Activity

Merge pull request #2260 from penpot/eva-layout-effect

Browse source 
🐛 Fix layout effects in viewer
This commit is contained in:
Andrey Antukh 2022-09-20 07:43:56 +02:00 • committed by GitHub
commit c6e33fa9bc
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
frontend/src/app/main/ui/viewer.cljs
View file

@ -354,14 +354,14 @@
(events/unlistenByKey key1)
(events/unlistenByKey key2))))
(mf/use-layout-effect
(mf/use-effect
(fn []
(set-up-new-size)
(.addEventListener js/window "resize" set-up-new-size)
(fn []
(.removeEventListener js/window "resize" set-up-new-size))))
(mf/use-layout-effect
(mf/use-effect
(mf/deps nav-scroll)
(fn []
;; Set scroll position after navigate
@ -381,19 +381,19 @@
(wapi/request-fullscreen wrapper)
(wapi/exit-fullscreen))))))
(mf/use-layout-effect
(mf/deps page)
(mf/use-effect
(mf/deps zoom-type)
(fn []
(case zoom-type
:fit (st/emit! dv/zoom-to-fit)
:fill (st/emit! dv/zoom-to-fill)
nil)))
(mf/use-layout-effect
(mf/deps index current-animations)
(mf/use-effect
(mf/deps index current-animations zoom-type)
(fn []
(case zoom-type
:fit (st/emit! dv/zoom-to-fit)
:fit (st/emit! dv/zoom-to-fit)
:fill (st/emit! dv/zoom-to-fill)
nil)
;; Navigate animation needs to be started after navigation
@ -410,7 +410,7 @@
orig-size
wrapper-size))))))
(mf/use-layout-effect
(mf/use-effect
(mf/deps current-animations)
(fn []
;; Overlay animations may be started when needed.