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

クリップの解除ができない #7020

Closed
futchitwo opened this issue Dec 25, 2020 · 11 comments · Fixed by #8823
Closed

クリップの解除ができない #7020

futchitwo opened this issue Dec 25, 2020 · 11 comments · Fixed by #8823
Assignees
Labels
packages/frontend Client side specific issue/PR

Comments

@futchitwo
Copy link
Contributor

futchitwo commented Dec 25, 2020

Removing clipped notes

💡 Summary

任意のノートのクリップを解除する方法がない。
間違ってクリップしてしまった場合などに必要。

🙂 Expected Behavior

任意のクリップを解除できる

☹️ Actual Behavior

クリップが解除できない

📌 Environment

Misskey 12.63.0

@futchitwo futchitwo added the ⚠️bug? This might be a bug label Dec 25, 2020
@syuilo syuilo added packages/frontend Client side specific issue/PR and removed ⚠️bug? This might be a bug labels Dec 10, 2021
@syuilo syuilo self-assigned this Dec 10, 2021
@futchitwo
Copy link
Contributor Author

これ @syuilo にアサインされてるけどやってみていいかな
syuilo も忙しそうだし

@tamaina tamaina assigned futchitwo and unassigned syuilo May 29, 2022
@tamaina
Copy link
Contributor

tamaina commented May 29, 2022

しゅいろからアサイン外した

@syuilo
Copy link
Member

syuilo commented May 29, 2022

🙏

@syuilo
Copy link
Member

syuilo commented May 29, 2022

どのようなUIにするかが考えどころ

@futchitwo
Copy link
Contributor Author

futchitwo commented May 29, 2022

ノートメニューのクリップの項目から、クリップに含めるときと同じ操作をするとすでにクリップされていますが解除しますか的なダイアログとともにクリップ解除
(本当はクリップ名とともにノートがそれぞれのクリップに含まれているかを表示できればいいんだけど、APIの変更いりそう)

あとはクリップ一覧ページにいるときにはノートメニューのトップにでもクリップ解除ボタンを出せたらいいんだけど難しいかな

クリップ一覧にクリップ解除モードみたいなのをつけるのもあり

@syuilo
Copy link
Member

syuilo commented May 29, 2022

あとはクリップ一覧ページにいるときにはノートメニューのトップにでもクリップ解除ボタンを出せたらいいんだけど難しいかな

やろとしたらこれなんだよね
クリップ一覧ページでクリップ情報をprovideして、noteコンポーネントでinjectしてよしなに処理するとか

@tamaina
Copy link
Contributor

tamaina commented May 29, 2022

メニューから「編集」を選択すると各ノートの左側にチェックマークが出現する→どこか(メニュー内?)に削除ボタンを設置して、それを押すとチェックした投稿がクリップから削除される

的な感じのUIを想像した

@syuilo
Copy link
Member

syuilo commented May 29, 2022

クリップ一覧にクリップ解除モードみたいなのをつけるのもあり

メニューから「編集」を選択すると各ノートの左側にチェックマークが出現する→どこか(メニュー内?)に削除ボタンを設置して、それを押すとチェックした投稿がクリップから削除される

実装が少し複雑になりそう
あとパフォーマンス上の懸念も少しある

@futchitwo
Copy link
Contributor Author

メニューから「編集」を選択すると各ノートの左側にチェックマークが出現する→どこか(メニュー内?)に削除ボタンを設置して、それを押すとチェックした投稿がクリップから削除される

メニュー内に削除ボタン付けるなら編集モードはいらないと思う

@tamaina
Copy link
Contributor

tamaina commented May 29, 2022

編集モード(チェックボックスが表示される間)でないと削除ボタンは表示されないので編集ボタンは必要

@Johann150
Copy link
Contributor

Johann150 commented May 29, 2022

I'm not sure if notes can be added to a clip more than once. Otherwise the existing clip menu entry on notes could have toggle entries which clip a note or unclip it if it is already contained.
But I think it would be necessary to know to which clip the note is attached, which would require an API change and might be bad for server performance.
But it would improve UX especially when trying to unclip a note when it was clipped by mistake because you would not have to go into the clip to unclip.

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 a pull request may close this issue.

4 participants