At功能在回复的时候没用，当字符串处理了

[atjiu]

Rt

[TakWolf]

创建一个评论是在本地渲染的markdown，然后模拟创建一个对象插入List
本地的Markdown渲染暂时没实现@协议

[atjiu]

At功能应该在服务端做对吧！nodeclub没处理这块吗？

[TakWolf]

@liygheart

服务端有md渲染功能，可以解析@协议
客户端也有md渲染功能，暂时没实现@协议

通常内容数据，都是开启mdrander参数由服务器渲染的，@都是正常解析的

客户端在本地使用md渲染的情况为：
1.在md编辑器编辑内容，然后点了预览
2.发表评论的这个动作成功后，客户端没有重新获取数据，而是在本地创建了一个数据，插到了List中

目前只有这两种情况，@不会解析
你提的这种情况属于第二种，你重新刷新一下页面，@就正常解析了

[atjiu]

你这样一说我就明白了，谢谢！

[TakWolf]

fix by a97bb95

[atjiu]

好快，感谢大大

[alsotang]

所以现在的意思是，本地回复之后，虽然没有渲染 @ 对应的跳转链接，但实际上服务器是 at 成功的，对吧？

2016-04-21 10:02 GMT+08:00 朋也 notifications@github.com:

Closed #19 #19.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#19 (comment)

[TakWolf]

@alsotang 对的，服务器是正常渲染的，只是我本地渲染的时候没处理@

[alsotang]

哈哈，服务器处理这个 at 符号，也是正则表达式做的。

为了确定这个正则的正确，还写了一大串测试：
https://github.com/cnodejs/nodeclub/blob/master/test%2Fcommon%2Fat.test.js

2016-04-21 12:19 GMT+08:00 TakWolf notifications@github.com:

@alsotang https://github.com/alsotang 对的，服务器是正常渲染的，只是我本地渲染的时候没处理@

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#19 (comment)

[atjiu]

怎么又reopen了？

[TakWolf]

这里的匹配存在问题：

1. 邮件不应该被匹配，如 takwolf@foxmail.com不应该被匹配
2. 已经生成的@不应该匹配如 [@TakWolf](/user/TakWolf) ，实际上以[开头或者以]结尾的不匹配，但是这样的情况 [ @TakWolf ](/user/TakWolf)，不确定是不是正确的书写
3. 在代码块中和引用中的@不应该匹配
4. 连接的@应该被匹配例如：@TakWolf@alsotong 应该解析出两个连接

nodeclub的正则解析参考：

https://github.com/cnodejs/nodeclub/blob/master/test%2Fcommon%2Fat.test.js
https://github.com/cnodejs/nodeclub/blob/master/common/at.js

[alsotang]

这个 at 功能已经很久没有改动了，我觉得你可以直接拿去用。

在 2016年5月27日 下午6:28，TakWolf notifications@github.com写道：

这里的匹配存在问题：

邮件不应该被匹配，如 takwolf@foxmail.com不应该被匹配
2.

已经生成的@不应该匹配如 @TakWolf ，实际上以[开头或者以]结尾的不匹配，但是这样的情况
@TakWolf ，不确定是不是正确的书写
3.

在代码块中和引用中的@不应该匹配
4.

连接的@应该被匹配例如：@TakWolf@alsotong 应该解析出两个连接

nodeclub的正则解析参考：

https://github.com/cnodejs/nodeclub/blob/master/common/at.js

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#19 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABGB7w4i1ez2yzoek0M05EanTVJ4Lbczks5qFsdCgaJpZM4ILwVR
.