Skip to content
This repository has been archived by the owner on Jun 7, 2020. It is now read-only.

[NEW] Extract urls from room description and make them clickable #2249

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

water2015sit
Copy link

@RocketChat/android

Changes:

Extract urls from room description.
Make urls clickable.

Screenshots or GIF for the change:

Screenshot_20190420-120954

IMG_2585.MOV.zip

@water2015sit
Copy link
Author

@bizzbyster Would you have time to take a look? Thank you 😊

@Adizbek
Copy link
Contributor

Adizbek commented Apr 21, 2019

@water2015sit, good, but actually it should be text "realtime contribution leaderboard" with url https://gsoc.rocket.chat

It's Makrdown like in GitHub.

@philipbrito
Copy link
Contributor

@water2015sit Are you going to update this PR (taking in consideration the comment above from @Adizbek)?

@water2015sit
Copy link
Author

@filipedelimabrito Yes, I will update this PR.

@philipbrito
Copy link
Contributor

Thanks @water2015sit!

@philipbrito philipbrito changed the title [FEATURE] Extract urls from room description and make them clickable [NEW] Extract urls from room description and make them clickable Jun 3, 2019
@philipbrito philipbrito added this to the 3.5.0 milestone Jun 3, 2019
for (clickablePart in clickablePartList) {
val clickableSpan = object : ClickableSpan() {
override fun onClick(widget: View) {
val intent = Intent(Intent.ACTION_VIEW, Uri.parse(clickablePart))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

val intent is not being used here.

Copy link
Contributor

@philipbrito philipbrito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@water2015sit It is not working properly. Just get a description where the rocket.chat word appears, it is being highlighted but it is not possible to open that supposed link we have there.

Something like [site name](site url) is not working too.

Can you fix it?

@water2015sit
Copy link
Author

@filipedelimabrito OK, I will fix it.

@philipbrito
Copy link
Contributor

@water2015sit Are you able to fix it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants