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

LayoutInflater extensions #423

Open
jaredsburrows wants to merge 1 commit into
base: master
Choose a base branch
from
Open

LayoutInflater extensions #423

jaredsburrows wants to merge 1 commit into from

Conversation

@jaredsburrows
Copy link
Contributor

@jaredsburrows jaredsburrows commented Mar 10, 2018 •
edited
Loading

@jaredsburrows
Copy link
Contributor Author

jaredsburrows commented Mar 10, 2018

Im wondering if I should include the following: #162 (comment)?

@AlexKrupa
Copy link

AlexKrupa commented Mar 13, 2018

Shouldn't the file's name be LayoutInflater.kt instead of LayoutInflator.kt?

@romainguy
Copy link
Contributor

romainguy commented Mar 13, 2018

It should be LayoutInflater yes.

@jaredsburrows
Copy link
Contributor Author

jaredsburrows commented Mar 13, 2018

@AlexKrupa Thanks.
@romainguy Fixed.

@jaredsburrows jaredsburrows changed the title LayoutInflator extensions LayoutInflater extensions Mar 13, 2018
indyfromoz
indyfromoz reviewed Apr 4, 2018
View reviewed changes
Copy link

@indyfromoz indyfromoz left a comment

Choose a reason for hiding this comment

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

👍

@jaredsburrows
Copy link
Contributor Author

jaredsburrows commented May 25, 2018

@romainguy @JakeWharton Review :)

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

Reviewers

2 more reviewers

@indyfromoz indyfromoz indyfromoz left review comments

@hluhovskyi hluhovskyi hluhovskyi approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jaredsburrows @AlexKrupa @romainguy @indyfromoz @hluhovskyi