Skip to content

Convert Plaid to use AndroidX #524

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.

Sign up for GitHub

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

Merged
merged 17 commits into from
Oct 21, 2018
Merged

Convert Plaid to use AndroidX #524

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
dd2ebf7
Initial commit after AndroidX conversion using Android Studio refacto…
tiembo Oct 12, 2018
6b23efa
Bump Glide to 4.8.0, which supports AndroidX annotations
tiembo Oct 12, 2018
8e60a35
Fix dex error with PathSegment
tiembo Oct 12, 2018
75b8ffd
Fix error with type mismatch
tiembo Oct 12, 2018
d2cefa3
Temporarily comment out plusAssign usage. App runs now
tiembo Oct 12, 2018
3a038ef
Clean up imports
tiembo Oct 12, 2018
aba91a9
Attempt to fix core-runtime CI error
tiembo Oct 12, 2018
b38f2cf
Upgrade Kotlin and Gradle wrapper versions
tiembo Oct 12, 2018
0c5a3d6
Extract versions out of build.gradle files
tiembo Oct 12, 2018
44dc2ba
Address PR comments from Nick
tiembo Oct 17, 2018
0a5a5a3
Address PR comment from Florina
tiembo Oct 17, 2018
a520c32
Rename version variable names
tiembo Oct 17, 2018
7aad300
Update palette params for applyFullImage and applyTopPalette
tiembo Oct 17, 2018
27fbae9
Fix format violations
tiembo Oct 17, 2018
9715e2f
Workaround for dependency conflict during assembleAndroidTest
tiembo Oct 17, 2018
a997200
Fix instrumentation tests
tiembo Oct 21, 2018
4109a57
Update Kotlin to 1.3.0-rc-190
tiembo Oct 21, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update palette params for applyFullImage and applyTopPalette
  • Loading branch information
@tiembo
tiembo committed Oct 17, 2018
commit 7aad3005ea8ab222443f1a2ea34252e25328d677
8 changes: 4 additions & 4 deletions dribbble/src/main/java/io/plaidapp/dribbble/ui/shot/ShotActivity.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ class ShotActivity : AppCompatActivity() {
Palette.from(bitmap)
.clearFilters() /* by default palette ignore certain hues
(e.g. pure black/white) but we don't want this. */
.generate { palette -> applyFullImagePalette(palette!!) }
.generate { palette -> applyFullImagePalette(palette) }

val twentyFourDip = TypedValue.applyDimension(
TypedValue.COMPLEX_UNIT_DIP,
Expand All @@ -98,7 +98,7 @@ class ShotActivity : AppCompatActivity() {
.clearFilters()
.setRegion(0, 0, bitmap.width - 1, twentyFourDip) /* - 1 to work around
https://code.google.com/p/android/issues/detail?id=191013 */
.generate { palette -> applyTopPalette(bitmap, palette!!) }
.generate { palette -> applyTopPalette(bitmap, palette) }

// TODO should keep the background if the image contains transparency?!
binding.shot.background = null
Expand Down Expand Up @@ -252,7 +252,7 @@ class ShotActivity : AppCompatActivity() {
}
}

internal fun applyFullImagePalette(palette: Palette) {
internal fun applyFullImagePalette(palette: Palette?) {
// color the ripple on the image spacer (default is grey)
binding.shotSpacer.background = ViewUtils.createRipple(
palette, 0.25f, 0.5f,
Expand All @@ -265,7 +265,7 @@ class ShotActivity : AppCompatActivity() {
)
}

internal fun applyTopPalette(bitmap: Bitmap, palette: Palette) {
internal fun applyTopPalette(bitmap: Bitmap, palette: Palette?) {
val lightness = ColorUtils.isDark(palette)
val isDark = if (lightness == ColorUtils.LIGHTNESS_UNKNOWN) {
ColorUtils.isDark(bitmap, bitmap.width / 2, 0)
Expand Down