Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhance(frontend): 絵文字管理画面β(ローカル)のUI・UX改善 #15349

Merged
merged 8 commits into from
Jan 26, 2025

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Jan 26, 2025

What

絵文字管理画面βのローカル絵文字一覧のUI・UXを改善

image

image

image

image

UI

  • ウィジェット一覧が強制的に出ないようにした
    • 表部分の面積向上
  • 検索・変更ログをWindowにした
    • 画面の自由度の向上・表部分の面積向上
  • 表のスクロールエリアを固定し、画面全体をスクロールするのではなく、表単体でスクロールできるようにした
    • 直感的に操作できるように
    • この変更を実現するために、admin側のMkSuperMenuがstickyになっています
  • Bool値のセルをチェックボックスで表示するように
    • より直感的に値の型を認識できるように
  • 絵文字の表示件数を制限できるように
    • 100件だと重くなる環境があったみたいなのでデフォルトも25に落とした

UX

  • 画面のリセット時に確認画面が出るように
  • ページネーション変更時に確認画面が出るように

その他

  • すでに似たような翻訳があるものはそちらを使うように

Why

UI・UX改善

Additional info (optional)

リモートのほうもそのうちやる

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

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jan 26, 2025
Copy link

codecov bot commented Jan 26, 2025

Codecov Report

Attention: Patch coverage is 1.26382% with 625 lines in your changes missing coverage. Please review.

Project coverage is 18.83%. Comparing base (e94c697) to head (0d07998).
Report is 4 commits behind head on develop.

Files with missing lines Patch % Lines
.../admin/custom-emojis-manager.local.list.search.vue 0.00% 212 Missing and 1 partial ⚠️
...c/pages/admin/custom-emojis-manager.local.list.vue 0.00% 187 Missing ⚠️
packages/frontend/src/components/grid/MkGrid.vue 0.00% 44 Missing ⚠️
...es/admin/custom-emojis-manager.local.list.logs.vue 0.00% 38 Missing and 1 partial ⚠️
...ckages/frontend/src/components/grid/MkDataCell.vue 0.00% 31 Missing ⚠️
...d/src/pages/admin/custom-emojis-manager.remote.vue 0.00% 22 Missing ⚠️
...end/src/pages/admin/custom-emojis-manager.logs.vue 0.00% 21 Missing ⚠️
...nd/src/pages/admin/custom-emojis-manager.local.vue 0.00% 19 Missing ⚠️
...ges/admin/custom-emojis-manager.local.register.vue 0.00% 10 Missing ⚠️
...rontend/src/pages/admin/custom-emojis-manager2.vue 0.00% 10 Missing ⚠️
... and 9 more
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #15349       +/-   ##
============================================
- Coverage    38.70%   18.83%   -19.88%     
============================================
  Files         1600      764      -836     
  Lines       203821   111010    -92811     
  Branches      4013     1233     -2780     
============================================
- Hits         78899    20907    -57992     
+ Misses      124285    89516    -34769     
+ Partials       637      587       -50     

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

@Sayamame-beans
Copy link
Member

表示件数の設定UIは表の方の確認ボタンの横などに移動して、25/50/100件から選べるようにするみたいな形の方が、時々見かけるUIの形式に近い気がしました(検索内容の設定の側には無いような感じがしました?)

@kakkokari-gtyih
Copy link
Contributor Author

表示件数、下が窮屈になりそうだったので上にして置いてみた

image

@kakkokari-gtyih kakkokari-gtyih added this to the v2025.1.0 milestone Jan 26, 2025
@mi-gh-maintainer mi-gh-maintainer bot merged commit 297186e into misskey-dev:develop Jan 26, 2025
22 of 24 checks passed
Copy link

Thank you 🙏

@kakkokari-gtyih kakkokari-gtyih deleted the fix-mkgrid-0 branch January 26, 2025 11:21
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
Projects
Development

Successfully merging this pull request may close these issues.

3 participants