🐛 Fix some warnings and format some files

This commit is contained in:
Eva 2023-07-03 12:57:59 +02:00 committed by Alejandro Alonso
parent 8955f87d5a
commit ef3fedee59
3 changed files with 35 additions and 34 deletions

View file

@ -25,7 +25,7 @@
on-key-down (gobj/get props "on-key-down") on-key-down (gobj/get props "on-key-down")
id (gobj/get props "id") id (gobj/get props "id")
klass (gobj/get props "klass") klass (gobj/get props "klass")
key (gobj/get props "key") key (gobj/get props "unique-key")
data-test (gobj/get props "data-test")] data-test (gobj/get props "data-test")]
[:li {:id id [:li {:id id
:class klass :class klass

View file

@ -243,6 +243,7 @@
(when (kbd/enter? event) (when (kbd/enter? event)
(team-selected (:default-team-id profile) event))) (team-selected (:default-team-id profile) event)))
:id "teams-selector-default-team" :id "teams-selector-default-team"
:unique-key "default-team"
:klass "team-name"} :klass "team-name"}
[:span.team-icon i/logo-icon] [:span.team-icon i/logo-icon]
[:span.team-text (tr "dashboard.your-penpot")] [:span.team-text (tr "dashboard.your-penpot")]
@ -256,7 +257,7 @@
(team-selected (:id team-item) event))) (team-selected (:id team-item) event)))
:id (str "teams-selector-" (:id team-item)) :id (str "teams-selector-" (:id team-item))
:klass "team-name" :klass "team-name"
:key (dm/str (:id team-item))} :unique-key (dm/str (:id team-item))}
[:span.team-icon [:span.team-icon
[:img {:src (cf/resolve-team-photo-url team-item) [:img {:src (cf/resolve-team-photo-url team-item)
:alt (:name team-item)}]] :alt (:name team-item)}]]
@ -270,7 +271,7 @@
(on-create-clicked event))) (on-create-clicked event)))
:id "teams-selector-create-team" :id "teams-selector-create-team"
:klass "team-name action" :klass "team-name action"
:key "teams-selector-create-team"} :unique-key "teams-selector-create-team"}
[:span.team-icon.new-team i/close] [:span.team-icon.new-team i/close]
[:span.team-text (tr "dashboard.create-new-team")]]])) [:span.team-text (tr "dashboard.create-new-team")]]]))
@ -364,16 +365,16 @@
(when (kbd/enter? event) (when (kbd/enter? event)
(go-members))) (go-members)))
:id "teams-options-members" :id "teams-options-members"
:key "teams-options-members" :unique-key "teams-options-members"
:data-test "team-members"} :data-test "team-members"}
(tr "labels.members")] (tr "labels.members")]
[:& dropdown-menu-item {:on-click go-invitations [:& dropdown-menu-item {:on-click go-invitations
:on-key-down (fn [event] :on-key-down (fn [event]
(when (kbd/enter? event) (when (kbd/enter? event)
(go-invitations))) (go-invitations)))
:id "teams-options-invitations" :id "teams-options-invitations"
:key "teams-options-invitations" :unique-key "teams-options-invitations"
:data-test "team-invitations"} :data-test "team-invitations"}
(tr "labels.invitations")] (tr "labels.invitations")]
(when (contains? cf/flags :webhooks) (when (contains? cf/flags :webhooks)
@ -382,7 +383,7 @@
(when (kbd/enter? event) (when (kbd/enter? event)
(go-webhooks))) (go-webhooks)))
:id "teams-options-webhooks" :id "teams-options-webhooks"
:key "teams-options-webhooks"} :unique-key "teams-options-webhooks"}
(tr "labels.webhooks")]) (tr "labels.webhooks")])
[:& dropdown-menu-item {:on-click go-settings [:& dropdown-menu-item {:on-click go-settings
@ -390,8 +391,8 @@
(when (kbd/enter? event) (when (kbd/enter? event)
(go-settings))) (go-settings)))
:id "teams-options-settings" :id "teams-options-settings"
:key "teams-options-settings" :unique-key "teams-options-settings"
:data-test "team-settings"} :data-test "team-settings"}
(tr "labels.settings")] (tr "labels.settings")]
[:hr] [:hr]
@ -401,8 +402,8 @@
(when (kbd/enter? event) (when (kbd/enter? event)
(on-rename-clicked))) (on-rename-clicked)))
:id "teams-options-rename" :id "teams-options-rename"
:key "teams-options-rename" :unique-key "teams-options-rename"
:data-test "rename-team"} :data-test "rename-team"}
(tr "labels.rename")]) (tr "labels.rename")])
(cond (cond
@ -412,7 +413,7 @@
(when (kbd/enter? event) (when (kbd/enter? event)
(leave-and-close))) (leave-and-close)))
:id "teams-options-leave-team" :id "teams-options-leave-team"
:key "teams-options-leave-team"} :unique-key "teams-options-leave-team"}
(tr "dashboard.leave-team")] (tr "dashboard.leave-team")]
@ -422,8 +423,8 @@
(when (kbd/enter? event) (when (kbd/enter? event)
(on-leave-as-owner-clicked))) (on-leave-as-owner-clicked)))
:id "teams-options-leave-team" :id "teams-options-leave-team"
:key "teams-options-leave-team" :unique-key "teams-options-leave-team"
:data-test "leave-team"} :data-test "leave-team"}
(tr "dashboard.leave-team")] (tr "dashboard.leave-team")]
(> (count members) 1) (> (count members) 1)
@ -432,7 +433,7 @@
(when (kbd/enter? event) (when (kbd/enter? event)
(on-leave-clicked))) (on-leave-clicked)))
:id "teams-options-leave-team" :id "teams-options-leave-team"
:key "teams-options-leave-team"} :unique-key "teams-options-leave-team"}
(tr "dashboard.leave-team")]) (tr "dashboard.leave-team")])
@ -442,9 +443,9 @@
(when (kbd/enter? event) (when (kbd/enter? event)
(on-delete-clicked))) (on-delete-clicked)))
:id "teams-options-delete-team" :id "teams-options-delete-team"
:key "teams-options-delete-team" :unique-key "teams-options-delete-team"
:klass "warning" :klass "warning"
:data-test "delete-team"} :data-test "delete-team"}
(tr "dashboard.delete-team")])])) (tr "dashboard.delete-team")])]))

