enhance: ノート詳細ページでリプライ一覧と引用一覧を別々に表示するように #13006
Conversation
github-actions
bot
added
packages/frontend
|
このPRによるapi.jsonの差分 差分はこちら--- base
+++ head
@@ -57740,6 +57740,174 @@
}
}
},
+ "/notes/quotes": {
+ "post": {
+ "operationId": "notes___quotes",
+ "summary": "notes/quotes",
+ "description": "No description provided.\n\n**Credential required**: *No*",
+ "externalDocs": {
+ "description": "Source code",
+ "url": "https://github.com/misskey-dev/misskey/blob/develop/packages/backend/src/server/api/endpoints/notes/quotes.ts"
+ },
+ "tags": [
+ "notes"
+ ],
+ "requestBody": {
+ "required": true,
+ "content": {
+ "application/json": {
+ "schema": {
+ "type": "object",
+ "properties": {
+ "noteId": {
+ "type": "string",
+ "format": "misskey:id"
+ },
+ "sinceId": {
+ "type": "string",
+ "format": "misskey:id"
+ },
+ "untilId": {
+ "type": "string",
+ "format": "misskey:id"
+ },
+ "limit": {
+ "type": "integer",
+ "minimum": 1,
+ "maximum": 100,
+ "default": 10
+ }
+ },
+ "required": [
+ "noteId"
+ ]
+ }
+ }
+ }
+ },
+ "responses": {
+ "200": {
+ "description": "OK (with results)",
+ "content": {
+ "application/json": {
+ "schema": {
+ "type": "array",
+ "items": {
+ "type": "object",
+ "$ref": "#/components/schemas/Note"
+ }
+ }
+ }
+ }
+ },
+ "400": {
+ "description": "Client error",
+ "content": {
+ "application/json": {
+ "schema": {
+ "$ref": "#/components/schemas/Error"
+ },
+ "examples": {
+ "INVALID_PARAM": {
+ "value": {
+ "error": {
+ "message": "Invalid param.",
+ "code": "INVALID_PARAM",
+ "id": "3d81ceae-475f-4600-b2a8-2bc116157532"
+ }
+ }
+ }
+ }
+ }
+ }
+ },
+ "401": {
+ "description": "Authentication error",
+ "content": {
+ "application/json": {
+ "schema": {
+ "$ref": "#/components/schemas/Error"
+ },
+ "examples": {
+ "CREDENTIAL_REQUIRED": {
+ "value": {
+ "error": {
+ "message": "Credential required.",
+ "code": "CREDENTIAL_REQUIRED",
+ "id": "1384574d-a912-4b81-8601-c7b1c4085df1"
+ }
+ }
+ }
+ }
+ }
+ }
+ },
+ "403": {
+ "description": "Forbidden error",
+ "content": {
+ "application/json": {
+ "schema": {
+ "$ref": "#/components/schemas/Error"
+ },
+ "examples": {
+ "AUTHENTICATION_FAILED": {
+ "value": {
+ "error": {
+ "message": "Authentication failed. Please ensure your token is correct.",
+ "code": "AUTHENTICATION_FAILED",
+ "id": "b0a7f5f8-dc2f-4171-b91f-de88ad238e14"
+ }
+ }
+ }
+ }
+ }
+ }
+ },
+ "418": {
+ "description": "I'm Ai",
+ "content": {
+ "application/json": {
+ "schema": {
+ "$ref": "#/components/schemas/Error"
+ },
+ "examples": {
+ "I_AM_AI": {
+ "value": {
+ "error": {
+ "message": "You sent a request to Ai-chan, Misskey's showgirl, instead of the server.",
+ "code": "I_AM_AI",
+ "id": "60c46cd1-f23a-46b1-bebe-5d2b73951a84"
+ }
+ }
+ }
+ }
+ }
+ }
+ },
+ "500": {
+ "description": "Internal server error",
+ "content": {
+ "application/json": {
+ "schema": {
+ "$ref": "#/components/schemas/Error"
+ },
+ "examples": {
+ "INTERNAL_ERROR": {
+ "value": {
+ "error": {
+ "message": "Internal error occurred. Please contact us if the error persists.",
+ "code": "INTERNAL_ERROR",
+ "id": "5d37dbcb-891e-41ca-a3d6-e690c97775ac"
+ }
+ }
+ }
+ }
+ }
+ }
+ }
+ }
+ }
+ },
"/notes/reactions": {
"get": {
"operationId": "notes___reactions", |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #13006 +/- ##
===========================================
+ Coverage 40.12% 41.50% +1.37%
===========================================
Files 1517 1521 +4
Lines 187210 192010 +4800
Branches 3468 3500 +32
===========================================
+ Hits 75118 79693 +4575
- Misses 111523 111747 +224
- Partials 569 570 +1 ☔ View full report in Codecov by Sentry. |
| <div v-if="tab === 'replies'" :class="$style.tab_replies"> | ||
| <MkPagination :pagination="repliesPagination" :disableAutoLoad="true"> | ||
| <template #default="{ items }"> | ||
| <MkNoteSub v-for="item in items" :key="item.id" :note="item" :class="$style.reply" :detail="true"/> | ||
| </template> | ||
| </MkPagination> | ||
| </div> |
There was a problem hiding this comment.
返信が勝手にfetchされるのを防ぎたいので「返信を見る」ボタンは残してほしい（引用RNのほうも同様）
There was a problem hiding this comment.
個人的にはいちいちロードボタンを押すのが面倒なので自動読み込みもできるようにして欲しいです
There was a problem hiding this comment.
利用者的には面倒だろうけど、運営者的には自動ロードされるとちょっと困りそう
There was a problem hiding this comment.
返信タブのもともとの挙動にあわせるのがよさそう
There was a problem hiding this comment.
初期ロードのノート数を少なくした上で初回のロードだけは自動にするか
|
@GrapeApple0 レビューしました🙏🙏 |
これやったらマージ可能 |
やるか |
|
リノートと引用が近いものだと思っているから隣接していて欲しい気持ちがややある（けどこれはもはや好みの問題かも） |
そうする |
|
デザインと挙動を修正したので概ね大丈夫かも
|
|
|
うーむ |
|
コンフリクト解消 |
|
コンフリクト解消 |
|
👀 |
|
コンフリクト解消 |
|
@GrapeApple0 どうされました？ |
|
コンフリクト解消:angry_ai::angry_ai::angry_ai: |
|
/preview |
|
タブがスクロールできることが分からないわね |
それどうにかしようと思っていたけど他コンポーネントでも同じような事態になっているところがあって設計にもう少し検討を要する可能性があったので一旦見送ってる |
|
影に頼るデザインはダークモードと相性が悪い可能性があるわね |
そうでもなさそう？（影の幅を大きくしてlinear-gradient的に使用すれば）
|
|
ヘッダーとかでも同様の問題あるし幅が足りない場合▼とか...で省略するコンポーネント欲しいわね |
これ具体的にどういうUIかしら |
What
ノート詳細ページでリプライ一覧と引用一覧を別々に表示するようにしました
Why
fix: #12116
Additional info (optional)
related: https://misskey.04.si/notes/9oij2d26dn (notes/repliesを使うようにし、paginationを使うようにしたため)
Checklist