f004_CameraSteuerung-neu #11

Merged
jared merged 5 commits from f004_CameraSteuerung-neu into development 2025-06-25 09:14:08 +02:00
Owner

Ich möchte den Branch in development mergen

Ich möchte den Branch in development mergen
mina added 4 commits 2025-06-24 10:32:14 +02:00
mina requested review from Daniel 2025-06-24 10:32:15 +02:00
mina requested review from jared 2025-06-24 10:32:15 +02:00
mina requested review from titus 2025-06-24 10:32:15 +02:00
mina requested review from trachi93 2025-06-24 10:32:15 +02:00
jared requested changes 2025-06-24 15:58:43 +02:00
Dismissed
@@ -25,3 +25,3 @@
let zRotation = 0;
let modelViewer;
import type { ModelViewerElement } from '@google/model-viewer';
Owner

import statements oben im file platzieren

import statements oben im file platzieren
mina marked this conversation as resolved
@@ -29,3 +31,3 @@
$: style = `width: ${progress}%`;
const onProgress = ({ detail }) => {
const onProgress = ({ detail }: { detail: { totalProgress: number } }) => {
Owner

Funktionnamen könnte besser sein. Was bedeutet onProgress genau?

Funktionnamen könnte besser sein. Was bedeutet onProgress genau?
Author
Owner

Ist ist der Prozess vom Ladebalken, ein Event-trigger vom Model-viewer, habs jetzt unbenannt zu updateModelProgress, vielleicht erklärt es das besser

Ist ist der Prozess vom Ladebalken, ein Event-trigger vom Model-viewer, habs jetzt unbenannt zu updateModelProgress, vielleicht erklärt es das besser
jared marked this conversation as resolved
@@ -94,2 +102,4 @@
}}>AR Testen</button
>
</div>
Owner

Kann man den vorherigen Code nicht löschen, da hier nicht viel passiert und dieser auch schnell wieder eingefügt ist. Wenn wir den behalten dann nicht mit hidden verstecken, sondern auskommentieren

Kann man den vorherigen Code nicht löschen, da hier nicht viel passiert und dieser auch schnell wieder eingefügt ist. Wenn wir den behalten dann nicht mit hidden verstecken, sondern auskommentieren
Author
Owner

Okay, gebe dir recht. Habs den Abschnitt nun gelöscht. Zu gegebener Zeit kann man hier den Teil einfügen, habs mir lokal abgespeichert, zur besseren Findbarkeit

Okay, gebe dir recht. Habs den Abschnitt nun gelöscht. Zu gegebener Zeit kann man hier den Teil einfügen, habs mir lokal abgespeichert, zur besseren Findbarkeit
jared marked this conversation as resolved
@@ -202,4 +198,1 @@
/* .active-border {
border: 2px blue solid;
} */
</style>
Owner

Das finde ich gut, dass wir kein Custom css verwenden

Das finde ich gut, dass wir kein Custom css verwenden
mina marked this conversation as resolved
mina added 1 commit 2025-06-24 18:07:05 +02:00
jared approved these changes 2025-06-25 09:13:57 +02:00
jared merged commit b4c669226f into development 2025-06-25 09:14:08 +02:00
jared deleted branch f004_CameraSteuerung-neu 2025-06-25 09:14:15 +02:00
Sign in to join this conversation.
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: innohub/tatort#11