- 🇬🇧United Kingdom catch
I think this is good background reading for the other issue, but let's explicitly postpone it on that one.
- 🇫🇷France pdureau Paris
Yes, extending existing Render Elements classes (as proposed in 📌 Slowly, very slowly start OOPifying the render system Needs review ) looks better than adding a Builder class for each Render Element as proposed in the current MR.
Also, when we will replace most (but not all 😉) of Render & Form elements by SDC, all the logic to remove will be contained in a single file.
- 🇫🇷France nod_ Lille
I spent quite some time working on making just 1 conversion, I agree with #153, this is not the way to go about it. 📌 Slowly, very slowly start OOPifying the render system Needs review is a much nicer solution.
- 🇨🇦Canada Charlie ChX Negyesi 🍁Canada
I tried #142 but the amount of methods you'd need to add is staggering and at the end of the day since people use arbitrary properties in render arrays you would need to support some sort of set/get anyways so I leaned heavily on that and so 📌 Slowly, very slowly start OOPifying the render system Needs review is born. If that issue gets accepted this issue might be outdated.