Skip to content
This repository was archived by the owner on Nov 21, 2024. It is now read-only.

[Reply] Set container colors for MaterialContainerTransforms and get rid of extra FrameLayout wrapper #42

Merged
merged 1 commit into from
Jun 16, 2020
Merged

[Reply] Set container colors for MaterialContainerTransforms and get rid of extra FrameLayout wrapper #42

merged 1 commit into from
Jun 16, 2020

Conversation

@dsn5ft
Copy link
Contributor

@dsn5ft dsn5ft commented Jun 16, 2020

  • Adds surface container colors to email transform which prevents home screen content from bleeding through
  • Reinforces effect of compose FAB color changing to surface
  • Removes MaterialContainerTransform sharedElementReturnTransition from EmailFragment since it is now the same as the enter transition and will be auto-reversed

reply-container-colors

@dsn5ft dsn5ft requested a review from hunterstich June 16, 2020 18:07
@dsn5ft dsn5ft merged commit af74b67 into develop Jun 16, 2020
@dsn5ft dsn5ft deleted the reply-container-colors branch June 16, 2020 18:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@hunterstich hunterstich hunterstich approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dsn5ft @hunterstich