Pinned messages are now opened properly in threads

[rapterjet2004]

• fixes pinned message in thread behavior #5860

This uses the correct context to launch the message viewer. Also since thread messages are just regular messages with a thread id, If you pin a message in a thread, and try to access it outside the thread, it just opens it up in the chat context view.

🏁 Checklist

• ⛑️ Tests (unit and/or integration) are included or not needed
• 🔖 Capability is checked or not needed
• 🔙 Backport requests are created or not needed: /backport to stable-xx.x
• 📅 Milestone is set
• 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

[sowjanyakch]

@rapterjet2004 - tested this PR and works as expected.

One small suggestion: when a pinned thread message is opened from the main conversation, the search results view points to the thread pinned message and from the UI it is hard to tell that it is a thread message (looks more like a reply to a message in this case). We could show a thread icon and title with the thread message, so it is obvious that the search result is a thread message. wdyt?

[rapterjet2004]

One small suggestion: when a pinned thread message is opened from the main conversation, the search results view points to the thread pinned message and from the UI it is hard to tell that it is a thread message (looks more like a reply to a message in this case). We could show a thread icon and title with the thread message, so it is obvious that the search result is a thread message. wdyt?

Might be better off implemented in another PR, as I think deliberating over the thread feature's UI is outside the scope of issue #5895

[github-actions]

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/5895.apk



To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

[github-actions]

Codacy

Lint

Type	master	PR
Warnings	98	98
Errors	0	0

SpotBugs

Category	Base	New
Bad practice	6	6
Correctness	10	10
Dodgy code	54	54
Internationalization	3	3
Malicious code vulnerability	3	3
Performance	4	4
Security	1	1
Total	81	81