Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Links in comments aren't clickable #571

Open
pemibo1986 opened this issue Jul 2, 2020 · 2 comments · May be fixed by #656
Open

Links in comments aren't clickable #571

pemibo1986 opened this issue Jul 2, 2020 · 2 comments · May be fixed by #656
Labels
Priority: Low On the radar, but not the most urgent thing Status: Confirmed Verified by someone other than the reporter

Comments

@pemibo1986
Copy link

pemibo1986 commented Jul 2, 2020 •
edited by danirabbit
Loading

Links in comments are plain text

screenshot
I often have to copy and paste long links from the comment field to the browser or other applications (Microsoft Teams).
Having clickable links would be a lot more comfortable.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
@pemibo1986
Copy link
Author

I was able to find where the comment_textview's text is set and I could parse links out of the string. Unfortunately I have no experience in Vala or GTK yet, so I haven't been able to do anything with it. I will look into GtkTextIter. Is this the right way?

@fabiozaramella fabiozaramella added Priority: Low On the radar, but not the most urgent thing Status: Confirmed Verified by someone other than the reporter labels Oct 14, 2020
@igordsm
Copy link

igordsm commented Mar 6, 2021

I've been working on this today and was able to get a first version working:

calendar-link

The only problem is getting the cursor to show a hand. I can get it almost working using the motion event, but if the mouse is not moving and control is pressed the cursor does not change... Any suggestions on how to do this are welcome.

@igordsm igordsm linked a pull request Mar 7, 2021 that will close this issue
Draft
@marbetschar marbetschar mentioned this issue Jun 24, 2021
Merged
@marbetschar marbetschar mentioned this issue Aug 19, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Priority: Low On the radar, but not the most urgent thing Status: Confirmed Verified by someone other than the reporter
Projects
None yet
Milestone
No milestone
3 participants
@igordsm @fabiozaramella @pemibo1986