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

Add test for EdgeButton and IconButton padding in TransformingLazyColumn #2455

Merged
merged 10 commits into from
Nov 8, 2024

Conversation

yschimke
Copy link
Collaborator

@yschimke yschimke commented Nov 8, 2024

WHAT

This commit adds a new test case to verify the correct padding behavior of TransformingLazyColumn when used with EdgeButton and IconButton. The test renders a TransformingLazyColumn with an EdgeButton and an IconButton and checks the top/bottom padding applied to the column content. Additionally, the test ensures that scrolling to the end of the column works as expected.

Screenshots are updated to reflect the new test case.

image

image

WHY

HOW

Checklist 📋

  • Add explicit visibility modifier and explicit return types for public declarations
  • Run spotless check
  • Run tests
  • Update metalava's signature text files

This commit adds a new test case to verify the correct padding behavior of `TransformingLazyColumn` when used with `EdgeButton` and `IconButton`.
The test renders a `TransformingLazyColumn` with an `EdgeButton` and an `IconButton` and checks the top/bottom padding applied to the column content.
Additionally, the test ensures that scrolling to the end of the column works as expected.

Screenshots are updated to reflect the new test case.
@yschimke yschimke requested a review from BowerSteve November 8, 2024 11:16
@yschimke yschimke requested a review from garanj November 8, 2024 12:36
@yschimke yschimke closed this Nov 8, 2024
@yschimke yschimke reopened this Nov 8, 2024
@yschimke yschimke closed this Nov 8, 2024
@yschimke yschimke reopened this Nov 8, 2024
@yschimke yschimke merged commit 5c4a280 into google:main Nov 8, 2024
6 checks passed
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

Successfully merging this pull request may close these issues.

2 participants