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.

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

Некорректно отображаются ссылки в чате/личке #707

Closed
pavel-pimenov opened this issue Aug 21, 2015 · 6 comments

Comments

@pavel-pimenov
Copy link
Owner

@pavel-pimenov pavel-pimenov commented Aug 21, 2015

From zippoz...@gmail.com on January 30, 2012 07:19:59

При вставке любой ссылки в чат (для примера ссылка на хаб), не важно в чате ADC или NMDC хаба, "спецсимволы" в конце ссылки воспринимаются как часть самой ссылки. А именно: пробел, ), {, }, [, ], и прочее

Пример:
текст-текст [dchub://somehub.cx] текст-текст НЕПРАВИЛЬНО
dchub://somehub.cx] (после символа ] нет пробелов и текстов) ПРАВИЛЬНО
dchub://somehub.cx]_ (_ = пробел) НЕПРАВИЛЬНО

Вторая ссылка отобразится правильно, выделив только основую ссылку, символ ] - чёрный, как обычный текст.
Остальные ссылки некорректные.
Аналогичная проблема с магнет-ссылками

Предположительно: неверно ищется конец части текста
Часть кода (список спецсимволов) ChatCtrl.cpp:

static const TCHAR* g_BadUrlSymbol[] = { _T("""), _T(" "), _T("""), _T("<"), _T(">"),
_T("["), _T("]") // TODO used in BB codes, needs refactoring
/* _T("^"),_T("`"),_T("{"),_T("|"),_T("}"),T(""),T("'")/

Original issue: http://code.google.com/p/flylinkdc/issues/detail?id=668

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 21, 2015

From a.rain...@gmail.com on January 29, 2012 23:39:50

Status: Started
Labels: -OpSys-Windows Component-UI

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 21, 2015

From a.rain...@gmail.com on February 05, 2012 14:16:47

This issue was closed by revision r9201 .

Status: Fixed

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 21, 2015

From tret2...@gmail.com on February 07, 2012 05:23:19

This issue was closed by revision r9206 .

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 21, 2015

From JhaoDa on February 07, 2012 05:43:59

Смайлик, стоящий вплотную к ссылке, считается её частью — [dchub://somehub.cx:)]. Надо регэксп какой-то делать, однако.

Status: Accepted

Attachment: link_smile.png

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 21, 2015

From a.rain...@gmail.com on February 07, 2012 05:51:09

mailto:John.Doe@example.com Проблему со смайлом оформить необходимо отдельной ишьёй.

Status: Fixed

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 21, 2015

From tret2...@gmail.com on February 23, 2012 07:15:41

Status: Verified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant