From 1014912f76d48473498342456b2b879e3498dc71 Mon Sep 17 00:00:00 2001 From: Torsten Dittmann Date: Fri, 1 Mar 2024 00:25:11 +0100 Subject: [PATCH 01/10] fix: tooltip arrows --- src/lib/components/drop.svelte | 70 ++++++++++++------- src/lib/elements/forms/inputDomain.svelte | 5 +- src/lib/elements/forms/inputEmail.svelte | 5 +- src/lib/elements/forms/inputFile.svelte | 5 +- src/lib/elements/forms/inputPassword.svelte | 5 +- src/lib/elements/forms/inputPhone.svelte | 5 +- src/lib/elements/forms/inputText.svelte | 10 +-- .../providers/settingsFormList.svelte | 6 -- 8 files changed, 61 insertions(+), 50 deletions(-) diff --git a/src/lib/components/drop.svelte b/src/lib/components/drop.svelte index f61ff6a24..22253de50 100644 --- a/src/lib/components/drop.svelte +++ b/src/lib/components/drop.svelte @@ -14,6 +14,8 @@ export let fullWidth = false; export let fixed = false; export let display = 'block'; + export let arrowSize = 10; + export let isPopover = false; const dispatch = createEventDispatcher<{ blur: undefined; @@ -32,13 +34,14 @@ { name: 'arrow', options: { - element: arrow + element: arrow, + padding: arrowSize * 2 } }, { name: 'offset', options: { - offset: [0, noArrow ? 0 : 6] + offset: [-arrowSize, noArrow ? 0 : arrowSize - 2] } }, { @@ -111,10 +114,15 @@
-
+
{#if show} {/if} @@ -122,36 +130,50 @@ diff --git a/src/lib/elements/forms/inputDomain.svelte b/src/lib/elements/forms/inputDomain.svelte index 26f0b972c..007c12e97 100644 --- a/src/lib/elements/forms/inputDomain.svelte +++ b/src/lib/elements/forms/inputDomain.svelte @@ -14,7 +14,6 @@ export let autofocus = false; export let autocomplete = false; export let maxlength: number = null; - export let isPopoverDefined = true; // https://www.geeksforgeeks.org/how-to-validate-a-domain-name-using-regular-expression/ const pattern = String.raw`(?!-)[A-Za-z0-9\-]+([\-\.]{1}[a-z0-9]+)*\.[A-Za-z]{2,18}`; @@ -51,8 +50,8 @@
diff --git a/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte b/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte index 92baa0d30..8d167bbdf 100644 --- a/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte +++ b/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte @@ -84,7 +84,7 @@ placeholder={input.placeholder} required={!input.optional} {popover} - popoverProps={getPopoverProps(input)} + {popoverProps} bind:value={params[input.name]}> {:else if input.type === 'domain'} From 997b1ec4ee36733aee1fabbc0fad14ad20987a1b Mon Sep 17 00:00:00 2001 From: Torsten Dittmann Date: Fri, 1 Mar 2024 02:31:24 +0100 Subject: [PATCH 06/10] fix: popover --- src/lib/elements/forms/inputDomain.svelte | 8 ++-- src/lib/elements/forms/inputEmail.svelte | 8 ++-- src/lib/elements/forms/inputFile.svelte | 8 ++-- src/lib/elements/forms/inputPassword.svelte | 8 ++-- src/lib/elements/forms/inputPhone.svelte | 8 ++-- src/lib/elements/forms/inputText.svelte | 8 ++-- .../messaging/providers/popoverContent.svelte | 10 ++--- .../providers/settingsFormList.svelte | 43 +++++++++++-------- 8 files changed, 59 insertions(+), 42 deletions(-) diff --git a/src/lib/elements/forms/inputDomain.svelte b/src/lib/elements/forms/inputDomain.svelte index 007c12e97..db798dea7 100644 --- a/src/lib/elements/forms/inputDomain.svelte +++ b/src/lib/elements/forms/inputDomain.svelte @@ -1,5 +1,5 @@
@@ -10,11 +10,11 @@

{@html line}

{/each} - {#if popoverImage} + {#if image} + darkSrc={image.src.dark} + lightSrc={image.src.light} + alt={image.alt} /> {/if}
diff --git a/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte b/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte index e0c330642..92baa0d30 100644 --- a/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte +++ b/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte @@ -40,21 +40,31 @@ | FCMProviderParams | APNSProviderParams >; + + function getPopoverProps(input: ProviderInput) { + if (!input.popover) { + return {}; + } + return { + lines: input.popover, + image: input.popoverImage + }; + } {#each inputs as input} - {@const popoverImage = input.popoverImage} + {@const popover = input.popover ? PopoverContent : null} + {@const popoverProps = getPopoverProps(input)} {#if input.type === 'text'} - - - {:else if input.type === 'password'} - - - {:else if input.type === 'email'} - - - {:else if input.type === 'domain'} - - - {:else if input.type === 'phone'} - - - {:else if input.type === 'file'} - - - {:else if input.type === 'switch'} From 3e1cecc40ae41457ac866ef038157b0a395a16dc Mon Sep 17 00:00:00 2001 From: Torsten Dittmann Date: Fri, 1 Mar 2024 02:33:35 +0100 Subject: [PATCH 07/10] fix: use template const for popover props with email input --- .../messaging/providers/popoverContent.svelte | 5 +---- .../messaging/providers/settingsFormList.svelte | 2 +- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/src/routes/console/project-[project]/messaging/providers/popoverContent.svelte b/src/routes/console/project-[project]/messaging/providers/popoverContent.svelte index 378e9424d..7b9a1e723 100644 --- a/src/routes/console/project-[project]/messaging/providers/popoverContent.svelte +++ b/src/routes/console/project-[project]/messaging/providers/popoverContent.svelte @@ -11,10 +11,7 @@ {/each} {#if image} - + {/if}
diff --git a/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte b/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte index 92baa0d30..8d167bbdf 100644 --- a/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte +++ b/src/routes/console/project-[project]/messaging/providers/settingsFormList.svelte @@ -84,7 +84,7 @@ placeholder={input.placeholder} required={!input.optional} {popover} - popoverProps={getPopoverProps(input)} + {popoverProps} bind:value={params[input.name]}> {:else if input.type === 'domain'} From 8f385564085ef7bc3c2778452733f534a29b2cde Mon Sep 17 00:00:00 2001 From: Torsten Dittmann Date: Fri, 1 Mar 2024 13:45:46 +0100 Subject: [PATCH 08/10] fix: popover card radius --- src/lib/elements/forms/inputDomain.svelte | 4 +++- src/lib/elements/forms/inputEmail.svelte | 4 +++- src/lib/elements/forms/inputFile.svelte | 4 +++- src/lib/elements/forms/inputPassword.svelte | 4 +++- src/lib/elements/forms/inputPhone.svelte | 4 +++- src/lib/elements/forms/inputText.svelte | 1 + 6 files changed, 16 insertions(+), 5 deletions(-) diff --git a/src/lib/elements/forms/inputDomain.svelte b/src/lib/elements/forms/inputDomain.svelte index db798dea7..91d0e51b0 100644 --- a/src/lib/elements/forms/inputDomain.svelte +++ b/src/lib/elements/forms/inputDomain.svelte @@ -68,7 +68,9 @@
+ style:--card-border-radius="var(--border-radius-small)" + style:--p-card-padding=".75rem" + style:box-shadow="var(--shadow-large)">
diff --git a/src/lib/elements/forms/inputEmail.svelte b/src/lib/elements/forms/inputEmail.svelte index b925a1b62..2cd761e4a 100644 --- a/src/lib/elements/forms/inputEmail.svelte +++ b/src/lib/elements/forms/inputEmail.svelte @@ -77,7 +77,9 @@
+ style:--card-border-radius="var(--border-radius-small)" + style:--p-card-padding=".75rem" + style:box-shadow="var(--shadow-large)">
diff --git a/src/lib/elements/forms/inputFile.svelte b/src/lib/elements/forms/inputFile.svelte index d3d88c033..2de200818 100644 --- a/src/lib/elements/forms/inputFile.svelte +++ b/src/lib/elements/forms/inputFile.svelte @@ -113,7 +113,9 @@
+ style:--card-border-radius="var(--border-radius-small)" + style:--p-card-padding=".75rem" + style:box-shadow="var(--shadow-large)">
diff --git a/src/lib/elements/forms/inputPassword.svelte b/src/lib/elements/forms/inputPassword.svelte index 7834dad57..423f0db2b 100644 --- a/src/lib/elements/forms/inputPassword.svelte +++ b/src/lib/elements/forms/inputPassword.svelte @@ -68,7 +68,9 @@
+ style:--card-border-radius="var(--border-radius-small)" + style:--p-card-padding=".75rem" + style:box-shadow="var(--shadow-large)">
diff --git a/src/lib/elements/forms/inputPhone.svelte b/src/lib/elements/forms/inputPhone.svelte index 1a39f6400..23e7c1c81 100644 --- a/src/lib/elements/forms/inputPhone.svelte +++ b/src/lib/elements/forms/inputPhone.svelte @@ -67,7 +67,9 @@
+ style:--card-border-radius="var(--border-radius-small)" + style:--p-card-padding=".75rem" + style:box-shadow="var(--shadow-large)">
diff --git a/src/lib/elements/forms/inputText.svelte b/src/lib/elements/forms/inputText.svelte index b7b8de090..ac354e3f2 100644 --- a/src/lib/elements/forms/inputText.svelte +++ b/src/lib/elements/forms/inputText.svelte @@ -95,6 +95,7 @@
From 3699cb85db7bac2afd9d58e4a2f605afe43f9a13 Mon Sep 17 00:00:00 2001 From: Torsten Dittmann Date: Wed, 6 Mar 2024 11:38:02 +0100 Subject: [PATCH 09/10] fix: build --- .../messaging/providers/settingsFormInput.svelte | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/routes/console/project-[project]/messaging/providers/settingsFormInput.svelte b/src/routes/console/project-[project]/messaging/providers/settingsFormInput.svelte index 93062d19a..8b17b758c 100644 --- a/src/routes/console/project-[project]/messaging/providers/settingsFormInput.svelte +++ b/src/routes/console/project-[project]/messaging/providers/settingsFormInput.svelte @@ -41,7 +41,8 @@ >; export let fullWidth = false; - let popoverImage = input.popoverImage; + const popover = input.popover ? PopoverContent : null; + const popoverProps = getPopoverProps(input); function getPopoverProps(input: ProviderInput) { if (!input.popover) { @@ -54,8 +55,6 @@ } -{@const popover = input.popover ? PopoverContent : null} -{@const popoverProps = getPopoverProps(input)} {#if input.type === 'text'} Date: Wed, 6 Mar 2024 11:41:55 +0100 Subject: [PATCH 10/10] fix: formatting --- .../messaging/providers/settingsFormInput.svelte | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/routes/console/project-[project]/messaging/providers/settingsFormInput.svelte b/src/routes/console/project-[project]/messaging/providers/settingsFormInput.svelte index 8b17b758c..615e13353 100644 --- a/src/routes/console/project-[project]/messaging/providers/settingsFormInput.svelte +++ b/src/routes/console/project-[project]/messaging/providers/settingsFormInput.svelte @@ -65,7 +65,7 @@ {popover} {popoverProps} {fullWidth} - bind:value={params[input.name]}/> + bind:value={params[input.name]} /> {:else if input.type === 'password'} + bind:value={params[input.name]} /> {:else if input.type === 'email'} + bind:value={params[input.name]} /> {:else if input.type === 'domain'} + bind:value={params[input.name]} /> {:else if input.type === 'phone'} + bind:value={params[input.name]} /> {:else if input.type === 'file'} + bind:files={files[input.name]} /> {:else if input.type === 'switch'}