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

feat(frontend): 「照会」ダイアログに説明文を追加 #11137

Closed

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Jul 6, 2023

What

  • 照会ダイアログに説明文を追加
    「詳しく」はmisskey-hubにリンクしているため、Docs追加を待ってからマージする必要あり
    image

副次的な変更(重要)

  • MFMのリンクホバー時のプレビューツールチップを無効化するオプションを追加

Why

初見で説明無しはわかりにくい (微Related to #10582

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 Jul 6, 2023
@tamaina
Copy link
Contributor

tamaina commented Jul 6, 2023

照会を廃止して検索ページに一本化したい

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Jul 6, 2023

照会はダイアログなので、検索ページに移動してクエリ入力…とするよりもすばやくページジャンプができて(慣れれば)便利だと個人的に思うのですが

@codecov
Copy link

codecov bot commented Jul 6, 2023

Codecov Report

Merging #11137 (023e94d) into develop (c065b97) will increase coverage by 0.59%.
The diff coverage is 94.11%.

@@             Coverage Diff             @@
##           develop   #11137      +/-   ##
===========================================
+ Coverage    77.41%   78.00%   +0.59%     
===========================================
  Files          909      172     -737     
  Lines        91802    21642   -70160     
  Branches      7551      498    -7053     
===========================================
- Hits         71064    16882   -54182     
+ Misses       20738     4760   -15978     
Impacted Files Coverage Δ
...src/components/global/MkMisskeyFlavoredMarkdown.ts 29.18% <33.33%> (+0.03%) ⬆️
packages/frontend/src/components/MkDialog.vue 100.00% <100.00%> (ø)
packages/frontend/src/components/MkLink.vue 100.00% <100.00%> (ø)
packages/frontend/src/components/global/MkUrl.vue 100.00% <100.00%> (ø)

... and 737 files with indirect coverage changes

@syuilo
Copy link
Member

syuilo commented Jul 6, 2023

そんなに頻繁に行うようなものではないと思うし1ステップ増えるくらいではそこまで影響ないと思う

@tamaina
Copy link
Contributor

tamaina commented Jul 6, 2023

検索ページをもっとダイアログっぽくできないかしら

(自動でフォーカスするとか、照会っぽい動作をするならpushではなくreplaceで移動するとか

@kakkokari-gtyih
Copy link
Contributor Author

そんなに頻繁に行うようなものではないと思うし1ステップ増えるくらいではそこまで影響ないと思う

それもそうか

@kakkokari-gtyih
Copy link
Contributor Author

検索に統合のほうがよさそうですね

@syuilo
Copy link
Member

syuilo commented Jul 6, 2023

ただ自分は統合する場合の良いUIを思い付いていない

@tamaina
Copy link
Contributor

tamaina commented Jul 6, 2023

今のノート検索ページで照会できるようにすれば良いのでは?と思ったけどタブの括りが邪魔か?

あとは…まず昔のようにダイアログを表示し、検索ページへは「詳細検索」でアクセスできるようにするとか?

@kakkokari-gtyih
Copy link
Contributor Author

ただ自分は統合する場合の良いUIを思い付いていない

#10582 に投げました

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
None yet
Development

Successfully merging this pull request may close these issues.

3 participants