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

Correct the number of activities displayed on the popup #47

Open
rkonno opened this issue Feb 18, 2019 · 1 comment
Open

Correct the number of activities displayed on the popup #47

rkonno opened this issue Feb 18, 2019 · 1 comment

Comments

@rkonno
Copy link
Member

rkonno commented Feb 18, 2019

Popupで表示されるアクティビティ数は最大で50個という仕様だが、49個までしか表示されないため、解決したい。

Context

  • 概要の通り、最大表示数が49個までとなっているため解決したい

Process

以下、調査の見通しです。

  1. ポップアップ側の処理でデータが欠落していないか確認する
  2. client.js 側のデータ取得の処理を確認する
  3. いずれも問題があれば修正する

Expected result

表示されるアクティビティの数は最大で50個

Current result

表示されるアクティビティの数は最大で49個

document.getElementsByClassName("l-histories-list-card-container").length;
// -> 49

popup screenshot

2019-02-18 18 47 53

@rkonno
Copy link
Member Author

rkonno commented Feb 27, 2019

あれ…気のせい?
2019-02-27 17 20 15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant