saket / Better-Link-Movement-Method

Attempts to improve how clickable links are detected, highlighted and handled in TextView
Apache License 2.0
780 stars 78 forks source link

Link highlighting is broken #51

Open TheBrokenRail opened 1 year ago

TheBrokenRail commented 1 year ago

Sometimes links highlight properly, but sometimes they have an extra highlight behind them.

https://github.com/saket/Better-Link-Movement-Method/assets/17478432/643abac4-8b5a-44ec-b2f8-69208051dd12

Here's a sample project that demonstrates this issue.

(Also, is this project actively maintained? I noticed some commits from 2021, but the last release was in 2018.)

saket commented 1 year ago

So weird, I haven't seen this before. Just to confirm, does this not happen with the standard LinkMovementMethod?

Also, is this project actively maintained? I noticed some commits from 2021, but the last release was in 2018.

It is indeed active, but there's not much to maintain these days.

TheBrokenRail commented 1 year ago

The "double-highlight" doesn't happen with the standard LinkMovementMethod. My guess is that for some reason, both the BetterLinkMovementMethod and LinkMovementMethod highlights are displaying on top of each other. Also, this is on Android 13.