supergarv, German
@supergarv@phpc.social avatar

Sooooo, wer mag denn meine erste Core contribution (saftige +3/-1 Zeilen) mal reviewen und zusätzlich mit mir über Anpassungen am ImageViewHelper zum nicht-croppen von SVGs sprechen? 😇

s2b,
@s2b@norden.social avatar

@supergarv Hast du schon gepusht? Wir können uns gerne mal dazu austauschen.

supergarv,
@supergarv@phpc.social avatar

@s2b Eigentlich schon, der Change liegt ja in review vor? Hab's aber noch nie gemacht und würde mich sehr über Unterstützung/Feedback freuen 😊

s2b,
@s2b@norden.social avatar

@supergarv Hast du nen Link? Habs gerade spontan nicht gefunden.

supergarv,
@supergarv@phpc.social avatar

@s2b Ach ich Honk hab gar keine Link gepostet 😂

https://review.typo3.org/c/Packages/TYPO3.CMS/+/80595

s2b,
@s2b@norden.social avatar

@supergarv Klingt für mich inhaltlich nicht abwegig. Also gerne vervollständigen und WIP wegnehmen, dann gibt‘s bestimmt zeitnah nen Review davon. :)

supergarv,
@supergarv@phpc.social avatar

@s2b Was müsste ich denn vervollständigen? Ein RST für "feature" beschreibung erstellen, und dann aber separat ein doku github PR, erst wenn es gemergt werden sollte...?

(Dachte das WIP braucht man um überhaupt diskutiert zu werden; das mach ich dann per commit amend raus?)

s2b,
@s2b@norden.social avatar

@supergarv Ich würde ein RST ergänzen und WIP rausnehmen per amend. Ab dem Punkt findet dann der Review statt. Doku klingt erst mal logisch so, den Fall hatte ich persönlich aber noch nicht. :)

supergarv,
@supergarv@phpc.social avatar

@s2b OK - man müsste ja in der Doku zum input-TCA ergänzen.

Super. Danke für deine Hilfe, werde das nach Family-Wochenende amenden! 👍

P.S: Statt WIP eher Feature oder Task? So ein wirkliches feature ist es ja nicht, eher ein task als "freischaltung"....?

s2b,
@s2b@norden.social avatar

@supergarv Ja, klingt für mich nach Task. 👍🏼

supergarv,
@supergarv@phpc.social avatar

@s2b Oooh wait, Du meinst die SVG Viewhelper Sache? Hatte da heute im slack coredev einiges geschrieben, und habe 1-2 Ansätze die ich nächste Woche gern probieren (&diskutieren) würde, ja!

s2b,
@s2b@norden.social avatar

@supergarv Das können wir gerne machen. Meld dich einfach per Slack, wenn du was gebastelt hast oder Diskussionsbedarf hast.

  • All
  • Subscribed
  • Moderated
  • Favorites
  • php
  • ngwrru68w68
  • rosin
  • GTA5RPClips
  • osvaldo12
  • love
  • Youngstown
  • slotface
  • khanakhh
  • everett
  • kavyap
  • mdbf
  • DreamBathrooms
  • thenastyranch
  • magazineikmin
  • megavids
  • InstantRegret
  • normalnudes
  • tacticalgear
  • cubers
  • ethstaker
  • modclub
  • cisconetworking
  • Durango
  • anitta
  • Leos
  • tester
  • provamag3
  • JUstTest
  • All magazines