Skip to content

enhance(frontend): improve nested popup menu ux#17187

Open
syuilo wants to merge 18 commits intodevelopfrom
improve-nested-menu-selection-ux
Open

enhance(frontend): improve nested popup menu ux#17187
syuilo wants to merge 18 commits intodevelopfrom
improve-nested-menu-selection-ux

Conversation

@syuilo
Copy link
Member

@syuilo syuilo commented Feb 22, 2026

What

Before

before

After

after

After (visualized)

after2

Why

UX向上

Additional info (optional)

todo

  • 右以外に開く場合の対応
  • メニューがスクロールしている場合の対応

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@codecov
Copy link

codecov bot commented Feb 22, 2026

Codecov Report

❌ Patch coverage is 0% with 45 lines in your changes missing coverage. Please review.
✅ Project coverage is 13.76%. Comparing base (78435dc) to head (023d45b).
⚠️ Report is 181 commits behind head on develop.

Files with missing lines Patch % Lines
packages/frontend/src/components/MkMenu.vue 0.00% 36 Missing and 7 partials ⚠️
packages/frontend/src/components/MkPopupMenu.vue 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #17187      +/-   ##
===========================================
- Coverage    13.97%   13.76%   -0.22%     
===========================================
  Files          237      241       +4     
  Lines        11273    11729     +456     
  Branches      3728     3963     +235     
===========================================
+ Hits          1575     1614      +39     
- Misses        7578     7940     +362     
- Partials      2120     2175      +55     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@syuilo syuilo marked this pull request as ready for review February 23, 2026 09:10
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Feb 23, 2026
@syuilo
Copy link
Member Author

syuilo commented Feb 27, 2026

/preview

@syuilo
Copy link
Member Author

syuilo commented Mar 5, 2026

入れるか

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packages/frontend Client side specific issue/PR size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Development

Successfully merging this pull request may close these issues.

1 participant