Bump to dev08 snapshot build. (android#67)

* Bump to dev08.

Changes:
 * Refactored modifiers to use fluent style throughout
 * Switched to new use Icon and Image composables
 * Switch to using IconButton where appropriate
 * Updated Scaffolding to use IconButton for navigationIcon
 * Removed vector helpers from sample that are now included in the library
 * Update typography of code paraghraphs to expand, and avoid setting background
    on text in code blocks as alpha is not respected – use the background of the    Box
 * Replaced all Container with Box
 * General code cleanup to remove nesting that can be replaced with modifiers
 * Moved Modifiers to *first default argument* to follow style
 * Aligned bullets with baseline of text
 * Sized bullets based on font scaling
 * Removed many named arguments that were not required for readibility
* Added note for bugfix related to font coloring requiring contentColor w/ link to b/143626708
 * Use ColorFilter.tint to semantically tint vectors
 * Continue using Modifier.None for default arugments which is the style in dev08, this is likely to be replaced with `Modifier` in dev09

Author: objcode

JetNews/app/src/androidTest/java/com/example/jetnews/JetnewsUiTest.kt

@@ -17,7 +17,6 @@
 package com.example.jetnews
 
 import androidx.test.filters.MediumTest
-import androidx.test.filters.Suppress
 import androidx.ui.test.assertIsDisplayed
 import androidx.ui.test.createComposeRule
 import androidx.ui.test.doClick
@@ -41,19 +40,11 @@ class JetnewsUiTest {
     }
 
     @Test
-    fun avoidemptyTestSuite() {
-        // this is an empty test to make gradle pass the suite
-        // TODO(b/150728822): remove this after test runner is fixed
-    }
-
-    @Test
-    @Suppress // TODO(b/150728822): re-enabled after test runner is fixed
     fun app_launches() {
         findByText("Jetnews").assertIsDisplayed()
     }
 
     @Test
-    @Suppress // TODO(b/150728822): re-enabled after test runner is fixed
     fun app_opensArticle() {
         findAllBySubstring("Manuel Vivo").first().doClick()
         findAllBySubstring("July 30 • 3 min read").first().assertIsDisplayed()

JetNews/app/src/androidTest/java/com/example/jetnews/TestHelper.kt

@@ -18,7 +18,7 @@ package com.example.jetnews
 
 import androidx.compose.Composable
 import androidx.ui.material.MaterialTheme
-import androidx.ui.material.surface.Surface
+import androidx.ui.material.Surface
 import androidx.ui.test.ComposeTestRule
 import androidx.ui.test.SemanticsNodeInteraction
 import androidx.ui.test.findAll

JetNews/app/src/main/java/com/example/jetnews/model/Post.kt

@@ -16,7 +16,7 @@
 
 package com.example.jetnews.model
 
-import androidx.ui.graphics.Image
+import androidx.ui.graphics.ImageAsset
 
 data class Post(
     val id: String,
@@ -28,8 +28,8 @@ data class Post(
     val paragraphs: List<Paragraph> = emptyList(),
     val imageId: Int,
     val imageThumbId: Int,
-    val image: Image? = null,
-    val imageThumb: Image? = null
+    val image: ImageAsset? = null,
+    val imageThumb: ImageAsset? = null
 )
 
 data class Metadata(

JetNews/app/src/main/java/com/example/jetnews/ui/JetnewsApp.kt

@@ -20,21 +20,24 @@ import androidx.annotation.DrawableRes
 import androidx.compose.Composable
 import androidx.ui.animation.Crossfade
 import androidx.ui.core.Modifier
-import androidx.ui.core.Text
+import androidx.ui.foundation.Image
+import androidx.ui.foundation.Text
 import androidx.ui.foundation.shape.corner.RoundedCornerShape
+import androidx.ui.graphics.ColorFilter
 import androidx.ui.layout.Arrangement
 import androidx.ui.layout.Column
-import androidx.ui.layout.LayoutGravity
-import androidx.ui.layout.LayoutHeight
-import androidx.ui.layout.LayoutPadding
-import androidx.ui.layout.LayoutSize
-import androidx.ui.layout.LayoutWidth
 import androidx.ui.layout.Row
 import androidx.ui.layout.Spacer
+import androidx.ui.layout.fillMaxSize
+import androidx.ui.layout.fillMaxWidth
+import androidx.ui.layout.padding
+import androidx.ui.layout.preferredHeight
+import androidx.ui.layout.preferredWidth
 import androidx.ui.material.Divider
 import androidx.ui.material.MaterialTheme
+import androidx.ui.material.Surface
 import androidx.ui.material.TextButton
-import androidx.ui.material.surface.Surface
+import androidx.ui.res.vectorResource
 import androidx.ui.tooling.preview.Preview
 import androidx.ui.unit.dp
 import com.example.jetnews.R
@@ -56,7 +59,7 @@ fun JetnewsApp() {
 @Composable
 private fun AppContent() {
     Crossfade(JetnewsStatus.currentScreen) { screen ->
-        Surface(color = MaterialTheme.colors().background) {
+        Surface(color = MaterialTheme.colors.background) {
             when (screen) {
                 is Screen.Home -> HomeScreen()
                 is Screen.Interests -> InterestsScreen()
@@ -71,49 +74,61 @@ fun AppDrawer(
     currentScreen: Screen,
     closeDrawer: () -> Unit
 ) {
-    Column(modifier = LayoutSize.Fill) {
-        Spacer(LayoutHeight(24.dp))
-        Row(modifier = LayoutPadding(16.dp)) {
-            VectorImage(
-                id = R.drawable.ic_jetnews_logo,
-                tint = MaterialTheme.colors().primary
-            )
-            Spacer(LayoutWidth(8.dp))
-            VectorImage(
-                id = R.drawable.ic_jetnews_wordmark,
-                tint = MaterialTheme.colors().onSurface
-            )
-        }
-        Divider(color = MaterialTheme.colors().onSurface.copy(alpha = .2f))
+    Column(modifier = Modifier.fillMaxSize()) {
+        Spacer(Modifier.preferredHeight(24.dp))
+        JetNewsLogo(Modifier.padding(16.dp))
+        Divider(color = MaterialTheme.colors.onSurface.copy(alpha = .2f))
         DrawerButton(
             icon = R.drawable.ic_home,
             label = "Home",
-            isSelected = currentScreen == Screen.Home
-        ) {
-            navigateTo(Screen.Home)
-            closeDrawer()
-        }
+            isSelected = currentScreen == Screen.Home,
+            action = {
+                navigateTo(Screen.Home)
+                closeDrawer()
+            }
+        )
 
         DrawerButton(
             icon = R.drawable.ic_interests,
             label = "Interests",
-            isSelected = currentScreen == Screen.Interests
-        ) {
-            navigateTo(Screen.Interests)
-            closeDrawer()
-        }
+            isSelected = currentScreen == Screen.Interests,
+            action = {
+                navigateTo(Screen.Interests)
+                closeDrawer()
+            }
+        )
+    }
+}
+
+@Composable
+private fun JetNewsLogo(modifier: Modifier = Modifier.None) {
+    Row(modifier = modifier) {
+        Image(
+            asset = vectorResource(R.drawable.ic_jetnews_logo),
+            colorFilter = ColorFilter.tint(MaterialTheme.colors.primary)
+        )
+        Spacer(Modifier.preferredWidth(8.dp))
+        Image(
+            asset = vectorResource(R.drawable.ic_jetnews_wordmark),
+            colorFilter = ColorFilter.tint(MaterialTheme.colors.onSurface)
+        )
     }
 }
 
 @Composable
 private fun DrawerButton(
-    modifier: Modifier = Modifier.None,
     @DrawableRes icon: Int,
     label: String,
     isSelected: Boolean,
-    action: () -> Unit
+    action: () -> Unit,
+    modifier: Modifier = Modifier.None
 ) {
-    val colors = MaterialTheme.colors()
+    val colors = MaterialTheme.colors
+    val imageAlpha = if (isSelected) {
+        1f
+    } else {
+        0.6f
+    }
     val textIconColor = if (isSelected) {
         colors.primary
     } else {
@@ -125,28 +140,26 @@ private fun DrawerButton(
         colors.surface
     }
 
-    val surfaceModifier = modifier +
-            LayoutPadding(start = 8.dp, top = 8.dp, end = 8.dp, bottom = 0.dp) +
-            LayoutWidth.Fill
+    val surfaceModifier = modifier
+        .padding(start = 8.dp, top = 8.dp, end = 8.dp)
+        .fillMaxWidth()
     Surface(
         modifier = surfaceModifier,
         color = backgroundColor,
         shape = RoundedCornerShape(4.dp)
     ) {
-        TextButton(onClick = action, modifier = LayoutWidth.Fill) {
-            Row(arrangement = Arrangement.Start, modifier = LayoutWidth.Fill) {
-                VectorImage(
-                    modifier = LayoutGravity.Center,
-                    id = icon,
-                    tint = textIconColor
+        TextButton(onClick = action, modifier = Modifier.fillMaxWidth()) {
+            Row(arrangement = Arrangement.Start, modifier = Modifier.fillMaxWidth()) {
+                Image(
+                    asset = vectorResource(icon),
+                    colorFilter = ColorFilter.tint(textIconColor),
+                    alpha = imageAlpha
                 )
-                Spacer(LayoutWidth(16.dp))
+                Spacer(Modifier.preferredWidth(16.dp))
                 Text(
                     text = label,
-                    style = (MaterialTheme.typography()).body2.copy(
-                        color = textIconColor
-                    ),
-                    modifier = LayoutWidth.Fill
+                    style = MaterialTheme.typography.body2.copy(color = textIconColor),
+                    modifier = Modifier.fillMaxWidth()
                 )
             }
         }

JetNews/app/src/main/java/com/example/jetnews/ui/PreviewUtils.kt

@@ -19,8 +19,8 @@ package com.example.jetnews.ui
 import androidx.compose.Composable
 import androidx.ui.material.ColorPalette
 import androidx.ui.material.MaterialTheme
+import androidx.ui.material.Surface
 import androidx.ui.material.Typography
-import androidx.ui.material.surface.Surface
 
 @Composable
 internal fun ThemedPreview(