Tdesktop: URLs at the end of Group/Channel Names are appended an additional 禄

Created on 22 Oct 2016  路  7Comments  路  Source: telegramdesktop/tdesktop

Steps to reproduce

  1. Change the groupname, ending in a URL
  2. Click the URL on the Desktop Client

    Expected behaviour

URL is opened as entered

Actual behaviour

The 禄 at the End of the Channelname in the Group view is appended to the URL (www.google.com would become www.google.com

Configuration

Win10

Version of Telegram Desktop:
TDesktop 0.10.16alpha

auto closed

Most helpful comment

@T3rminat0r Well, currently links parsing is applied to the resulting string and it is parsed as it should be (like here on github), looks like that quote is a valid character for a link. It can be changed to parsing links specifically in the new title text and then forcing those links in the final message text, but it is a big change to the current logic.

It is easier and more consistent with mobile apps to disable links parsing in the service messages at all.

All 7 comments

"in the Group view" where do you see the links being clickable in the group name?

When you change the group name, there's a message <<xyz changed the groupname to new name http://www.google.com>>

It is clickable there.

I've tried, I see the service message "... changed group name to 芦test super https://google.com禄", but I can't click on https://google.com there. Tried "... changed group name to 芦test super www.google.com禄" also -- can't click as well.

Wow, GitHub takes that symbol to the link as well :D

Yea ... Github doesn't know any better... ;) But seeing that TDesktop is setting that service message up, it should be able to tell that the closing quotes aren't part of the link? (sent you a screenshot of the clickable Link via TG)

@T3rminat0r Well, currently links parsing is applied to the resulting string and it is parsed as it should be (like here on github), looks like that quote is a valid character for a link. It can be changed to parsing links specifically in the new title text and then forcing those links in the final message text, but it is a big change to the current logic.

It is easier and more consistent with mobile apps to disable links parsing in the service messages at all.

Hey there!

We're automatically closing this issue since there was no activity in this issue since 381 days ago. We therefore assume that the user has lost interest or resolved the problem on their own. Closed issues that remain inactive for a long period may get automatically locked.

Don't worry though; if this is in error, let us know with a comment and we'll be happy to reopen the issue.

Thanks!

(Please note that this is an automated comment.)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

TotalKrill picture TotalKrill  路  3Comments

Justinzobel picture Justinzobel  路  3Comments

beppe9000 picture beppe9000  路  3Comments

Liudvikas picture Liudvikas  路  3Comments

whywhyy picture whywhyy  路  3Comments