Update NewPipeExtractor and properly linkify comments

[Stypox]

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

• Update the extractor to the latest commit, with breaking changes from Use Description object for comments text. NewPipeExtractor#987: now comments have a Description and not a String anymore
• Changes in TextLinkifier:
  • Move the method that determines how to linkify a description based on its type from DescriptionFragment to TextLinkifier
  • Rename methods in TextLinkifier to not contain the redundant "link" word already contained in "Linkifier"
  • Allow passing a callback to TextLinkifier methods to be notified when the text has been set to the TextView, needed to set ellipsis and movement method in comments
  • Since TextLinkifier only needs stream url and service id to generate popup-timestamps and hashtags, and since comments only have that information and not a full Info, now the TextLinkifier methods take those two parameters instead of an `Info.
• Rework the comments touch handling because it wouldn't work anymore since the links are now linkified with a different approach than before (I don't know why it stopped working)
  • Now calling getText() does not return a Spannable, but rather a Spanned (even though in TextLinkifier a SpannableBuilder is used). This means that the text selection (i.e. the highlight that appears when tapping on links) can't be manually set by us as done before in CommentTextOnTouchListener, unfortunately.
  • Note that long clicks still don't work, I'll leave it to a future PR. The same logic as LongPressLinkMovementMethod needs to be implemented in CommentTextOnTouchListener.
• Rework comment ellipsis handling, since it added ... by cutting away too many characters in case of links, and also because it made the text view size smaller on some cases for some reason (?)
  • Now links are not clickable when the comment is ellipsized (this should have been the case before, too, but it used to be buggy)
  • Only the last 2 characters are chopped off, and are only chopped off if there would not already be space for the ellipsis

Fixes the following issue(s)

It definitely fixes some problems with comments, but I couldn't find any related issue.

APK testing

The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR.

Due diligence

• I read the contribution guidelines.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

[TobiGr]

Code looks good. I tested briefly and did not find any bugs related to the changed linking.

There are a few bugs related to other extractor PRs which are introduced to NP in this PR, but are fixed separately. The extractor version can be updated independently.

The base branch was changed.

[Stypox]

I'm merging this now so that we can have it in the RC

[TobiGr]

we should also update the extractor commit. otherwise, there will be reports about broken comments