innohub/tatort
7
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

f047_Edit-der-Namen #15

Closed
jared wants to merge 11 commits from f047_Edit-der-Namen into development
pull from: f047_Edit-der-Namen
merge into: innohub:development
Conversation 1 Commits 11 Files Changed 18 +3 -15
3 changed files with 3 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 631ce77f7d - Show all commits

10
src/lib/components/EditableItem.svelte
View File

@@ -20,18 +20,13 @@
let localName = $state(currentName); let localName = $state(currentName);
mina marked this conversation as resolved Outdated
jared commented 2025-06-30 07:08:51 +02:00
Outdated
Review
Copy Link

Bitte Console.log entfernen.

Bitte Console.log entfernen.
let wasCancelled = $state(false); let wasCancelled = $state(false);
// Automatisch berechneter Fehler
let error: string = $derived(validateName(localName)); let error: string = $derived(validateName(localName));
// Manuell steuerbarer Fehlerstatus
let manualError = $state(''); let manualError = $state('');
mina marked this conversation as resolved Outdated
jared commented 2025-06-30 07:10:56 +02:00
Outdated
Review
Copy Link

Typo editSart to editStart

Typo editSart to editS**t**art
console.log('EditableItem: Beginn', names, editedName);
let isEditing = $state(false); let isEditing = $state(false);
let inputRef: HTMLInputElement; let inputRef: HTMLInputElement;
// Validierungsfunktion
function validateName(name: string) { function validateName(name: string) {
jared marked this conversation as resolved Outdated
jared commented 2025-06-30 07:15:16 +02:00
Outdated
Review
Copy Link

The signature '(): EventDispatcher<{ editSart: {}; save: {}; delete: {}; cancel: void; }>' of 'createEventDispatcher' is deprecated.

Mittlerweile wird ein Event mittels Callback function dispatched. Gucke ggf. noch einmal in die Docs.

Zum Beispiel:

<script type="ts"> const props: { onClick(): void; onExplode(name: string, age: number): void; } = $props(); </script>

<MyComponent
onClick={() => alert('clicked')}
onExplode={(name, age) => alert(name + ' ' + age)}
/>

The signature '(): EventDispatcher<{ editSart: {}; save: {}; delete: {}; cancel: void; }>' of 'createEventDispatcher' is deprecated. Mittlerweile wird ein Event mittels Callback function dispatched. Gucke ggf. noch einmal in die Docs. Zum Beispiel: <script type="ts"> const props: { onClick(): void; onExplode(name: string, age: number): void; } = $props(); </script> <MyComponent onClick={() => alert('clicked')} onExplode={(name, age) => alert(name + ' ' + age)} />
mina commented 2025-06-30 16:38:51 +02:00
Outdated
Review
Copy Link

Ich habe es angepasst, es musste einiges verändert werden, bitte schaue es dir nochmal an.

Ich habe es angepasst, es musste einiges verändert werden, bitte schaue es dir nochmal an.
const trimmed = name.trim(); const trimmed = name.trim();
jared marked this conversation as resolved Outdated
jared commented 2025-06-30 07:23:25 +02:00
Outdated
Review
Copy Link

Villt die Variable noch besser bennennen. InternalValue oder oldValue, kann halt alles sein...

Villt die Variable noch besser bennennen. InternalValue oder oldValue, kann halt alles sein...
mina commented 2025-06-30 16:42:22 +02:00
Outdated
Review
Copy Link

Allgemein, weil es eine Komponente ist, die vielseitig einsetzbar. Es geht darum quasi den Startwert? zu speichern, für den Abbruch und den aktuellen Wert, um ein editieren in Gang zu bringen. Man könnte es auch cancelValue und EditValue nennen? Vielleicht besser?

Allgemein, weil es eine Komponente ist, die vielseitig einsetzbar. Es geht darum quasi den Startwert? zu speichern, für den Abbruch und den aktuellen Wert, um ein editieren in Gang zu bringen. Man könnte es auch cancelValue und EditValue nennen? Vielleicht besser?
jared commented 2025-07-21 14:04:20 +02:00
Outdated
Review
Copy Link

Naja, der Begriff Value bringt halt nicht, weil das quasi immer ein value ist. Daher so explizit bennen wie es geht

Naja, der Begriff Value bringt halt nicht, weil das quasi immer ein value ist. Daher so explizit bennen wie es geht
@@ -48,18 +43,16 @@
return ''; return '';
} }
// Speichern, wenn gültig und zurück an Eltern
function commitIfValid() { function commitIfValid() {
if (!error && !wasCancelled && localName != currentName) { if (!error && !wasCancelled && localName != currentName) {
editedName = localName.trim(); editedName = localName.trim();
onSave(editedName, currentName); //Eltern benachrichtigen onSave(editedName, currentName);
} else { } else {
localName = currentName; localName = currentName;
resetEdit(); resetEdit();
} }
} }
// Abbrechen – Eingabe zurücksetzen
function resetEdit() { function resetEdit() {
wasCancelled = false; wasCancelled = false;
manualError = ''; manualError = '';
@@ -67,7 +60,6 @@
isEditing = false; isEditing = false;
} }
// Tastatursteuerung
function handleKeydown(event: KeyboardEvent) { function handleKeydown(event: KeyboardEvent) {
if (event.key === 'Enter') { if (event.key === 'Enter') {
event.preventDefault(); event.preventDefault();

2
src/routes/(angemeldet)/list/+page.svelte
View File

@@ -13,7 +13,6 @@
const caseList: ListItem[] = $state(data.caseList); const caseList: ListItem[] = $state(data.caseList);
//Variabeln für EditableItem
let names: string[] = $state(caseList.map((l) => l.name)); let names: string[] = $state(caseList.map((l) => l.name));
let editedName: string = $state(''); let editedName: string = $state('');
let currentName: string = $state(''); let currentName: string = $state('');
@@ -49,7 +48,6 @@
} }
async function handleUpdateCase(newName: string, oldName: string) { async function handleUpdateCase(newName: string, oldName: string) {
console.log('Eltern, speichern erfolgreich', newName, oldName);
try { try {
const res = await fetch(`/api/list/${oldName}`, { const res = await fetch(`/api/list/${oldName}`, {
method: 'PUT', method: 'PUT',

4
src/routes/(angemeldet)/list/+page.ts
View File

@@ -1,11 +1,10 @@
export async function load({ fetch, url }) { export async function load({ fetch }) {
//const caseToken = url.searchParams.get('token'); //const caseToken = url.searchParams.get('token');
const adminRes = await fetch(`/api/user`) const adminRes = await fetch(`/api/user`)
const user = await adminRes.json() const user = await adminRes.json()
console.log("load Vorgang; ", user);
const res = await fetch(`/api/list`, { const res = await fetch(`/api/list`, {
/* headers: { /* headers: {
Authorization: `Bearer ${caseToken}` Authorization: `Bearer ${caseToken}`
@@ -15,7 +14,6 @@ console.log("load Vorgang; ", user);
const data = await res.json(); const data = await res.json();
const caseList = data.caseList const caseList = data.caseList
console.log("load Vorgang; ", caseList, user);
return { return {
caseList, caseList,