cnodejs / nodeclub

:baby_chick:Nodeclub 是使用 Node.js 和 MongoDB 开发的社区系统
http://cnodejs.org/
MIT License
9.34k stars 3.12k forks source link

SIngle column layout #329

Closed tiye closed 10 years ago

tiye commented 10 years ago

尝试修改界面成为单栏布局, 变动太大, 商量下: 分支: https://github.com/jiyinyiyong/nodeclub/tree/single-column Demo: http://nodeclub.tiye.me/

另外我打开了 Tag 界面, 计划使用 <select> 标签来选择 Tag. 将 Tag 传入服务端的逻辑目前没有完成.. 后端需要确认下

alsotang commented 10 years ago

这是为了单栏而单栏吗?空那么多出来哪里好看了?

tiye commented 10 years ago

@alsotang 简化界面是一个重要的动机, 具体修改不是单纯为了单栏, 原因:

我想突出每个页面独立的功能, 如果以后再增加功能, 可以清晰一些.

JacksonTian commented 10 years ago

应该自适应,当屏幕小于一定宽度时才开启单单栏模式。不要啥情况都单栏好不。

在 2014年4月21日,下午2:44,JiyinYiyong 题叶 notifications@github.com 写道:

@alsotang 简化界面是一个重要的动机, 具体修改不是单纯为了单栏, 原因:

帖子正文能显示在中间 主页的列表可以显示更多东西, 比如喜欢的次数 放开空间方便启用标签后显示标签 几个细的功能拆到新的页面, 比如无人回复和热门话题, 显示更多内容, 但不是在主页上 和移动设备访问稍微统一一些, 不过这个还得继续改 我想突出每个页面独立的功能, 如果以后再增加功能, 可以清晰一些.

— Reply to this email directly or view it on GitHub.

tiye commented 10 years ago

其实每次有点大的改界面都是一点点跟 Discourse 靠拢... http://discuss.atom.io/ 两栏的布局怎功能上并没有体现出明显优势, 而且不方便增加其他功能.

tiye commented 10 years ago

没有太大问题我继续在这个分支做了, 拖了比较长时间, 影响到其他功能修改. 做法有点激进, 希望不影响现有功能的情况下能快点推进了. 单栏以后, 打算强化系标签的功能, 用标签作为分类, 突出几个主题: 分享, 招聘, 问答, Wiki

alsotang commented 10 years ago

别急,这个改动太大,稍缓。免得你辛苦做了我合不了。

tiye commented 10 years ago

现在分支上已经把难的部分做了, 剩下是关于标签使用这部分的逻辑

alsotang commented 10 years ago

怎么又要开始使用标签了?? On 2014年4月22日, at 13:26, JiyinYiyong 题叶 notifications@github.com wrote:

现在分支上已经把难的部分做了, 剩下是关于标签使用这部分的逻辑

— Reply to this email directly or view it on GitHub.

tiye commented 10 years ago

想把上边几个主题的帖子分离出来, 招聘比较明显, 如果有人真想通过论坛找工作或者找开发者, 现在的版面是不够用的. 而且部分新手问题的帖子, 也可以通过这个标签从主页分流掉. 目前想法是这样, 具体的实现以后还要想

alsotang commented 10 years ago

网站流量太小了,先不这样分。 On 2014年4月22日, at 13:37, JiyinYiyong 题叶 notifications@github.com wrote:

想把上边几个主题的帖子分离出来, 招聘比较明显, 如果有人真想通过论坛找工作或者找开发者, 现在的版面是不够用的. 而且部分新手问题的帖子, 也可以通过这个标签从主页分流掉. 目前想法是这样, 具体的实现以后还要想

— Reply to this email directly or view it on GitHub.

tiye commented 10 years ago

你那边关于 CNode 增减功能的方向有和他们几位常商量吗? 我可能没对上你们的节奏啊.. 要不后面有更新都做一些提前的说明白, 我这边这次是考虑过多了

alsotang commented 10 years ago

商量倒是不多。

改动比较小的话,一般都是直接merge的。

改动比较大的话,就先在 cnode 上开个帖子讨论看看嘛。

否则也怕浪费劳动。

tiye commented 10 years ago

单栏问题我昨天在 CNode 发贴了的, 没有很明确的意见. 论坛似乎 504 了.. 没法拷链接..

alsotang commented 10 years ago

image

这个帖子导致 504 的,一点开就 CPU 100%,真无奈。

alsotang commented 10 years ago

marked 是不是出新 bug 了?

tiye commented 10 years ago

不清楚啊.. 后台能看帖子内容么? 备份一遍内容先清掉(或者整个标记成代码)保证论坛能跑? .... 不小心我也点了一次 >_<

saighost commented 10 years ago
发现一个 marked的bug,CPU高占,求解决 http://cnodejs.org/topic/535607120d7d0faf1404731f http://cnodejs.org/topic/535607120d7d0faf1404731f

如果包含大量的下划线,Marked 处理 em的正则会出现 CPU高占,直至Chrome崩溃的现象。 Firefox没有这个正则的bug。

Marked的正则:^\b((?:|[\s\S])+?)\b|^*((?:**|[\s\S])+?)*(?!*)

处理文本: Result __Calculate __TempFile __tmpForCompareFile TransFile __datFile ___2

hertz Tue, 22 Apr 2014 06:07:14 GMT
alsotang commented 10 years ago

@saighost 这个 bug 拜托能不能去 marked 那里提?这八成是正则表达式嵌套的导致的 cpu 飙高。

alsotang commented 10 years ago

@saighost 你是把论坛的帖子删了吗?

saighost commented 10 years ago

我没权限啊,我看的RSS,我去试试提交给marked

tiye commented 10 years ago

@saighost 来一个.. https://github.com/chjj/marked/issues/new

alsotang commented 10 years ago
var marked = require('marked');

console.log(marked('如果包含大量的下划线,Marked 处理 em的正则会出现 CPU高占,直至Chrome崩溃的现象。  Firefox没有这个正则的bug。\r\n\r\nMarked的正则:^\\b_((?:__|[\\s\\S])+?)_\\b|^\\*((?:\\*\\*|[\\s\\S])+?)\\*(?!\\*)\r\n\r\n处理文本:\r\n_Result\r\n__________Calculate\r\n__________TempFile\r\n__________tmpForCompareFile TransFile\r\n__________datFile\r\n__________2\r\n'));

我复现了这个 bug,确实卡住了。

@saighost 你去 marked 那里提提吧

saighost commented 10 years ago

https://github.com/chjj/marked/issues/59 我看他曾经有过这个bug啊。。。

saighost commented 10 years ago

不过我已经提了一个了。github也针对这种情况特殊处理了,所以在这没问题。

alsotang commented 10 years ago

chjj/marked#59

这个只是说,有多个,而不是连续多个吧。

alsotang commented 10 years ago

我开着 gfm 模式也卡。看来 marked 是要好好处理一下这个问题了。

var marked = require('marked');
marked.setOptions({
  renderer: new marked.Renderer(),
  gfm: true,
  tables: true,
  breaks: false,
  pedantic: false,
  sanitize: true,
  smartLists: true,
  smartypants: false
});

console.log(marked('如果包含大量的下划线,Marked 处理 em的正则会出现 CPU高占,直至Chrome崩溃的现象。  Firefox没有这个正则的bug。\r\n\r\nMarked的正则:^\\b_((?:__|[\\s\\S])+?)_\\b|^\\*((?:\\*\\*|[\\s\\S])+?)\\*(?!\\*)\r\n\r\n处理文本:\r\n_Result\r\n__________Calculate\r\n__________TempFile\r\n__________tmpForCompareFile TransFile\r\n__________datFile\r\n__________2\r\n'));
alsotang commented 10 years ago

目前先不改单栏