cnodejs / cnodejs-reader

CNode.js reader in React
http://repo.tiye.me/cnodejs/cnodejs-reader/#/
66 stars 8 forks source link

Content in middle panel and right panel disappears when the same topic is clicked #1

Closed shuson closed 8 years ago

shuson commented 8 years ago

How to reproduce:

  1. open home page: http://r.nodejs-china.org/
  2. click a topic on the left panel, the middle and right panel is filled with content
  3. click the same topic again, content in middle and right panel disappears.
alsotang commented 8 years ago

试了一下,我这也重现了

2015-10-28 10:21 GMT+08:00 Shusen Wang notifications@github.com:

How to reproduce:

  1. open home page: http://r.nodejs-china.org/
  2. click a topic on the left panel, the middle and right panel is filled with content
  3. click the same topic again, content in middle and right panel disappears.

— Reply to this email directly or view it on GitHub https://github.com/cnodejs/cnodejs-reader/issues/1.

tiye commented 8 years ago

There's still bugs in router, which is router-view, especially in Hash mode. I need to fix that later.

alsotang commented 8 years ago

牛人啊,组件都是自己写的

2015-10-28 12:33 GMT+08:00 题叶 notifications@github.com:

There's still bugs in router, which is router-view https://github.com/teambition/router-view, especially in Hash mode. I need to fix that later.

— Reply to this email directly or view it on GitHub https://github.com/cnodejs/cnodejs-reader/issues/1#issuecomment-151718778 .

tiye commented 8 years ago

前端困难的事情, 一个是多种多样的设备的坑, 一个制造开发方案比如 Webpack, React, Vue. 我这只是个百来行的小组件, 而且没有解决掉所有细节... 没什么高大上的技术~

tiye commented 8 years ago

should work now, please check

alsotang commented 8 years ago

嗯嗯好了。 我以为再次点击会有类似刷新的效果,现在看来是直接忽略点击事件了。。

2015-11-04 1:02 GMT+08:00 题叶 notifications@github.com:

should work now, please check

— Reply to this email directly or view it on GitHub https://github.com/cnodejs/cnodejs-reader/issues/1#issuecomment-153418311 .

tiye commented 8 years ago

刷新需要通过 refresh 按钮来做, 是整个话题列表刷新的.

alsotang commented 8 years ago

点击同一话题的时候,可以单个 topic 的刷新,而非整个话题列表刷新啊

2015-11-04 13:24 GMT+08:00 题叶 notifications@github.com:

刷新需要通过 refresh 按钮来做, 是整个话题列表刷新的.

— Reply to this email directly or view it on GitHub https://github.com/cnodejs/cnodejs-reader/issues/1#issuecomment-153576906 .

tiye commented 8 years ago

架构设计的时候没有从这一点考虑的.. 而是尽量用本地的缓存数据, 加快渲染.