diff --git a/internal/server/alpine_browser_test.go b/internal/server/alpine_browser_test.go index 7e08787..a884e20 100644 --- a/internal/server/alpine_browser_test.go +++ b/internal/server/alpine_browser_test.go @@ -484,9 +484,13 @@ func checkEntrypointEdit( ) { t.Helper() + // Cancel and Save are found inside the edit form, since the add + // target form has buttons of the same names. const ( editForm = `form[action$="/edit"]` input = editForm + ` input[name="description"]` + cancelEdit = `//form[contains(@action, "/edit")]/button[text()="Cancel"]` + saveEdit = `//form[contains(@action, "/edit")]/button[text()="Save"]` description = `//span[text()="Entrypoint"]` edit = `//button[text()="Edit"]` ) @@ -507,7 +511,7 @@ func checkEntrypointEdit( require.NoError(t, chromedp.Run( ctx, chromedp.SendKeys(input, "draft", chromedp.ByQuery), )) - click(ctx, t, `//button[text()="Cancel"]`) + click(ctx, t, cancelEdit) assert.True(t, hidden(ctx, editForm), "clicking Cancel does not hide the edit form") assert.True(t, shown(ctx, description), @@ -549,7 +553,7 @@ func checkEntrypointEdit( require.NoError(t, chromedp.Run( ctx, chromedp.SendKeys(input, "Billing sender", chromedp.ByQuery), )) - click(ctx, t, `//button[text()="Save"]`) + click(ctx, t, saveEdit) assert.True(t, shown(ctx, `//span[text()="Billing sender"]`), "saving the edit form does not change the description") diff --git a/templates/source_detail.html b/templates/source_detail.html index 321357f..44c9171 100644 --- a/templates/source_detail.html +++ b/templates/source_detail.html @@ -133,11 +133,13 @@ and the hidden type field submitted with them, exist only while that type is chosen. A refused submission comes back open on its type, with the values entered; - Cancel empties the form. --> + Cancel empties the form. The type choice's p-2, narrower + than an input's own padding, keeps it, Next and Cancel on + one row on a 360px-wide phone. -->
-