From 01ac1529e15bbc929ff694a245089b9fa6668bea Mon Sep 17 00:00:00 2001 From: Dexterity <173429049+Dexterity104@users.noreply.github.com> Date: Fri, 29 May 2026 03:41:21 -0400 Subject: [PATCH] :recycle: Migrate perf/profiler to modern component syntax (#9429) Co-authored-by: Andrey Antukh --- frontend/src/app/util/perf.cljs | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/frontend/src/app/util/perf.cljs b/frontend/src/app/util/perf.cljs index e3fb8fce47..7e7d6b8230 100644 --- a/frontend/src/app/util/perf.cljs +++ b/frontend/src/app/util/perf.cljs @@ -15,11 +15,11 @@ [rumext.v2 :as mf])) ;; For use it, just wrap the component you want to profile with -;; `perf/profiler` component and pass a label for debug purpose. +;; `perf/profiler*` component and pass a label for debug purpose. ;; ;; Example: ;; -;; [:& perf/profiler {:label "viewport"} +;; [:> perf/profiler* {:label "viewport"} ;; [:section ;; [:& some-component]]] ;; @@ -90,17 +90,12 @@ (conj! td adur) (log phase td)))) -(mf/defc profiler - {::mf/wrap-props false} - [props] - (let [children (unchecked-get props "children") - label (unchecked-get props "label") - enabled? (unchecked-get props "enabled") - enabled? (if (nil? enabled?) true enabled?) - on-render (mf/use-memo +(mf/defc profiler* + [{:keys [children label enabled] :or {enabled true}}] + (let [on-render (mf/use-memo (mf/deps label) #(on-render-factory label))] - (if enabled? + (if enabled [:> react/Profiler #js {:id label :onRender on-render} children]