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

投稿フォームにノートのURLを貼り付けた時の挙動を改善する #10598

Open
EbiseLutica opened this issue Apr 12, 2023 · 3 comments
Assignees
Labels
✨Feature This adds/improves/enhances a feature packages/frontend Client side specific issue/PR

Comments

@EbiseLutica
Copy link
Contributor

Summary

投稿フォームにノートのURLを貼り付ける形で引用するときに2点問題がある。

  • 引用付き [ x ] とだけ表示され、ノートのプレビューが表示されない
    • (そもそもボタンがスタイル無しでダサい)
  • ノートを検証していないため、他者の非公開ノートも貼り付けられてしまう
    • APIレベルで弾かれるため大した問題ではないけど、フロントエンドとして良くはない

以上の問題を解決するため、投稿フォームにノートURLを貼り付けた時の挙動を以下のように改善する。

  • URL内のノートIDから notes/show をクエリする
    • Renote不可能なノートなら
      • URL貼り付けの挙動
    • Renote可能なノートなら
      • 「引用として送付しますか?」を表示
        • いいえ → URL貼り付けの挙動
        • はい → 引用ノートと同じようにプレビュー表示する
@EbiseLutica EbiseLutica added the ✨Feature This adds/improves/enhances a feature label Apr 12, 2023
@syuilo
Copy link
Member

syuilo commented Apr 12, 2023

URL内のノートIDから notes/show をクエリする
Renote不可能なノートなら
URL貼り付けの挙動
Renote可能なノートなら
「引用として送付しますか?」を表示
いいえ → URL貼り付けの挙動
はい → 引用ノートと同じようにプレビュー表示する

notes/showをしてからだと、APIレスポンスが遅い場合に挙動が不自然になりそう?(貼り付けたのに貼り付けられないような感じになると思う)

@syuilo syuilo added the packages/frontend Client side specific issue/PR label Apr 12, 2023
@EbiseLutica
Copy link
Contributor Author

EbiseLutica commented Apr 12, 2023

notes/showをしてからだと、APIレスポンスが遅い場合に挙動が不自然になりそう?(貼り付けたのに貼り付けられないような感じになると思う)

apiDialogでクルクルを出すことになると思う 待たされるのは確かに微妙なんだけど、ダイアログを出してからクエリして、Renote不可能だった場合にURL貼り付け挙動にしたり、エラーを出したりするのは、それはそれで良くない気がする

@syuilo
Copy link
Member

syuilo commented Apr 12, 2023

YOSASOUI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature packages/frontend Client side specific issue/PR
Projects
Status: No status
Development

No branches or pull requests

2 participants