Skip to content

Yform Value hinzugefügt#52

Merged
skerbis merged 32 commits intomasterfrom
yform
Nov 6, 2024
Merged

Yform Value hinzugefügt#52
skerbis merged 32 commits intomasterfrom
yform

Conversation

@skerbis
Copy link
Copy Markdown
Member

@skerbis skerbis commented Oct 29, 2024

Croppen nun auch mit YFORM. As simple as possible. Aktuell ohne Controls ...

bildschirmfoto_2024-10-29_um_21 25 15_720

@skerbis
Copy link
Copy Markdown
Member Author

skerbis commented Oct 29, 2024

Bildschirmfoto 2024-10-29 um 16 58 21

@skerbis skerbis requested a review from alexwenz October 29, 2024 16:02
Copy link
Copy Markdown

@AWqxKAWERbXo AWqxKAWERbXo left a comment

Choose a reason for hiding this comment

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

M.E.

  • Feld sollte be_media_crop heißen, da es nur fürs Backend ist
  • Media-Kategorie-Select statt id
  • crop/preview width und height als ganzzahliges positives Integer-Felder
  • Zeile 104: Warnung nicht übersetzt
  • Zeille 28: $_POST verwendet anstelle von rex_post()
  • Umsetzung erfordert yform-Page, sollte aber auch auf Addon-Pages funktionieren (neues, events, ...)

@skerbis
Copy link
Copy Markdown
Member Author

skerbis commented Oct 30, 2024

Feld sollte be_media_crop heißen, da es nur fürs Backend ist

Nein , es kann auch im FE verwendet werden

document.addEventListener('DOMContentLoaded', initMediaCrops);

Umsetzung erfordert yform-Page, sollte aber auch auf Addon-Pages funktionieren (neues, events, ...)

mal schauen , können auch einen anderen page Part nehmen

@skerbis
Copy link
Copy Markdown
Member Author

skerbis commented Oct 30, 2024

Media-Kategorie-Select statt id

Keine Ahnung wie man das hier reinfummelt

@skerbis
Copy link
Copy Markdown
Member Author

skerbis commented Oct 30, 2024

die preview akzeptiert auch % vh usw.

@skerbis
Copy link
Copy Markdown
Member Author

skerbis commented Oct 30, 2024

@alxndr-w
Um nicht unnötig bei jedem AddOn das YForm verwendet die Assets zu laden, müssten wir uns was einfallen lassen.
Aktuell würde ich es so belassen.
Man kann die Assets ja auch aus dem jeweiligen AddOn heraus laden. Vielleicht könnten wir eine Methode dafür bauen.

@AWqxKAWERbXo
Copy link
Copy Markdown

Was ist die Aufgabe des Parameters "Preview"?

Für die Media Category, pack choice rein mit SQL auf rex_media_category

@skerbis
Copy link
Copy Markdown
Member Author

skerbis commented Oct 30, 2024

Für die Media Category, pack choice rein mit SQL auf rex_media_category

Gerne übernehmen

Preview?

@AWqxKAWERbXo
Copy link
Copy Markdown

Preview?

@skerbis
Copy link
Copy Markdown
Member Author

skerbis commented Oct 30, 2024

du hast nach preview gefragt 😜

@skerbis skerbis requested a review from AWqxKAWERbXo October 30, 2024 15:43
@AWqxKAWERbXo
Copy link
Copy Markdown

Ja und immer noch keine Antwort.

Was ist die Aufgabe des Parameters "Preview"?

Deutlicher: Was macht preview_width und preview_height? Wo? Wie ist das gedacht? Ich verstehe nichts darunter, außer irgendwas mit Vorschau? Kann man 2 Bildausschnitte wählen, ein zugeschnittenes und eins für eine Vorschau?

@skerbis
Copy link
Copy Markdown
Member Author

skerbis commented Oct 30, 2024

Es gibt ne Bühne
und es gibt das Crop

Bildschirmfoto 2024-10-30 um 17 36 19

@AWqxKAWERbXo
Copy link
Copy Markdown

Ich raff es nicht. Ich würde nicht gern wissen, was es gibt, sondern wozu.

@skerbis
Copy link
Copy Markdown
Member Author

skerbis commented Nov 2, 2024

Wozu? Um es so zu gestalten wie man es haben will

Bildschirmfoto 2024-11-02 um 17 17 07

@skerbis skerbis merged commit 67adbe2 into master Nov 6, 2024
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.

2 participants