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

ノートの詳細表示から閲覧できるノート後のノート履歴の順番がおかしい #12625

Closed
Kur0den opened this issue Dec 11, 2023 · 11 comments · Fixed by #13837
Labels
⚠️bug? This might be a bug

Comments

@Kur0den
Copy link

Kur0den commented Dec 11, 2023

💡 Summary

ノートの詳細表示からそのあとのノート(上矢印のやつ)を見るとノートの順番が新しいものから表示される

🥰 Expected Behavior

そのノート/リノートをした直後の古いノートから表示される
image

🤬 Actual Behavior

最新のノートから順に表示される
image

📝 Steps to Reproduce

  1. ノートの投稿か既存のノートのリノートを行う
  2. そのノートの詳細表示からそのあとのノート(上矢印のやつ)を表示する

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL: koliosky.com
* Misskey: 2023.12.0-beta.3

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:
@Kur0den Kur0den added the ⚠️bug? This might be a bug label Dec 11, 2023
@acid-chicken
Copy link
Member

#12564 (comment) で確認していて #12586 で対応されたものと思っていたけど直ってない感じかしら

@samunohito
Copy link
Member

フロント側で逆順にしている説ないですか…?

@acid-chicken
Copy link
Member

フロント側で逆順にしている説ないですか…?

このパラメーターはページネーション方向(に伴ってページの結合処理方向)のパラメーターじゃなかったかしら

@samunohito
Copy link
Member

samunohito commented Dec 11, 2023

このフラグはMkPaginationにもわたっているんですが、そちらでは仰る通りの実装になっているように見えます。

ただ…中継地点にあるMkDateSeparatedListの中で以下を付与するフラグにも使用されており、これが悪さしてるんじゃないかなと予想してます

flex-direction: column-reverse

なんか分からなくなってきた、後で試します

@samunohito
Copy link
Member

issueの現象を起こす→数ノート投稿する→もっと見るを押下した様子。

積み上がり方もおかしいので、リリース後にゆっくり見ます…

@anatawa12
Copy link
Member

redisTimeline.sort((a, b) => a.id > b.id ? -1 : 1);

ここみてsinceIdだけ指定のときもAPIから戻ってくるtimelineは今>過去だと解釈してたのですが逆順なかんじですか?

@samunohito
Copy link
Member

や、バックエンド側に相違はないと思います。
フロント側の見せ方がちょっとおかしい可能性が大きいです

@tamaina
Copy link
Contributor

tamaina commented Dec 11, 2023

MkPaginationのfetchMoreAheadあたりがおかしい?

@kakkokari-gtyih
Copy link
Contributor

このフラグはMkPaginationにもわたっているんですが、そちらでは仰る通りの実装になっているように見えます。

ただ…中継地点にあるMkDateSeparatedListの中で以下を付与するフラグにも使用されており、これが悪さしてるんじゃないかなと予想してます

flex-direction: column-reverse

なんか分からなくなってきた、後で試します

これな気がする(flex-direction: columnにしたら改善された)

@syuilo
Copy link
Member

syuilo commented Mar 25, 2024

FTT有効時にAPIのレスポンスが期待したものとは逆の順番になることが原因と思われる

@syuilo
Copy link
Member

syuilo commented Mar 25, 2024

sinceIdを指定してリクエストした場合、結果は古いものが先頭に来ることが期待されているけど、FTT有効だとそれとは逆に新しいものが先頭になる

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment