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

优化标签页的显示 #60

Closed
wants to merge 1 commit into from
Closed

优化标签页的显示 #60

wants to merge 1 commit into from

Conversation

stevenjoezhang
Copy link

@stevenjoezhang stevenjoezhang commented Jan 3, 2019

如果一篇文章没有标签,标签页仍然会显示出来,有些不美观。
2019-01-03 1 05 37
可惜Typecho并没有提供获取Tags数量的函数,因而只能变通地解决这一问题,例如通过加上“标签”二字来避免空白内容——也是这个PR的解决方案。除此之外,page.php中的“标签”和“编辑”还没有做多语言支持。

@idawnlight
Copy link
Owner

显示一个「标签」似乎也并不美观...期末后尝试解决一下
(顺别申明这些 typo 一定不是我写的,一定不是

@stevenjoezhang
Copy link
Author

stevenjoezhang commented Jan 3, 2019

或者交给前端去实现?有点蛋疼😂
不过在$this->tags()里加上一个</li>还是必要的,不然标签不闭合...当然,现代浏览器会识别出来

@stevenjoezhang
Copy link
Author

还有一个问题是sidebar-toggle的逻辑
2019-01-28 13-51-23 2019-01-28 13_52_21
在点击之后,sidebar-toggle会变为箭头形式,但是由于z-index的原因会被展开的侧边栏挡住。收回侧边栏则需要点击另外一个sidebar-toggle。这个似乎也不是很优美。

@idawnlight
Copy link
Owner

@stevenjoezhang 我在 macOS 下用 Safari 和 Firefox 都未能复现(Chrome 没装

@stevenjoezhang
Copy link
Author

@idawnlight 可能没说清楚…这个sidebar-toggle的样式为z-index: 8,因此会被展开后的侧栏挡住,所以浏览器看上去没有问题。上面的截图是手动将其修改为z-index: 11的结果,可以看到侧栏展开后,箭头形式的sidebar-toggle是存在的。仅以此猜测原始的Material Design期望用户点击这个箭头图标(A)收回侧栏,而目前的设计是点击侧栏上的图标(B)。
2019-01-28 13-51-23 2019-01-28 13_52_21
其实也没什么问题,就是强迫症有点难受😂

@idawnlight
Copy link
Owner

@stevenjoezhang 看了一下,原版 demo 是不带侧边栏的,应该是 viosey 加的时候遗留下来的问题,你觉得怎么改比较好就怎么改呗

@stevenjoezhang
Copy link
Author

研究了一下Material Design的官方文档,感觉将下图(A)处的button移除即可(它是多余的,原始设计逻辑是用户点击侧栏右侧的屏幕将侧栏收回,而不是点击侧栏上的某个按钮)
2019-01-29 9 36 42
以及,(B)处在登陆后会显示用户邮箱,在未登录状态下是空白的,这个也有些奇怪😂

@idawnlight
Copy link
Owner

A 处也许能让侧边栏没那么单调,加了也没大问题对吧,反正这个主题已经很不符合规范了(
B 处想不到应该放什么玩意,Hexo 版放了联系邮箱,我因为懒直接用了旧版本的逻辑,你觉得放什么比较合适啊

idawnlight added a commit that referenced this pull request Feb 1, 2019
@stevenjoezhang
Copy link
Author

好像这样也没有什么问题...我把这个PR关掉啦

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.

2 participants