Skip to content

Conversation

@tammy000
Copy link

本次修改内容如下:
1, 修复leftMenu报错length of undefined
2, 修复路由重复报错 "/5/?page_id=23"
3, 提出header组件,作为全局页面的头部使用, 不然页面太难看了,就像没有头一样,而且原项目每个页面都引入了一个空Header
4, 修改小屏时右侧按钮的位置样式

@star7th
Copy link
Owner

star7th commented Jul 21, 2020

感谢贡献代码。我下载打包看了下。对报错的修复,是可取的,实用的。不过UI上的话,我可能有些不一样的看法。
针对你当前这个header,我发现几个问题:1,没有游客和登录用户的区别 。游客看到的还是登录用户看到的按钮。这是不合理的。2,对移动端不友好。无论是header还是小屏时右侧按钮的,再移动屏幕下的观感不好。3,对表格类型的项目,有这个header看上去很奇怪。
我当初预留空header组件是想着以后有没有可能会需要全局加载点什么,但后来还是没启用。每个页面我都几乎做了返回按钮,都是可以联通的,其实我偏向于不需要全局header,不够简洁。

@star7th
Copy link
Owner

star7th commented Jul 21, 2020

其实我觉得这个配色蛮好看。不过当前和showdoc的蓝白灰黑这样的配色不搭。比如下图的这个。可能以后我会改造下主题,把这个配色融进去,把原来的蓝色基调去掉。然后,可能采用比较窄一点的header , 放在顶部或者最左侧之类的。这个事情我需要重新规划好整个UI配色,可能不是加个header就能解决。但谢谢你提供的这个灵感。

image

@star7th
Copy link
Owner

star7th commented Jul 21, 2020

我上面说的右侧按钮对移动端不友好指的是 下图的两个按钮,其实它们在同一条水平线上会更好。此外,很多用户用showdoc来放教程文档,就是说他写给他的目标群体看的。如果我强制在这个页面加一个这么大的logo,会不会显得过于碍眼(因为跟他的目标用户群体无关),这个我需要再额外思考。

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants