The media removal is not currently added to history. That's fixed in this PR. Also the logic to remove media span is moved to the AztecText so that it's reusable. I've also fixed the coroutine usage and made the PlaceholderManager a coroutine scope.
Test
Set up PlaceholderManager in the main activity with the ImageWithCaptionAdapter
Make the EXAMPLE contain a placeholder that you can identify
Add removeItem action to something you can control (for example the camera button)
In the removeItem identify the placeholder you want to remove
Run the app
Remove the placeholder
Notice it gets removed correctly
Review
@danilo04
Make sure strings will be translated:
[x] If there are new strings that have to be translated, I have added them to the client's strings.xml as a part of the integration PR.
Fix
The media removal is not currently added to history. That's fixed in this PR. Also the logic to remove media span is moved to the
AztecTextso that it's reusable. I've also fixed the coroutine usage and made thePlaceholderManagera coroutine scope.Test
PlaceholderManagerin the main activity with theImageWithCaptionAdapterEXAMPLEcontain aplaceholderthat you can identifyremoveItemaction to something you can control (for example the camera button)removeItemidentify the placeholder you want to removeReview
@danilo04
Make sure strings will be translated:
strings.xmlas a part of the integration PR.