Skip to content

Conversation

@alexrelux
Copy link
Contributor

Update p3 d types with reflector depth

add ceiling recessed cylinder
update round recessed
update using reflector depth
@alexrelux alexrelux self-assigned this Nov 5, 2024
@alexrelux alexrelux requested a review from Kurpanik November 5, 2024 16:41
update open luminaire
@Kurpanik
Copy link
Member

Kurpanik commented Nov 8, 2024

Die Überarbeitung weicht vom Stil der Seite ab. Alle Wörter, die Elementen in XML entsprechen, sind in der Doku besonders hervorgehoben. Wir sollten weiterhin einen einheitlichen Stil pflegen:

Bisher, richtig:
grafik

Jetzt
grafik

@Kurpanik
Copy link
Member

Kurpanik commented Nov 8, 2024

Das Bild passt nicht mehr zu den Parametern

Copy link
Member

@Kurpanik Kurpanik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Siehe Kommentare

update xml samples and images
@alexrelux alexrelux requested a review from Kurpanik November 11, 2024 12:59
fix xml
@alexrelux alexrelux marked this pull request as draft November 14, 2024 09:36
update style
@Kurpanik
Copy link
Member

@alexrelux Du arbeitest noch an dem PR? Er ist im Draft Modus

@alexrelux
Copy link
Contributor Author

Ja, das Team muss noch Änderungen umsetzen. Solange sich das verzögert, sollten wir die Veröffentlichung zurückhalten.

@Kurpanik
Copy link
Member

Alles klar

resolves #90
update parametric geo types
@alexrelux alexrelux requested a review from Kurpanik February 5, 2025 16:42
@alexrelux
Copy link
Contributor Author

@Kurpanik Hi Alex, so wurde das jetzt implementiert. Gruss Alex

@Kurpanik
Copy link
Member

Kurpanik commented Feb 6, 2025

@alexrelux Der PR ist noch im Draft. Sprich die Doku ist jetzt aber noch nicht fertig?

@alexrelux alexrelux marked this pull request as ready for review February 6, 2025 15:08
@alexrelux
Copy link
Contributor Author

ist fertig

@Kurpanik Kurpanik merged commit 8ca100c into develop Feb 10, 2025
1 check passed
@Kurpanik Kurpanik deleted the UpdateP3DTypesWithReflectorDepth branch February 10, 2025 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants