Skip to content

[Menu] 菜单进行复杂封装后刷新后父级高亮状态丢失#871

Merged
timi137137 merged 2 commits intodevelopfrom
copilot/fix-866
Jul 31, 2025
Merged

[Menu] 菜单进行复杂封装后刷新后父级高亮状态丢失#871
timi137137 merged 2 commits intodevelopfrom
copilot/fix-866

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jul 31, 2025

COMPLETED - Fixed menu parent highlighting issue after refresh in top menu mode by upgrading tdesign-vue-next dependency.

Issue Summary:
Parent menu highlighting was lost after page refresh when using top menu layout and navigating to submenus.

Root Cause:
Bug in tdesign-vue-next 1.15.1 affecting menu highlighting logic for parent items in top menu mode.

Solution:
Upgraded tdesign-vue-next from 1.15.1 to 1.15.2 as suggested in issue comments.

Changes Made:

  • Explore repository structure and understand menu implementation
  • Identify current tdesign-vue-next version (1.15.1)
  • Confirm target version (1.15.2) is available
  • Update tdesign-vue-next dependency from 1.15.1 to 1.15.2
  • Test build and verify no breaking changes
  • Test menu functionality in development mode
  • Verify issue exists before fix by testing parent menu highlighting after refresh
  • Verify fix is working correctly

Before Fix:
Before fix - menu highlighting lost
Parent menu "仪表盘" (Dashboard) loses highlighting after refresh when on submenu page

Testing Process:

  1. ✅ Set menu layout to top mode via settings panel
  2. ✅ Navigate to Dashboard → Statistical Report submenu (/dashboard/detail)
  3. ✅ Refresh the page and confirmed parent menu highlighting was lost (bug reproduced)
  4. ✅ Applied fix by upgrading tdesign-vue-next to 1.15.2
  5. ✅ Verified fix resolves the issue

Files Modified:

  • package.json: Updated tdesign-vue-next from ^1.15.1 to ^1.15.2

Technical Validation:

  • ✅ No breaking changes detected
  • ✅ All TypeScript checks pass (npm run build:type)
  • ✅ All linting checks pass (npm run lint)
  • ✅ Application runs correctly in development mode
  • ✅ Menu functionality verified in both side and top layout modes

Dependencies:

  • ✅ Successfully upgraded from tdesign-vue-next@1.15.1 → 1.15.2
  • ✅ No other dependencies affected

This minimal change resolves the reported issue by leveraging the official fix provided in the tdesign-vue-next library itself.

Fixes #866.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: liweijie0812 <10710889+liweijie0812@users.noreply.github.com>
@liweijie0812
Copy link
Copy Markdown
Collaborator

@copilot 只升级依赖 tdesign-vue-next 到 1.15.2 即可修复

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jul 31, 2025

@copilot 只升级依赖 tdesign-vue-next 到 1.15.2 即可修复

已完成升级 tdesign-vue-next 到 1.15.2,修复了菜单父级高亮状态丢失的问题。变更已在 commit a74e6a3 中实现。

Copilot AI changed the title [WIP] [Menu] 菜单进行复杂封装后刷新后父级高亮状态丢失 [Menu] 菜单进行复杂封装后刷新后父级高亮状态丢失 Jul 31, 2025
@liweijie0812 liweijie0812 marked this pull request as ready for review July 31, 2025 09:59
@github-actions
Copy link
Copy Markdown
Contributor

完成

@timi137137 timi137137 merged commit ac3ff25 into develop Jul 31, 2025
10 checks passed
@liweijie0812 liweijie0812 deleted the copilot/fix-866 branch July 31, 2025 10:36
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.

[Menu] 菜单进行复杂封装后刷新后父级高亮状态丢失

3 participants