Skip to content
This repository has been archived by the owner on Jan 21, 2024. It is now read-only.

refactor: theme setting page #380

Merged
merged 5 commits into from
Jan 14, 2022

Conversation

ruibaby
Copy link
Member

@ruibaby ruibaby commented Nov 26, 2021

  1. 提供主题设置的路由和菜单。从菜单进入主题设置默认加载当前激活主题。
  2. 取消抽屉式的主题设置界面。
  3. 提供主题的基本信息展示。

image

@ruibaby ruibaby added the kind/feature Categorizes issue or PR as related to a new feature. label Nov 26, 2021
@ruibaby ruibaby added this to the 1.5.x milestone Nov 26, 2021
@ruibaby ruibaby marked this pull request as draft November 26, 2021 15:34
@ruibaby ruibaby marked this pull request as ready for review January 13, 2022 10:19
@guqing
Copy link
Member

guqing commented Jan 14, 2022

关于tab要不要放到最后去

@guqing
Copy link
Member

guqing commented Jan 14, 2022

sdk升级一下 不然保存主题这个设置的功能不生效就不方便验证参数是否正确

@ruibaby
Copy link
Member Author

ruibaby commented Jan 14, 2022

关于tab要不要放到最后去

可以,但考虑到当主题的 tab 很多的话,这个 tab 会被隐藏。

@ruibaby
Copy link
Member Author

ruibaby commented Jan 14, 2022

sdk升级一下 不然保存主题这个设置的功能不生效就不方便验证参数是否正确

等我研究一下 js-sdk 的发版。

Copy link
Member

@guqing guqing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@JohnNiang JohnNiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ruibaby ruibaby merged commit d3070ac into halo-dev:master Jan 14, 2022
@ruibaby ruibaby deleted the refactor/theme-setting-page branch January 14, 2022 07:37
@ruibaby ruibaby mentioned this pull request Feb 21, 2022
JohnNiang pushed a commit to JohnNiang/halo that referenced this pull request Mar 2, 2023
* refactor: theme setting page

Signed-off-by: Ryan Wang <[email protected]>

* feat: add theme descriptions

Signed-off-by: Ryan Wang <[email protected]>

* refactor: local upgrade component

Signed-off-by: Ryan Wang <[email protected]>

* refactor: theme descriptions

Signed-off-by: Ryan Wang <[email protected]>

* chore: bump admin-api deps
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants