-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
リノートの〇時間前を押した時にリノートの詳細に飛ぶようにする #14958
base: develop
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #14958 +/- ##
============================================
- Coverage 41.73% 19.34% -22.39%
============================================
Files 1564 727 -837
Lines 202871 103564 -99307
Branches 3670 990 -2680
============================================
- Hits 84660 20035 -64625
+ Misses 117605 82976 -34629
+ Partials 606 553 -53 ☔ View full report in Codecov by Sentry. |
挙動に一貫性がなくなる(投稿時間の部分をクリックするとその投稿の詳細に移動するというメンタルモデルが崩れる)のが気になりますね |
リノートの詳細画面はリノート元ノートの詳細画面の機能をほぼ全部持っているので実害は無いのではないかと考えています。 |
URLが違うから共有しようとした時に困るかも |
リノートの時間の方を押したときにリノートを開くようにではいけないでしょうか? (ノート本体の方はそのままで…) |
リノートのpopupMenuのなかに「リノートの詳細」とかのリンクを貼るのでも良さそう |
リノートのリンクのコピーはそこで出来るようになってるので、同じように項目として追加でも実際良さそうですね |
関連してついでぐらいのややオフトピなんだけども連合されてきたリノートを「リモートで見る」したときにリノートに飛べずにActivityのJSONが出るのも気になる |
JSON出るの、CFのキャッシュが云々という話があった気がします(多分別issueあると思うのでそっちに…?) |
What
Why
「詳細画面からリノートの削除ができないのが不便」という声があったため。
(正確には「〇時間前」をクリックした時にリノートではなくリノート元ノートに飛んでいることで起こる現象ですが、不便には違いないと思ってPRを投げました)
Additional info (optional)
Checklist