Fixing some issues found with back button/close icon, making sure it is consistent and does what a user would expect.

This commit is contained in:
mike12345567 2021-10-19 15:09:38 +01:00
parent 1e35d507c6
commit 3c2941536a
2 changed files with 17 additions and 20 deletions

View File

@ -20,6 +20,7 @@
import TemplateList from "./TemplateList.svelte" import TemplateList from "./TemplateList.svelte"
export let template export let template
export let inline
const values = writable({ name: null }) const values = writable({ name: null })
const errors = writable({}) const errors = writable({})
@ -39,11 +40,10 @@
let submitting = false let submitting = false
let valid = false let valid = false
let wasShowingTemplates let initialTemplateInfo = template?.fromFile || template?.key
$: checkValidity($values, validator) $: checkValidity($values, validator)
$: showTemplateSelection = $: showTemplateSelection = !template && !initialTemplateInfo
!template || (!template?.fromFile && !template?.key)
onMount(async () => { onMount(async () => {
await hostingStore.actions.fetchDeployedApps() await hostingStore.actions.fetchDeployedApps()
@ -77,10 +77,12 @@
} }
async function createNewApp() { async function createNewApp() {
const letTemplateToUse =
Object.keys(template).length === 0 ? null : template
submitting = true submitting = true
// Check a template exists if we are important // Check a template exists if we are important
if (template?.fromFile && !$values.file) { if (letTemplateToUse?.fromFile && !$values.file) {
$errors.file = "Please choose a file to import" $errors.file = "Please choose a file to import"
valid = false valid = false
submitting = false submitting = false
@ -91,10 +93,10 @@
// Create form data to create app // Create form data to create app
let data = new FormData() let data = new FormData()
data.append("name", $values.name.trim()) data.append("name", $values.name.trim())
data.append("useTemplate", template != null) data.append("useTemplate", letTemplateToUse != null)
if (template) { if (letTemplateToUse) {
data.append("templateName", template.name) data.append("templateName", letTemplateToUse.name)
data.append("templateKey", template.key) data.append("templateKey", letTemplateToUse.key)
data.append("templateFile", $values.file) data.append("templateFile", $values.file)
} }
@ -108,7 +110,7 @@
analytics.captureEvent(Events.APP.CREATED, { analytics.captureEvent(Events.APP.CREATED, {
name: $values.name, name: $values.name,
appId: appJson.instance._id, appId: appJson.instance._id,
template, letTemplateToUse,
}) })
// Select Correct Application/DB in prep for creating user // Select Correct Application/DB in prep for creating user
@ -146,8 +148,7 @@
showConfirmButton={false} showConfirmButton={false}
size="L" size="L"
onConfirm={() => { onConfirm={() => {
showTemplateSelection = false template = {}
wasShowingTemplates = true
return false return false
}} }}
showCancelButton={false} showCancelButton={false}
@ -156,14 +157,9 @@
<TemplateList <TemplateList
onSelect={(selected, { useImport } = {}) => { onSelect={(selected, { useImport } = {}) => {
if (!selected) { if (!selected) {
if (useImport) { template = useImport ? { fromFile: true } : {}
template = { fromFile: true }
}
showTemplateSelection = false
wasShowingTemplates = true
return return
} }
template = selected template = selected
}} }}
/> />
@ -173,8 +169,9 @@
title={template?.fromFile ? "Import app" : "Create app"} title={template?.fromFile ? "Import app" : "Create app"}
confirmText={template?.fromFile ? "Import app" : "Create app"} confirmText={template?.fromFile ? "Import app" : "Create app"}
onConfirm={createNewApp} onConfirm={createNewApp}
onCancel={wasShowingTemplates ? () => (template = null) : null} onCancel={inline ? () => (template = null) : null}
cancelText={wasShowingTemplates ? "Back" : undefined} cancelText={inline ? "Back" : undefined}
showCloseIcon={!inline}
disabled={!valid} disabled={!valid}
> >
{#if template?.fromFile} {#if template?.fromFile}

View File

@ -268,7 +268,7 @@
{#if !enrichedApps.length && !creatingApp && loaded} {#if !enrichedApps.length && !creatingApp && loaded}
<div class="empty-wrapper"> <div class="empty-wrapper">
<Modal inline> <Modal inline>
<CreateAppModal {template} /> <CreateAppModal {template} inline={true} />
</Modal> </Modal>
</div> </div>
{/if} {/if}