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

✨ Remove a/b testing code for onboarding questions

Browse source
This commit is contained in:
Alejandro Alonso 2024-06-14 07:10:16 +02:00
parent 68741bb56f
commit 043769c255
8 changed files with 163 additions and 192 deletions
Download patch file Download diff file Expand all files Collapse all files

3
frontend/resources/styles/common/refactor/basic-rules.scss
View file

@ -590,9 +590,6 @@
width: 100%; width: 100%;
z-index: $z-index-modal; z-index: $z-index-modal;
background-color: var(--overlay-color); background-color: var(--overlay-color);
&.onboarding-a-b-test {
background-color: var(--overlay-color-onboarding-a-b-test);
}
} }
.modal-container-base { .modal-container-base {

2
frontend/resources/styles/common/refactor/color-defs.scss
View file

@ -11,7 +11,6 @@
// Dark background // Dark background
--db-primary: #18181a; --db-primary: #18181a;
--db-primary-60: #{color.change(#18181a, $alpha: 0.6)}; --db-primary-60: #{color.change(#18181a, $alpha: 0.6)};
--db-primary-90: #{color.change(#18181a, $alpha: 0.9)};
--db-secondary: #000000; --db-secondary: #000000;
--db-secondary-30: #{color.change(#000000, $alpha: 0.3)}; --db-secondary-30: #{color.change(#000000, $alpha: 0.3)};
--db-secondary-80: #{color.change(#000000, $alpha: 0.8)}; --db-secondary-80: #{color.change(#000000, $alpha: 0.8)};
@ -36,7 +35,6 @@
// Light background // Light background
--lb-primary: #ffffff; --lb-primary: #ffffff;
--lb-primary-60: #{color.change(#ffffff, $alpha: 0.6)}; --lb-primary-60: #{color.change(#ffffff, $alpha: 0.6)};
--lb-primary-90: #{color.change(#ffffff, $alpha: 0.9)};
--lb-secondary: #e8eaee; --lb-secondary: #e8eaee;
--lb-secondary-30: #{color.change(#e8eaee, $alpha: 0.3)}; --lb-secondary-30: #{color.change(#e8eaee, $alpha: 0.3)};
--lb-secondary-80: #{color.change(#e8eaee, $alpha: 0.8)}; --lb-secondary-80: #{color.change(#e8eaee, $alpha: 0.8)};

1
frontend/resources/styles/common/refactor/themes/default-theme.scss
View file

@ -37,7 +37,6 @@
--color-info-foreground: var(--status-color-info-500); --color-info-foreground: var(--status-color-info-500);
--overlay-color: var(--db-primary-60); --overlay-color: var(--db-primary-60);
--overlay-color-onboarding-a-b-test: var(--db-primary-90);
--shadow-color: var(--db-secondary-30); --shadow-color: var(--db-secondary-30);
--radio-button-box-shadow: 0 0 0 1px var(--db-secondary-30) inset; --radio-button-box-shadow: 0 0 0 1px var(--db-secondary-30) inset;

1
frontend/resources/styles/common/refactor/themes/light-theme.scss
View file

@ -37,7 +37,6 @@
--color-info-foreground: var(--status-color-info-500); --color-info-foreground: var(--status-color-info-500);
--overlay-color: var(--lb-primary-60); --overlay-color: var(--lb-primary-60);
--overlay-color-onboarding-a-b-test: var(--lb-primary-90);
--shadow-color: var(--lf-secondary-40); --shadow-color: var(--lf-secondary-40);
--radio-button-box-shadow: 0 0 0 1px var(--lb-secondary) inset; --radio-button-box-shadow: 0 0 0 1px var(--lb-secondary) inset;

9
frontend/src/app/main/ui/onboarding/newsletter.cljs
View file

@ -7,7 +7,6 @@
(ns app.main.ui.onboarding.newsletter (ns app.main.ui.onboarding.newsletter
(:require-macros [app.main.style :as stl]) (:require-macros [app.main.style :as stl])
(:require (:require
[app.config :as cf]
[app.main.data.events :as-alias ev] [app.main.data.events :as-alias ev]
[app.main.data.messages :as msg] [app.main.data.messages :as msg]
[app.main.data.users :as du] [app.main.data.users :as du]
@ -45,14 +44,10 @@
(assoc :label "newsletter:subscriptions") (assoc :label "newsletter:subscriptions")
(assoc :step 6))] (assoc :step 6))]
(st/emit! (ptk/data-event ::ev/event params) (st/emit! (ptk/data-event ::ev/event params)
(du/update-profile-props state))))) (du/update-profile-props state)))))]
onboarding-a-b-test?
(cf/external-feature-flag "signup-background" "test")]
[:div {:class (stl/css-case [:div {:class (stl/css-case
:modal-overlay true :modal-overlay true)}
:onboarding-a-b-test onboarding-a-b-test?)}
[:div.animated.fadeInDown {:class (stl/css :modal-container)} [:div.animated.fadeInDown {:class (stl/css :modal-container)}
[:div {:class (stl/css :modal-left)} [:div {:class (stl/css :modal-left)}

9
frontend/src/app/main/ui/onboarding/questions.cljs
View file

@ -10,7 +10,6 @@
(:require (:require
[app.common.data :as d] [app.common.data :as d]
[app.common.data.macros :as dm] [app.common.data.macros :as dm]
[app.config :as cf]
[app.main.data.events :as-alias ev] [app.main.data.events :as-alias ev]
[app.main.data.users :as du] [app.main.data.users :as du]
[app.main.store :as st] [app.main.store :as st]
@ -497,14 +496,10 @@
(fn [form] (fn [form]
(let [data (merge @clean-data (:clean-data @form))] (let [data (merge @clean-data (:clean-data @form))]
(reset! clean-data data) (reset! clean-data data)
(st/emit! (du/mark-questions-as-answered data))))) (st/emit! (du/mark-questions-as-answered data)))))]
onboarding-a-b-test?
(cf/external-feature-flag "signup-background" "test")]
[:div {:class (stl/css-case [:div {:class (stl/css-case
:modal-overlay true :modal-overlay true)}
:onboarding-a-b-test onboarding-a-b-test?)}
[:div {:class (stl/css :modal-container) [:div {:class (stl/css :modal-container)
:ref container} :ref container}

9
frontend/src/app/main/ui/onboarding/team_choice.cljs
View file

@ -9,7 +9,6 @@
(:require (:require
[app.common.data.macros :as dm] [app.common.data.macros :as dm]
[app.common.spec :as us] [app.common.spec :as us]
[app.config :as cf]
[app.main.data.dashboard :as dd] [app.main.data.dashboard :as dd]
[app.main.data.events :as ev] [app.main.data.events :as ev]
[app.main.data.messages :as msg] [app.main.data.messages :as msg]
@ -260,14 +259,10 @@
on-back on-back
(mf/use-fn (mf/use-fn
(fn [] (fn []
(swap! name* (constantly nil)))) (swap! name* (constantly nil))))]
onboarding-a-b-test?
(cf/external-feature-flag "signup-background" "test")]
[:div {:class (stl/css-case [:div {:class (stl/css-case
:modal-overlay true :modal-overlay true)}
:onboarding-a-b-test onboarding-a-b-test?)}
[:div.animated.fadeIn {:class (stl/css :modal-container)} [:div.animated.fadeIn {:class (stl/css :modal-container)}
[:& left-sidebar] [:& left-sidebar]

19
frontend/src/app/main/ui/releases/v2_0.cljs
View file

@ -8,19 +8,16 @@
(:require-macros [app.main.style :as stl]) (:require-macros [app.main.style :as stl])
(:require (:require
[app.common.data.macros :as dm] [app.common.data.macros :as dm]
[app.config :as cf]
[app.main.ui.releases.common :as c] [app.main.ui.releases.common :as c]
[rumext.v2 :as mf])) [rumext.v2 :as mf]))
;; TODO: Review all copies and alt text ;; TODO: Review all copies and alt text
(defmethod c/render-release-notes "2.0" (defmethod c/render-release-notes "2.0"
[{:keys [slide klass next finish navigate version]}] [{:keys [slide klass next finish navigate version]}]
(let [onboarding-a-b-test? (cf/external-feature-flag "signup-background" "test")]
(mf/html (mf/html
(case slide (case slide
:start :start
[:div {:class (stl/css-case :modal-overlay true [:div {:class (stl/css-case :modal-overlay true)}
:onboarding-a-b-test onboarding-a-b-test?)}
[:div.animated {:class klass} [:div.animated {:class klass}
[:div {:class (stl/css :modal-container)} [:div {:class (stl/css :modal-container)}
[:img {:src "images/features/2.0-intro-image.png" [:img {:src "images/features/2.0-intro-image.png"
@ -64,8 +61,7 @@
:on-click next} "Continue"]]]]]] :on-click next} "Continue"]]]]]]
0 0
[:div {:class (stl/css-case :modal-overlay true [:div {:class (stl/css-case :modal-overlay true)}
:onboarding-a-b-test onboarding-a-b-test?)}
[:div.animated {:class klass} [:div.animated {:class klass}
[:div {:class (stl/css :modal-container)} [:div {:class (stl/css :modal-container)}
[:img {:src "images/features/2.0-css-grid.gif" [:img {:src "images/features/2.0-css-grid.gif"
@ -105,8 +101,7 @@
:class (stl/css :next-btn)} "Continue"]]]]]] :class (stl/css :next-btn)} "Continue"]]]]]]
1 1
[:div {:class (stl/css-case :modal-overlay true [:div {:class (stl/css-case :modal-overlay true)}
:onboarding-a-b-test onboarding-a-b-test?)}
[:div.animated {:class klass} [:div.animated {:class klass}
[:div {:class (stl/css :modal-container)} [:div {:class (stl/css :modal-container)}
[:img {:src "images/features/2.0-new-ui.gif" [:img {:src "images/features/2.0-new-ui.gif"
@ -140,8 +135,7 @@
:class (stl/css :next-btn)} "Continue"]]]]]] :class (stl/css :next-btn)} "Continue"]]]]]]
2 2
[:div {:class (stl/css-case :modal-overlay true [:div {:class (stl/css-case :modal-overlay true)}
:onboarding-a-b-test onboarding-a-b-test?)}
[:div.animated {:class klass} [:div.animated {:class klass}
[:div {:class (stl/css :modal-container)} [:div {:class (stl/css :modal-container)}
[:img {:src "images/features/2.0-components.gif" [:img {:src "images/features/2.0-components.gif"
@ -176,8 +170,7 @@
:class (stl/css :next-btn)} "Continue"]]]]]] :class (stl/css :next-btn)} "Continue"]]]]]]
3 3
[:div {:class (stl/css-case :modal-overlay true [:div {:class (stl/css-case :modal-overlay true)}
:onboarding-a-b-test onboarding-a-b-test?)}
[:div.animated {:class klass} [:div.animated {:class klass}
[:div {:class (stl/css :modal-container)} [:div {:class (stl/css :modal-container)}
[:img {:src "images/features/2.0-html.gif" [:img {:src "images/features/2.0-html.gif"
@ -206,5 +199,5 @@
:total 4}] :total 4}]
[:button {:on-click finish [:button {:on-click finish
:class (stl/css :next-btn)} "Let's go"]]]]]])))) :class (stl/css :next-btn)} "Let's go"]]]]]])))