[SuperEditor][SuperReader] - Don't linkify values with multiple periods

[matthew-carroll]

I discovered that linkification in SuperEditor currently linkifies text with multiple periods, e.g., something....abc converts to a link.

Check for any other related error patterns, fix those patterns, and add tests to ensure those error patterns don't linkify.

[rutvik110]

This seems to be an issue arising from the linkify package url parsers which is identifying such patterns a valid url. I've filed issue for this here.

Do we intend to handle this on our side? or want to wait till this is fixed within the package? I'll file up a PR on the package with a fix but based on the activity on the package, it may take some time to have the fix merged in there.

[matthew-carroll]

We can wait for the linkify package to fix it, if that's something that's going to happen.

[quaaantumdev]

@matthew-carroll i had numerous isssues with the dart linkify implementation and therefore translated the linkifyjs implementation to dart (manually). I belive it's a way better implementation and I am happy to share it and I'd even contribute some updates in the future, i just don't want to maintain the pub.dev publishing and repo in general. It looks to me like you're pretty good with maintaining public packages, would you be interested in managing it?

I changed a few things compared to the javascript version to get rid of all global variables and introduce some type safety. I belive the unit test coverage is pretty good as well and it should even be pretty simple to upgrade stuff when the original javascript source is improved as the structure is pretty mich the same. I did remove some specialties such as handling DOM nodes as i belive that is not really relevant in dart/flutter.

I think the package could solve some issues here, and propably in the future.

Thanks for the your great work in general :)