Skip to content

Commit 25d4015

Browse files
authored
Fix Safari bug where + Add another step button wasn't hidden properly (#5897)
* Fix Safari bug where `+ Add another step` button wasn't hidden properly * Increase spacing between add step button and conversion rate when both are shown
1 parent a9050ab commit 25d4015

File tree

1 file changed

+8
-9
lines changed
  • extra/lib/plausible_web/live/funnel_settings

1 file changed

+8
-9
lines changed

extra/lib/plausible_web/live/funnel_settings/form.ex

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ defmodule PlausibleWeb.Live.FunnelSettings.Form do
8484
Funnel steps
8585
</.label>
8686
87-
<div :for={step_idx <- @step_ids} class="flex mb-3 mt-3">
87+
<div :for={step_idx <- @step_ids} class="flex my-3">
8888
<div class="w-2/5 flex-1">
8989
<.live_component
9090
selected={find_preselected(@funnel, @funnel_modified?, step_idx)}
@@ -117,13 +117,12 @@ defmodule PlausibleWeb.Live.FunnelSettings.Form do
117117
</div>
118118
</div>
119119
120-
<.add_step_button :if={
121-
length(@step_ids) < Funnel.max_steps() and
122-
map_size(@selections_made) < length(@goals)
123-
} />
124-
125-
<div class="mt-6">
126-
<p id="funnel-eval" class="text-gray-500 dark:text-gray-400 text-sm mt-2 mb-2">
120+
<div class="flex flex-col gap-y-4 mt-6">
121+
<.add_step_button :if={
122+
length(@step_ids) < Funnel.max_steps() and
123+
map_size(@selections_made) < length(@goals)
124+
} />
125+
<p id="funnel-eval" class="text-gray-800 dark:text-gray-200 text-sm">
127126
<%= if @evaluation_result do %>
128127
Last month conversion rate: <strong><%= List.last(@evaluation_result.steps).conversion_rate %></strong>%
129128
<% end %>
@@ -179,7 +178,7 @@ defmodule PlausibleWeb.Live.FunnelSettings.Form do
179178

180179
def add_step_button(assigns) do
181180
~H"""
182-
<a class="underline text-indigo-500 text-sm cursor-pointer mt-6" phx-click="add-step">
181+
<a class="text-indigo-500 text-sm font-medium cursor-pointer" phx-click="add-step">
183182
+ Add another step
184183
</a>
185184
"""

0 commit comments

Comments
 (0)