There is an issue where there is a multiline text element before a placeholder. The custom method that was calculating line for offset doesn't take that into account. I've replaced it with the default getLineForOffset and it seems to work well now.
In addition to this I've also made some of the functions in the PlaceholderManager suspended to improve performance and allow async computation
Test
Enable placeholders and add a multiline text followed by a placeholder to the Example field
Run the app
Notice the placeholder is drawn 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
There is an issue where there is a multiline text element before a placeholder. The custom method that was calculating line for offset doesn't take that into account. I've replaced it with the default
getLineForOffset
and it seems to work well now.In addition to this I've also made some of the functions in the
PlaceholderManager
suspended to improve performance and allow async computationTest
Example
fieldReview
@danilo04
Make sure strings will be translated:
strings.xml
as a part of the integration PR.