PKUHPC / OpenSCOW

Super Computing On Web
https://www.pkuscow.com/
Mulan Permissive Software License, Version 2
220 stars 49 forks source link

fix(web): 修复当跳往扩展页面的导航项位于已有导航项的下面时,此扩展页面的导航结构不显示的问题 #1393

Closed ddadaal closed 3 months ago

ddadaal commented 3 months ago

img_v3_02dl_5906595d-858e-42f4-a7bb-b244b6ce3ceg

主要逻辑:

  1. 如果一个导航项的任何一个子项被选中,那么一个导航项也会被选中,而即使此导航项本身的path和当前path没有关系
  2. 大屏幕的侧边栏显示的子导航为第一个被选中的一级导航项的子项
changeset-bot[bot] commented 3 months ago

🦋 Changeset detected

Latest commit: c2c39e7ea4d29d42e329b2f7049095c6efdb9c98

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages | Name | Type | | ------------------- | ----- | | @scow/lib-web | Patch | | @scow/ai | Patch | | @scow/mis-web | Patch | | @scow/portal-web | Patch | | @scow/auth | Patch | | @scow/gateway | Patch | | @scow/mis-server | Patch | | @scow/portal-server | Patch | | @scow/cli | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

piccaSun commented 3 months ago

@ddadaal 有一个小问题是下列配置项中的name会写入路由

uiExtension:
  -
    # 您的UI扩展页部署URL。
    name: EXT-TEST
    url: http://localhost:端口号

如果这个name配置为中文或者需要转义的特殊字符,是无法获取activeKeys的 aaa bbb ccc

但是现在导航项位于已有导航项下的情况,其实可以不配置name,直接更改扩展页的标题来进行页面标题的显示 #1389 所以请看看这个要不要修改,还是修改一下文档中关于name配置的说明?

ddadaal commented 3 months ago

已修复当extensionName为需要encode的字符的时候此修复不生效的问题

image