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

二次开发调整 #1022

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

二次开发调整 #1022

wants to merge 1 commit into from

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants