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

[Based on #766] add equalable content provider #773

Merged
merged 8 commits into from
Feb 3, 2019

Conversation

takahirom
Copy link
Member

@takahirom takahirom commented Feb 3, 2019

Overview (Required)

@jmatsu-bot
Copy link
Collaborator

@takahirom takahirom force-pushed the feature/issue550_add_equalable_content_provider branch from a701c2c to 372307d Compare February 3, 2019 16:48
@@ -98,7 +98,7 @@ class SearchFragment : DaggerFragment() {
result.query
)
}.sortedBy { it.speaker.name.toUpperCase() }
setHeader(HeaderItem(requireContext().getString(R.string.speaker_label)))
setHeader(SearchSectionHeaderItem(requireContext().getString(R.string.speaker_label)))
Copy link
Collaborator

Choose a reason for hiding this comment

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

🚫 Exceeded max line length (100) (cannot be auto-corrected)

@@ -136,7 +136,7 @@ class SearchFragment : DaggerFragment() {
else -> StickyHeaderItemDecoration.DEFAULT_TITLE
}
}
setHeader(HeaderItem(requireContext().getString(R.string.session_title)))
setHeader(SearchSectionHeaderItem(requireContext().getString(R.string.session_title)))
Copy link
Collaborator

Choose a reason for hiding this comment

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

🚫 Exceeded max line length (100) (cannot be auto-corrected)

@jmatsu-bot
Copy link
Collaborator

Apk comparision results

Property Summary
New File Size 8450543 Bytes. (8.06 MB)
File Size Change +7726 Bytes. (+7.54 KB)
Download Size Change +5590 Bytes. (+5.46 KB)
New Method Reference Count 62486
Method Reference Count Change 230
New Number of dex file(s) 1
Number of dex file(s) Change 0

Generated by 🚫 Danger

@jmatsu-bot
Copy link
Collaborator

Asserted successfully. 💯

Generated by 🚫 Danger

@takahirom takahirom merged commit 47862c5 into master Feb 3, 2019
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.

3 participants