Markdown Preview: Make hyperlinks clickable

[thomas-mc-work]

At the moment the hyperlinks are just formatted as links (underlined and blue font colour) but not clickable. Would be nice to have them functionally.

[tobiasKaminsky]

@AndyScherzinger do you have an idea?

[AndyScherzinger]

I'll have a look, while it might come out of the box when the client supports Nextcloud Text.

[AndyScherzinger]

https://noties.io/Markwon/docs/v4/core/configuration.html#linkresolver / noties/Markwon#118 should do the trick while this doesn't seem to get triggered, so maybe somebody has an idea why, maybe @BaptisteRFoxIntel or @noties can shed some light onto this? 🙏

android/src/main/java/com/owncloud/android/ui/preview/PreviewTextFragment.java

Lines 286 to 300 in dbe12e6

	final Markwon markwon = Markwon.builder(context)
	.usePlugin(new AbstractMarkwonPlugin() {
	@Override
	public void configureTheme(@NonNull MarkwonTheme.Builder builder) {
	TextPaint textPaint = new TextPaint();
	textPaint.setColorFilter(new PorterDuffColorFilter(ThemeUtils.primaryColor(context), PorterDuff.Mode.SRC_ATOP));
	builder.linkColor(ThemeUtils.primaryColor(context, true));
	}
	})
	.usePlugin(TablePlugin.create(context))
	.usePlugin(TaskListPlugin.create(drawable))
	.usePlugin(StrikethroughPlugin.create())
	.usePlugin(HtmlPlugin.create())
	.usePlugin(SyntaxHighlightPlugin.create(prism4j, prism4jTheme))
	.build();

[noties]

Hey! It is always MovementMethod when links are not clickable. But Markwon will set one automatically if it's missing after text is rendered. And it works just fine in most of the cases until textIsSelectable is added. So, try to set LinkMovementMethod to your TextView before applying markdown content, like this:

// in onCreateView for example, or right before applying markdown
textView.setMovementMethod(LinkMovementMethod.getInstance());

/* ... */

markwon.setMarkdown(textView, markdown);

At least this worked for me

[AndyScherzinger]

@noties Thanks a lot for the hint! 🥇 Works for me too just fine.

@thomas-mc-work @tobiasKaminsky will open a PR is just a minute.