Jeroen Thora
Jeroen Thora
@hersing Thanks for the PR! Can you add a test to cover this new behavior?
Oh sorry I've overlooked the check in the tests. I thought it was only a test to see if it is chainable!
https://github.com/Kunstmaan/KunstmaanBundlesCMS/releases/tag/7.1.2 was released!
@goodforenergy yes indeed that is an known issue, but we are going to fix that soon! See #1532, it's currently planned for 1.6 but I'm thinking about moving it forward...
@asieira or @brunocascio can you provide a jsfiddle with a possible usecase? I'm going to implement this method but want some usecase to test it!
Currently not planned in detail, but we should begin with planning soon! Will ping you when the roadmap is ready!
@Seldaek Any chance this can be looked at? Thanks in advance!
Thank you @nunoplopes
The argument https://github.com/symfony-cmf/core-bundle/pull/292/files#diff-1321e425c72f090b283c6b80c2dc411e7e1d60b47f144ed1b5bf11e04177386fR16 can also be removed, as it is not used anymore. Should I do it in this PR or is that consired a bc break?
Yeah I would choose to just change the translations. But maybe for a transition periode keep something like "X (Twitter)"?