Fix unwanted rich text formatting when pasting

[imericxu]

What is it?

• Bugfix (user facing)
• Feature (user facing)
• Codebase improvement (dev facing)
• Meta improvement to the project (dev facing)

Description of the changes in your PR

• Use getSpans and removeSpan to remove text styling when pasting

Fixes the following issue(s)

• Fixes Text copied from HTML is rendered in search bar #5912

APK testing

On the website, the APK can be found by going to the "Checks" tab below the title and then on "artifacts" on the right.

Due diligence

• I read the contribution guidelines.

[TobiGr]

thank you. please add a comment in the code that it removes rich text formatting

[imericxu]

Since this issue seems to only occur when pasting, should I modify the solution to an override of the paste method so there are no unnecessary checks in the TextWatcher?

Edit: Nvm, looks like you’d need to override a private member or do a lot of extra stuff to achieve that unless I missed something.

[TobiGr]

thank you. Is there are reason for placing the code in afterTextChanged and not in onTextChanged?

[imericxu]

@TobiGr Unless I’m mistaken, only afterTextChanged(…) allows you to edit the text. Here’s the StackOverflow post where I got my info about TextWatcher methods: Link.

[TobiGr]

Thanks for the explanation!