search

luogu-dev / luogu-blog-frontend

Frontend Theme Packs for Luogu Blog.
GNU Affero General Public License v3.0
93 stars 27 forks source link

完成theme/ant #29

Closed cleverdango closed 5 years ago

cleverdango commented 5 years ago

完成theme/ant

lin-toto commented 5 years ago

文章页面这样传递数据的方式可能会有SEO上的问题，建议改成在js加载之前先显示（并配合简单的样式），JS加载后再隐藏

cleverdango commented 5 years ago

改成这样：

<div id="article-comments">
  {{!-- 在加载Vue组件之后会把下面的内容替换掉 --}}
  <div id="article-content">
    {{{ postContent }}}
  </div>
</div>

Vue在加载的时候会自动替换掉里面的内容 在禁用js的情况下是存在id="article-content"部分的内容的

lin-toto commented 5 years ago

测试发现的问题：

  1. 列表页，左边的分类和搜索并不能用
  2. 发表评论后不显示评论人的用户名
  3. 文章页缺少返回列表页的导航按钮
  4. 需要一个Footer，或者至少一个到 Sitemap 的链接
cleverdango commented 5 years ago
  1. 修改了点击事件的bug，network的请求是发生改变了，这个mock服务器是不是测不出来
  2. 已增加
  3. 已增加至文章页标题栏旁
  4. 已增加至左栏
lin-toto commented 5 years ago

现在还有如下问题：

  1. 左边的搜索栏和tag筛选触发之后，network有请求，但右边内容并没有真的改变。
  2. 用户头像似乎显示不了
  3. 文章页面右侧的编辑按钮链接生成有问题，直接输出了模板字符串，以及那个设置按钮直接链接到了/，意义不明
lin-toto commented 5 years ago

残留问题：

  1. 编辑文章的链接链接到了undefined
  2. 两个往管理后台的链接，最好能在新页面打开

目前没有发现更多问题，这两个修好之后就部署到线上版本