View file

@ -93,13 +93,13 @@
:space-between i/align-content-row-between :space-between i/align-content-row-between
:stretch nil)) :stretch nil))
(case val (case val
:start i/align-content-row-start :start i/align-content-row-start
:end i/align-content-row-end :end i/align-content-row-end
:center i/align-content-row-center :center i/align-content-row-center
:space-around i/align-content-row-around :space-around i/align-content-row-around
:space-between i/align-content-row-between :space-between i/align-content-row-between
:stretch nil) :stretch nil)
:align-self :align-self
(if is-col? (if is-col?
@ -275,8 +275,8 @@
{:placeholder "--" {:placeholder "--"
:on-change (partial on-change :simple :p1) :on-change (partial on-change :simple :p1)
:on-focus #(do :on-focus #(do
(dom/select-target %) (dom/select-target %)
(select-paddings true false true false)) (select-paddings true false true false))
:value p1}]] :value p1}]]
[:div.padding-item.tooltip.tooltip-bottom-left [:div.padding-item.tooltip.tooltip-bottom-left
@ -354,7 +354,7 @@
i/auto-gap] i/auto-gap]
[:> numeric-input {:no-validate true [:> numeric-input {:no-validate true
:placeholder "--" :placeholder "--"
:on-focus (fn [event] :on-focus (fn [event]
(select-gap :row-gap) (select-gap :row-gap)
(reset! gap-selected? :row-gap) (reset! gap-selected? :row-gap)
(dom/select-target event)) (dom/select-target event))
@ -638,12 +638,12 @@
(if (and (not multiple) (:layout values)) (if (and (not multiple) (:layout values))
[:div.title-actions [:div.title-actions
#_[:div.layout-btns #_[:div.layout-btns
[:button {:on-click set-flex [:button {:on-click set-flex
:class (dom/classnames :class (dom/classnames
:active (= :flex layout-type))} "Flex"] :active (= :flex layout-type))} "Flex"]
[:button {:on-click set-grid [:button {:on-click set-grid
:class (dom/classnames :class (dom/classnames
:active (= :grid layout-type))} "Grid"]] :active (= :grid layout-type))} "Grid"]]
[:button.remove-layout {:on-click on-remove-layout} i/minus]] [:button.remove-layout {:on-click on-remove-layout} i/minus]]
[:button.add-page {:on-click #(on-add-layout :flex)} i/close])]] [:button.add-page {:on-click #(on-add-layout :flex)} i/close])]]