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

imprv(auditlog): Activity paging UI #6444

Merged
merged 11 commits into from
Aug 19, 2022

Conversation

miya
Copy link
Member

@miya miya commented Aug 9, 2022

Task

#100577 [AuditLog] ページング UI の改善
#102177 改善

ScreenRecord

ScreenShot.2022-08-12.13.59.36.mov

@miya miya self-assigned this Aug 9, 2022
@miya miya changed the title impv: Activity paging UI imprv: Activity paging UI Aug 9, 2022
@miya miya temporarily deployed to VRT August 9, 2022 04:46 Inactive
@miya miya temporarily deployed to VRT August 9, 2022 05:16 Inactive
@miya miya changed the title imprv: Activity paging UI imprv(auditlog): Activity paging UI Aug 12, 2022
@miya miya temporarily deployed to VRT August 12, 2022 05:12 Inactive
}
setJumpPageNum(inputNumber);
}
}, [totalPagingPages]);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ワンライナーで書ける
https://stackoverflow.com/a/5842770

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ワンライナーで書き直しました

type="text"
className="jump-page-input"
onChange={jumpPageInputChangeHandler}
/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Enter キー押下に対応してほしい

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

対応しました。

@miya miya temporarily deployed to VRT August 19, 2022 02:30 Inactive
@miya miya temporarily deployed to VRT August 19, 2022 02:46 Inactive
@reg-suit
Copy link

reg-suit bot commented Aug 19, 2022

reg-suit detected visual differences.

Check this report, and review them.

🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴

🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵

What do the circles mean? The number of circles represent the number of changed images.
🔴 : Changed items, ⚪ : New items, ⚫ : Deleted items, and 🔵 Passed items

How can I change the check status? If reviewers approve this PR, the reg context status will be green automatically.

@Yohei-Shiina Yohei-Shiina merged commit 6f8cf9a into master Aug 19, 2022
@Yohei-Shiina Yohei-Shiina deleted the imprv/102177-activity-paging-ui branch August 19, 2022 03:53
@github-actions github-actions bot mentioned this pull request Aug 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants